test: Explain that a bug should be filed when the tests fail #19110

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2005-testBug changing 2 files +6 −0
  1. MarcoFalke commented at 7:33 pm on May 29, 2020: member
    Without a bug report it is harder to fix the issue
  2. test: Explain that a bug should be filed when the test fail fad21a1a7a
  3. MarcoFalke force-pushed on May 29, 2020
  4. DrahtBot added the label Build system on May 29, 2020
  5. DrahtBot added the label Tests on May 29, 2020
  6. hebasto approved
  7. hebasto commented at 6:33 am on May 30, 2020: member
    ACK fad21a1a7aa8804f4699e5821f074f5d3845c78b, I have not tested the code, but I have reviewed it and it looks OK, I agree it can be merged.
  8. fanquake approved
  9. fanquake commented at 8:20 am on May 30, 2020: member

    ACK fad21a1a7aa8804f4699e5821f074f5d3845c78b

     0    assert_equal(self.nodes[0].getbestblockhash(), block.blahblah)
     1AttributeError: 'CBlock' object has no attribute 'blahblah'
     22020-05-30T07:58:41.461000Z TestFramework (INFO): Stopping nodes
     32020-05-30T07:58:41.724000Z TestFramework (WARNING): Not cleaning up dir /var/folders/z2/cn877pxd3czdfh47mfkmbwgm0000gn/T/test_runner_₿_🏃_20200530_155836/feature_cltv_0
     42020-05-30T07:58:41.724000Z TestFramework (ERROR): Test failed. Test logging available at /var/folders/z2/cn877pxd3czdfh47mfkmbwgm0000gn/T/test_runner_₿_🏃_20200530_155836/feature_cltv_0/test_framework.log
     52020-05-30T07:58:41.725000Z TestFramework (ERROR): 
     62020-05-30T07:58:41.725000Z TestFramework (ERROR): Hint: Call /bitcoin/test/functional/combine_logs.py '/var/folders/z2/cn877pxd3czdfh47mfkmbwgm0000gn/T/test_runner_₿_🏃_20200530_155836/feature_cltv_0' to consolidate all logs
     72020-05-30T07:58:41.725000Z TestFramework (ERROR): 
     82020-05-30T07:58:41.725000Z TestFramework (ERROR): If this failure happened unexpectedly or intermittently, please file a bug and provide a link or upload of the combined log.
     92020-05-30T07:58:41.725000Z TestFramework (ERROR): https://github.com/bitcoin/bitcoin/issues
    102020-05-30T07:58:41.725000Z TestFramework (ERROR): 
    
  10. MarcoFalke merged this on May 30, 2020
  11. MarcoFalke closed this on May 30, 2020

  12. MarcoFalke deleted the branch on May 30, 2020
  13. hebasto commented at 5:56 am on June 2, 2020: member

    @MarcoFalke How to preserve the travis log with an intermittent error for issue submitting? e.g., https://travis-ci.org/github/bitcoin/bitcoin/jobs/692067912#L5040

    If I re-run build the log pointed by url will be re-written. Re-submit the latest commit?

  14. MarcoFalke commented at 10:29 am on June 2, 2020: member
    Yes, only way is to (force) push or re-open or download manually :( See #15847 (comment)
  15. Fabcien referenced this in commit db540ee053 on Feb 24, 2021
  16. PastaPastaPasta referenced this in commit e7955bea24 on Dec 22, 2021
  17. PastaPastaPasta referenced this in commit 8a5c30d212 on Dec 22, 2021
  18. PastaPastaPasta referenced this in commit e7ae05ee39 on Dec 22, 2021
  19. PastaPastaPasta referenced this in commit baebdd2768 on Dec 28, 2021
  20. DrahtBot locked this on Feb 15, 2022

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-09-29 01:12 UTC

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