tests: Possible fix the permission error when the tests open the cookie file #14788

1. 
   ken2812221 commented at 8:08 pm on November 22, 2018: contributor

   This PR would wait until the .cookie file is readable Possible fix no. 5 PermissionError in #14446
2. 
   MarcoFalke commented at 8:49 pm on November 22, 2018: member

   utACK 0cb4518d355738b2530a66cded2fc4b673ba1c2a
3. MarcoFalke added the label Tests on Nov 22, 2018
4. 
   laanwj commented at 6:55 am on November 23, 2018: member

   There’s so much polling and waiting in the tests; from a determinism point of view I’d prefer to do this waiting once after starting a node, this is the only time where one’d expect the cookie to possibly not be there yet (maybe make it part of the current poll-after-run loop).

   (If the same happens and it’s trying to find the cookie after shutdown this is a problem with the test and it shouldn’t simply wait 10 seconds. Or if the cookie goes missing while a node is supposed to be running this is a problem with the software that should be detected immediately.)

5. 
   promag commented at 3:19 pm on November 23, 2018: member

   Another error https://ci.appveyor.com/project/DrahtBot/bitcoin/builds/20508203.
6. tests: check readability of cookie file d6b3790d1a
7. ken2812221 force-pushed on Nov 23, 2018
8. 
   ken2812221 commented at 6:35 pm on November 23, 2018: contributor

   @laanwj Now it would also raise ValueError if the cookie file is not readable. It can be caught in wait_for_rpc_connection()
9. 
   MarcoFalke commented at 5:47 pm on December 6, 2018: member

   utACK d6b3790d1a41030f4ac83556fdbe9f019b9b4e59
10. MarcoFalke merged this on Dec 7, 2018
11. MarcoFalke closed this on Dec 7, 2018

    ---

12. MarcoFalke referenced this in commit 2753285be7 on Dec 7, 2018
13. ken2812221 deleted the branch on Dec 8, 2018
14. Munkybooty referenced this in commit bb975fbc84 on Aug 2, 2021
15. Munkybooty referenced this in commit b9b5e75962 on Aug 3, 2021
16. Munkybooty referenced this in commit bac4b6dc77 on Aug 5, 2021
17. Munkybooty referenced this in commit 6a1747bb5e on Aug 5, 2021
18. Munkybooty referenced this in commit 563ad654c4 on Aug 8, 2021
19. Munkybooty referenced this in commit 7bbd22c68d on Aug 11, 2021
20. Munkybooty referenced this in commit f089273ed0 on Aug 11, 2021
21. Munkybooty referenced this in commit b92a8c2019 on Aug 13, 2021
22. Munkybooty referenced this in commit 29ba5a37de on Aug 15, 2021
23. DrahtBot locked this on Sep 8, 2021

Contributors
ken2812221 MarcoFalke laanwj promag

Labels
Tests