diff --git a/bitcoind-rpc-test/src/test/scala/org/bitcoins/rpc/v17/BitcoindV17RpcClientTest.scala b/bitcoind-rpc-test/src/test/scala/org/bitcoins/rpc/v17/BitcoindV17RpcClientTest.scala index b81a4b46d6..1d7108a49b 100644 --- a/bitcoind-rpc-test/src/test/scala/org/bitcoins/rpc/v17/BitcoindV17RpcClientTest.scala +++ b/bitcoind-rpc-test/src/test/scala/org/bitcoins/rpc/v17/BitcoindV17RpcClientTest.scala @@ -16,7 +16,6 @@ import org.bitcoins.rpc.client.v17.BitcoindV17RpcClient import org.bitcoins.rpc.util.AsyncUtil import org.bitcoins.testkit.rpc.BitcoindRpcTestUtil import org.bitcoins.testkit.util.BitcoindRpcTest -import java.time.LocalDateTime import scala.concurrent.Future @@ -108,13 +107,16 @@ class BitcoindV17RpcClientTest extends BitcoindRpcTest { } yield assert(signed.complete) } + private val SpreadInSeconds = 30 + it should "be able to get the address info for a given address" in { for { (client, _) <- clientsF addr <- client.getNewAddress info <- client.getAddressInfo(addr) } yield assert( - info.timestamp.exists(_.getDayOfYear == LocalDateTime.now.getDayOfYear)) + info.timestamp.map(_.toEpochSecond).getOrElse(0L) === System + .currentTimeMillis() / 1000 +- SpreadInSeconds) } it should "be able to get the address info for a given P2SHSegwit address" in { @@ -123,7 +125,8 @@ class BitcoindV17RpcClientTest extends BitcoindRpcTest { addr <- client.getNewAddress(addressType = AddressType.P2SHSegwit) info <- client.getAddressInfo(addr) } yield assert( - info.timestamp.exists(_.getDayOfYear == LocalDateTime.now.getDayOfYear)) + info.timestamp.map(_.toEpochSecond).getOrElse(0L) === System + .currentTimeMillis() / 1000 +- SpreadInSeconds) } it should "be able to get the address info for a given Legacy address" in { @@ -132,7 +135,8 @@ class BitcoindV17RpcClientTest extends BitcoindRpcTest { addr <- client.getNewAddress(addressType = AddressType.Legacy) info <- client.getAddressInfo(addr) } yield assert( - info.timestamp.exists(_.getDayOfYear == LocalDateTime.now.getDayOfYear)) + info.timestamp.map(_.toEpochSecond).getOrElse(0L) === System + .currentTimeMillis() / 1000 +- SpreadInSeconds) } // needs #360 to be merged @@ -144,7 +148,8 @@ class BitcoindV17RpcClientTest extends BitcoindRpcTest { } yield { assert(info.address.networkParameters == RegTest) assert( - info.timestamp.exists(_.getDayOfYear == LocalDateTime.now.getDayOfYear)) + info.timestamp.map(_.toEpochSecond).getOrElse(0L) === System + .currentTimeMillis() / 1000 +- SpreadInSeconds) } } diff --git a/bitcoind-rpc/src/main/scala/org/bitcoins/rpc/jsonmodels/WalletResult.scala b/bitcoind-rpc/src/main/scala/org/bitcoins/rpc/jsonmodels/WalletResult.scala index c69b4b52e9..6df31b0220 100644 --- a/bitcoind-rpc/src/main/scala/org/bitcoins/rpc/jsonmodels/WalletResult.scala +++ b/bitcoind-rpc/src/main/scala/org/bitcoins/rpc/jsonmodels/WalletResult.scala @@ -1,7 +1,7 @@ package org.bitcoins.rpc.jsonmodels import java.io.File -import java.time.LocalDateTime +import java.time.ZonedDateTime import org.bitcoins.core.crypto.{ DoubleSha256DigestBE, @@ -221,7 +221,7 @@ case class AddressInfoResult( pubkey: Option[ECPublicKey], embedded: Option[EmbeddedResult], label: String, - timestamp: Option[LocalDateTime], + timestamp: Option[ZonedDateTime], hdkeypath: Option[BIP32Path], hdseedid: Option[RipeMd160Digest], hdmasterkeyid: Option[RipeMd160Digest],