X Tutup
Skip to content

Fix skipping test_UNC_path on AppVeyor due to a different error being raised#1920

Merged
zooba merged 1 commit intopython:masterfrom
segevfiner:fix-skip-test-unc-path-appveyor
Jun 2, 2017
Merged

Fix skipping test_UNC_path on AppVeyor due to a different error being raised#1920
zooba merged 1 commit intopython:masterfrom
segevfiner:fix-skip-test-unc-path-appveyor

Conversation

@segevfiner
Copy link
Copy Markdown
Contributor

We get ERROR_BAD_NETPATH (53) on AppVeyor which is translated to ENOENT (2).

Let AppVeyor run this of course first.

… raised

We get `ERROR_BAD_NETPATH` (53) on AppVeyor which is translated to
ENOENT (2).
@zware
Copy link
Copy Markdown
Member

zware commented Jun 2, 2017

AppVeyor is investigating why this changed; see appveyor/ci#1579.

@zooba
Copy link
Copy Markdown
Member

zooba commented Jun 2, 2017

Sure, but ENOENT is an equally good reason to skip network share tests. It's clearly an environmental issue and not ours (or am I missing something?)

zware pushed a commit to zware/cpython that referenced this pull request Jun 4, 2017
… being raised (pythonGH-1920)

We get `ERROR_BAD_NETPATH` (53) on AppVeyor which is translated to
ENOENT (2).
(cherry picked from commit 7a99625)
zware pushed a commit to zware/cpython that referenced this pull request Jun 4, 2017
… being raised (pythonGH-1920)

We get `ERROR_BAD_NETPATH` (53) on AppVeyor which is translated to
ENOENT (2).
(cherry picked from commit 7a99625)
zware added a commit that referenced this pull request Jun 4, 2017
… being raised (GH-1920)

We get `ERROR_BAD_NETPATH` (53) on AppVeyor which is translated to
ENOENT (2).
(cherry picked from commit 7a99625)
zware added a commit that referenced this pull request Jun 4, 2017
… being raised (GH-1920)

We get `ERROR_BAD_NETPATH` (53) on AppVeyor which is translated to
ENOENT (2).
(cherry picked from commit 7a99625)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup