example_test.py: fixup coinbase height argument, derive number clearly #14664

pull instagibbs wants to merge 1 commits into bitcoin:master from instagibbs:example_cleanup changing 1 files +2 −2
  1. instagibbs commented at 6:19 PM on November 5, 2018: member

    No description provided.

  2. example_test.py: fixup coinbase height argument, derive number clearly 6c5355e43d
  3. MarcoFalke commented at 6:34 PM on November 5, 2018: member

    utACK 6c5355e43d7c68c9ba71bc3170b15a6cf96bb9b4

  4. promag commented at 6:45 PM on November 5, 2018: member

    utACK 6c5355e.

  5. MarcoFalke added the label Tests on Nov 5, 2018
  6. MarcoFalke added the label Refactoring on Nov 5, 2018
  7. in test/functional/example_test.py:167 in 6c5355e43d
     163 | @@ -164,13 +164,13 @@ def run_test(self):
     164 |          self.tip = int(self.nodes[0].getbestblockhash(), 16)
     165 |          self.block_time = self.nodes[0].getblock(self.nodes[0].getbestblockhash())['time'] + 1
     166 |  
     167 | -        height = 1
     168 | +        height = self.nodes[0].getblockcount()
    


    MarcoFalke commented at 6:54 PM on November 5, 2018:

    Actually the + 1 should be suffixed here (similar to block_time above?)

    And then remove the change further down?

            height = self.nodes[0].getblockcount() + 1
    

    instagibbs commented at 6:58 PM on November 5, 2018:

    2+10=12, and we're waiting for block 11, I'd have to change that too.


    MarcoFalke commented at 7:03 PM on November 5, 2018:

    Ok, fine.

  8. MarcoFalke assigned MarcoFalke on Nov 5, 2018
  9. MarcoFalke merged this on Nov 5, 2018
  10. MarcoFalke closed this on Nov 5, 2018

  11. MarcoFalke referenced this in commit 9c3f6c5fad on Nov 5, 2018
  12. jasonbcox referenced this in commit 438ec29b07 on Oct 21, 2020
  13. PastaPastaPasta referenced this in commit 1aa7cbd052 on Jun 27, 2021
  14. PastaPastaPasta referenced this in commit 84975a3588 on Jun 28, 2021
  15. PastaPastaPasta referenced this in commit f24f228e7a on Jun 29, 2021
  16. PastaPastaPasta referenced this in commit afc96d7c2b on Jul 1, 2021
  17. PastaPastaPasta referenced this in commit 358ed23765 on Jul 1, 2021
  18. PastaPastaPasta referenced this in commit a0dd421a77 on Jul 1, 2021
  19. PastaPastaPasta referenced this in commit ca000ff281 on Jul 3, 2021
  20. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-22 06:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me