doc: Describe log files + consistent paths in test READMEs #17177

pull fjahr wants to merge 1 commits into bitcoin:master from fjahr:pr15830 changing 2 files +31 −19

fjahr commented at 2:55 PM on October 17, 2019: member

picks up #15830

I saw this was almost ready to merge but the test logging part was not 100% correct. I reworked that part, the rest is the same.
fanquake added the label Docs on Oct 17, 2019
doc: Describe log files + consistent paths in test READMEs 9576614d2d
fjahr force-pushed on Oct 17, 2019
GChuf commented at 6:17 PM on October 17, 2019: contributor

ACK 9576614d2d91ca946d164dfffca441f8bcbd6e2c
fanquake requested review from MarcoFalke on Oct 17, 2019

in test/README.md:133 in 9576614d2d

 129 | @@ -130,7 +130,7 @@ tests will fail. If this happens, remove the cache directory (and make
 130 |  sure bitcoind processes are stopped as above):
 131 |  
 132 |  ```bash
 133 | -rm -rf cache
 134 | +rm -rf test/cache

MarcoFalke commented at 6:35 PM on October 17, 2019:

This is only created for tests that have self.setup_clean_chain = False set

MarcoFalke commented at 6:35 PM on October 17, 2019: member

ACK
MarcoFalke referenced this in commit 88eff969c2 on Oct 17, 2019
MarcoFalke merged this on Oct 17, 2019
MarcoFalke closed this on Oct 17, 2019
Fabcien referenced this in commit 09d8cd8486 on Aug 24, 2021
DrahtBot locked this on Dec 16, 2021

Contributors

Labels

Linked (view graph)

#15830 docs: Improvements to two READMEs in /test