Skip to content

Commit

Permalink
Remove noqa workaround
Browse files Browse the repository at this point in the history
  • Loading branch information
hugovk committed Sep 8, 2019
1 parent fd5ffb6 commit 0575cd7
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions tests/functional/test_install.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
import textwrap
from os.path import curdir, join, pardir

# Ignore because flake8 can't detect the use inside skipif().
import pip._vendor.six # noqa: F401
import pip._vendor.six
import pytest

from pip._internal import pep425tags
Expand All @@ -21,6 +20,9 @@
from tests.lib.local_repos import local_checkout
from tests.lib.path import Path

python3_only = pytest.mark.skipif(not pip._vendor.six.PY2,
reason="Python 3 only")


@pytest.mark.parametrize('command', ('install', 'wheel'))
@pytest.mark.parametrize('variant', ('missing_setuptools', 'bad_setuptools'))
Expand Down Expand Up @@ -530,7 +532,7 @@ def test_editable_install__local_dir_no_setup_py_with_pyproject(
assert 'A "pyproject.toml" file was found' in msg


@pytest.mark.skipif("pip._vendor.six.PY3")
@python3_only
@pytest.mark.xfail
def test_install_argparse_shadowed(script):
# When argparse is in the stdlib, we support installing it
Expand Down

0 comments on commit 0575cd7

Please sign in to comment.