Skip to content

bpo-33617: Use os.fspath() for subprocess sequences on Windows - #7071

Closed
altendky wants to merge 1 commit into
python:masterfrom
altendky:33617-subprocess_pathlike_in_sequence
Closed

bpo-33617: Use os.fspath() for subprocess sequences on Windows#7071
altendky wants to merge 1 commit into
python:masterfrom
altendky:33617-subprocess_pathlike_in_sequence

Conversation

@altendky

@altendky altendky commented May 23, 2018

Copy link
Copy Markdown
Contributor

@altendky

altendky commented May 23, 2018

Copy link
Copy Markdown
Contributor Author

https://travis-ci.org/python/cpython/jobs/382707216#L8506-L8522

======================================================================
FAIL: test_stdin_broken_pipe (test.test_asyncio.test_subprocess.SubprocessFastWatcherTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/python/cpython/Lib/test/test_asyncio/test_subprocess.py", line 238, in test_stdin_broken_pipe
    self.loop.run_until_complete, coro)
AssertionError: (<class 'BrokenPipeError'>, <class 'ConnectionResetError'>) not raised by run_until_complete
======================================================================
FAIL: test_stdin_broken_pipe (test.test_asyncio.test_subprocess.SubprocessSafeWatcherTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/python/cpython/Lib/test/test_asyncio/test_subprocess.py", line 238, in test_stdin_broken_pipe
    self.loop.run_until_complete, coro)
AssertionError: (<class 'BrokenPipeError'>, <class 'ConnectionResetError'>) not raised by run_until_complete
----------------------------------------------------------------------

I ran this locally 10x and didn't get the errors. Is there a place where flaky tests are tracked that I could check? Otherwise I'll try making my own Travis build of it to see if I can recreate it.

 ~/cpython   33617-subprocess_pathlike_in_sequence  ./python -m test -j3 test_asyncio.test_subprocess
Run tests in parallel using 3 child processes
0:00:04 load avg: 3.28 [1/1] test_asyncio.test_subprocess passed
1 test OK.

Total duration: 5 sec
Tests result: SUCCESS

@zooba

zooba commented Aug 2, 2019

Copy link
Copy Markdown
Member

This has been implemented in another PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants