From 7c7abd26b17280be520ea812c7453abc0697f56a Mon Sep 17 00:00:00 2001 From: Chris Kleeschulte Date: Mon, 21 Sep 2015 16:11:49 -0400 Subject: [PATCH] isSpent logic - Bitcoind's CCoinsViewMempool brings the mempool into view, but will not consider outputs therein as spent - Changed the test description to match what is happening in that view - Once a given tx has one confirmation, then isSpent will be true for those outputs --- integration/regtest-node.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/integration/regtest-node.js b/integration/regtest-node.js index c22ce171..c3d38af2 100644 --- a/integration/regtest-node.js +++ b/integration/regtest-node.js @@ -723,7 +723,8 @@ describe('Node Functionality', function() { result.should.equal(true); done(); }); - it('will incorrectly return false for an input that is spent in an unconfirmed transaction', function(done) { + //CCoinsViewMemPool only checks for spent outputs that are not the mempool + it('will correctly return false for an input that is spent in an unconfirmed transaction', function(done) { node.services.address.getUnspentOutputs(address, false, function(err, results) { if (err) { throw err;