Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Dec 9, 2017

  • Pass sys.executable when calling a python script via the subprocess
    module
  • Don't remove the log file while it is still open and written to
  • Properly use os.pathsep and os.path.sep when modifying the PATH
    environment variable
  • util-tests: Use os.path.join for Windows compatibility

Ref: #8227

@maflcko maflcko added the Tests label Dec 9, 2017
@maflcko maflcko force-pushed the Mf1712-qaWinForRealNow branch from fae1850 to fa39994 Compare December 9, 2017 23:45
@laanwj
Copy link
Member

laanwj commented Dec 12, 2017

Thanks for resurrecting this. utACK fa39994c2b68912276a7274c08745f864b1cd7f2

@maflcko
Copy link
Member Author

maflcko commented Dec 30, 2017

Added a commit to make test/util work on Windows as well.

@maflcko maflcko changed the title qa: Prepare functional tests for Windows (laanwj, MarcoFalke) qa: Prepare tests for Windows (laanwj, MarcoFalke) Dec 30, 2017
@maflcko maflcko changed the title qa: Prepare tests for Windows (laanwj, MarcoFalke) qa: Prepare tests for Windows Jan 24, 2018
@maflcko maflcko force-pushed the Mf1712-qaWinForRealNow branch 3 times, most recently from 0dc20f0 to 653ad17 Compare January 24, 2018 20:51
* Pass `sys.executable` when calling a python script via the subprocess
  module
* Don't remove the log file while it is still open and written to
* Properly use os.pathsep and os.path.sep when modifying the PATH
  environment variable
* util-tests: Use os.path.join for Windows compatibility
@maflcko maflcko force-pushed the Mf1712-qaWinForRealNow branch from 653ad17 to faefd29 Compare January 24, 2018 20:52
@maflcko
Copy link
Member Author

maflcko commented Jan 24, 2018

Rebased

@laanwj laanwj merged commit faefd29 into bitcoin:master Feb 12, 2018
laanwj added a commit that referenced this pull request Feb 12, 2018
faefd29 qa: Prepare functional tests for Windows (MarcoFalke)

Pull request description:

  * Pass `sys.executable` when calling a python script via the subprocess
    module
  * Don't remove the log file while it is still open and written to
  * Properly use os.pathsep and os.path.sep when modifying the PATH
    environment variable
  * util-tests: Use os.path.join for Windows compatibility

  Ref:  #8227

Tree-SHA512: c507a536af104b3bde4366b6634099db826532bd3e7c35d694b5883c550920643b3eab79c76703ca67e1044ed09979e855088f7324321c8d52112514e334d614
@maflcko maflcko deleted the Mf1712-qaWinForRealNow branch February 12, 2018 12:50
maflcko pushed a commit that referenced this pull request Feb 15, 2018
ada1af6 Fix typo in test_runner.py causing error (MeshCollider)

Pull request description:

  In the case that a test fails, the typo in run_tests() (introduced in #11858) will cause an error rather than printing out the combined logs, hiding the cause of the failure.

Tree-SHA512: 7d7aa406d92750320ed20610cc5f174cdc94086f630af8c0c4db2003497132e0c56d59b94312fb42ad4507904a2fa858226a4a9337450930bf206183fc35c0a0
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 17, 2020
faefd29 qa: Prepare functional tests for Windows (MarcoFalke)

Pull request description:

  * Pass `sys.executable` when calling a python script via the subprocess
    module
  * Don't remove the log file while it is still open and written to
  * Properly use os.pathsep and os.path.sep when modifying the PATH
    environment variable
  * util-tests: Use os.path.join for Windows compatibility

  Ref:  bitcoin#8227

Tree-SHA512: c507a536af104b3bde4366b6634099db826532bd3e7c35d694b5883c550920643b3eab79c76703ca67e1044ed09979e855088f7324321c8d52112514e334d614
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 17, 2020
…ng error

ada1af6 Fix typo in test_runner.py causing error (MeshCollider)

Pull request description:

  In the case that a test fails, the typo in run_tests() (introduced in bitcoin#11858) will cause an error rather than printing out the combined logs, hiding the cause of the failure.

Tree-SHA512: 7d7aa406d92750320ed20610cc5f174cdc94086f630af8c0c4db2003497132e0c56d59b94312fb42ad4507904a2fa858226a4a9337450930bf206183fc35c0a0
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants