a negative number of confirmations?

Hello,

I am currently testing my app built with bit. I’m using the bitcoin testnet and I noticed the function .get_balance took in account unconfirmed transactions. I would like the transactions to have at least one confirmation (and if possible a configurable number) to be counted in the balance. I then tried to create my own function:

    def fetch_balance(self):
        balance = 0
        for unspent in self.key.get_unspents():
            print(unspent)
            if unspent.confirmations >= BitcoinTicket.validations:
                balance += unspent.amount
        return balance

Here is what I got from the print:

Unspent(amount=50000, confirmations=-1805169, script='a914b98e3212e5afe4363942ce9b7b21c7057241782f87', txid='d594eb22bafcb8363271734e6d2e71d35e72cffc635e64f4f7b840ca605f6283', txindex=0, segwit=True)
Unspent(amount=1, confirmations=-1804929, script='a914b98e3212e5afe4363942ce9b7b21c7057241782f87', txid='194e7b59a8ea2fa1ce21664a81784dc7f7cf0cde0892df2356ff6da34b14f314', txindex=0, segwit=True)

According to this website: https://blockstream.info/testnet/address/2NAAMLhL3gGsPb67LwZ3bgPkbX93j8e3Vn9 I should have get 6 and 246 confirmations. What does that mean? Is this only for testnet?

[pedr0-fr]

I think you can leave it open as it will be closed when #129 is merged

[pedr0-fr]

@bjarnemagnussen The PR I submitted (#129) just now should fix it. Also noticed that the confirmations from the BlockstreamAPI were off by one.

@Th0rgal , as @bjarnemagnussen said, that’s a bug that won’t occur on mainnet due to #120 . For now, just assume that a negative number of confirmations means zero confirmations.