[qa] Remove misleading “errorString syntax” #7801

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1604-qaTestErrorString changing 3 files +16 −19
  1. MarcoFalke commented at 7:37 pm on April 3, 2016: member

    Personally, I find the errorString “syntax” to “see” when an exception happens confusing. It is error prone in the best case.

    Imo, valid syntax is either

    0try:
    1  something()
    2  assert(False)
    3except:
    4  assert(thing())
    

    or

    0try:
    1  something()
    2except:
    3  assert(thing())
    4else:
    5  assert(False)
    
  2. [qa] Remove misleading "errorString syntax" ffff866da8
  3. jonasschnelli added the label Tests on Apr 4, 2016
  4. laanwj commented at 12:12 pm on April 6, 2016: member
    ACK ffff866
  5. laanwj merged this on Apr 6, 2016
  6. laanwj closed this on Apr 6, 2016

  7. laanwj referenced this in commit 70ac71b877 on Apr 6, 2016
  8. MarcoFalke deleted the branch on Apr 6, 2016
  9. MarcoFalke referenced this in commit 28ba22c202 on Apr 19, 2016
  10. nomnombtc referenced this in commit 5e759241a9 on Nov 11, 2016
  11. nomnombtc referenced this in commit c734807b0b on Nov 12, 2016
  12. nomnombtc referenced this in commit d039ef7b8d on Nov 13, 2016
  13. sickpig referenced this in commit 12cfa323b3 on Nov 14, 2016
  14. codablock referenced this in commit 0c2b3fbf52 on Sep 16, 2017
  15. codablock referenced this in commit 4932418ee7 on Sep 19, 2017
  16. codablock referenced this in commit a8aeff143b on Dec 20, 2017
  17. MarcoFalke locked this on Sep 8, 2021


MarcoFalke laanwj

Labels
Tests


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: 2024-11-17 12:12 UTC

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