TST: Move tests/scripts to scripts/tests

Author: gfyoung

.travis.yml

@@ -57,6 +57,9 @@ matrix:
     - dist: trusty
       env:
         - JOB="3.6, coverage" ENV_FILE="ci/travis-36.yaml" TEST_ARGS="--skip-slow --skip-network" PANDAS_TESTING_MODE="deprecate" COVERAGE=true DOCTEST=true
+    - dist: trusty
+      env:
+        - JOB="3.6, scripts" ENV_FILE="ci/travis-36.yaml" SCRIPTS=true
     # In allow_failures
     - dist: trusty
       env:

ci/script_multi.sh

@@ -35,6 +35,9 @@ elif [ "$SLOW" ]; then
     echo pytest -r xX -m "not single and slow" -v --junitxml=/tmp/multiple.xml --strict $TEST_ARGS pandas
     pytest -r xX -m "not single and slow" -v --junitxml=/tmp/multiple.xml --strict $TEST_ARGS pandas
 
+elif [ "$SCRIPTS" ]; then
+    echo "We are not running this type of test on scripts"
+
 else
     echo pytest -n 2 -r xX -m "not single" --junitxml=/tmp/multiple.xml --strict $TEST_ARGS pandas
     pytest -n 2 -r xX -m "not single" --junitxml=/tmp/multiple.xml --strict $TEST_ARGS pandas # TODO: doctest

ci/script_single.sh

@@ -28,6 +28,9 @@ elif [ "$COVERAGE" ]; then
     echo pytest -s -m "single" -r xXs --strict --cov=pandas --cov-report xml:/tmp/cov-single.xml --junitxml=/tmp/single.xml $TEST_ARGS pandas
     pytest      -s -m "single" -r xXs --strict --cov=pandas --cov-report xml:/tmp/cov-single.xml --junitxml=/tmp/single.xml $TEST_ARGS pandas
 
+elif [ "$SCRIPTS" ]; then
+    echo pytest -s -r xXs --strict scripts
+    pytest      -s -r xXs --strict scripts
 else
     echo pytest -m "single" -r xXs --junitxml=/tmp/single.xml --strict $TEST_ARGS pandas
     pytest      -m "single" -r xXs --junitxml=/tmp/single.xml --strict $TEST_ARGS pandas # TODO: doctest

pandas/tests/scripts/__init__.py scripts/tests/__init__.pypandas/tests/scripts/__init__.py renamed to scripts/tests/__init__.py

@@ -0,0 +1,3 @@
+def pytest_addoption(parser):
+    parser.addoption("--strict-data-files", action="store_true",
+                     help="Unused. For compat with setup.cfg.")

scripts/tests/conftest.py

@@ -1,8 +1,10 @@
-import os
-import sys
-
-import numpy as np
 import pytest
+import string
+import random
+import numpy as np
+
+validate_docstrings = pytest.importorskip("validate_docstrings")
+validate_one = validate_docstrings.validate_one
 
 
 class GoodDocStrings(object):
@@ -44,7 +46,7 @@ def sample(self):
         float
             Random number generated.
         """
-        return random.random()  # noqa: F821
+        return random.random()
 
     def random_letters(self):
         """
@@ -60,9 +62,9 @@ def random_letters(self):
         letters : str
             String of random letters.
         """
-        length = random.randint(1, 10)  # noqa: F821
-        letters = ''.join(random.choice(string.ascii_lowercase)  # noqa: F821
-                          for i in range(length))
+        length = random.randint(1, 10)
+        letters = "".join(random.choice(string.ascii_lowercase)
+                          for _ in range(length))
         return length, letters
 
     def sample_values(self):
@@ -78,7 +80,7 @@ def sample_values(self):
             Random number generated.
         """
         while True:
-            yield random.random()  # noqa: F821
+            yield random.random()
 
     def head(self):
         """
@@ -491,44 +493,6 @@ def no_punctuation(self):
 
 class TestValidator(object):
 
-    @pytest.fixture(autouse=True, scope="class")
-    def import_scripts(self):
-        """
-        Import the validation scripts from the scripts directory.
-
-        Because the scripts directory is above the top level pandas package,
-        we need to modify `sys.path` so that Python knows where to find it.
-
-        The code below traverses up the file system to find the scripts
-        directory, adds the location to `sys.path`, and imports the required
-        module into the global namespace before as part of class setup.
-
-        During teardown, those changes are reverted.
-        """
-
-        up = os.path.dirname
-        global_validate_one = "validate_one"
-        file_dir = up(os.path.abspath(__file__))
-
-        script_dir = os.path.join(up(up(up(file_dir))), "scripts")
-        sys.path.append(script_dir)
-
-        try:
-            from validate_docstrings import validate_one
-            globals()[global_validate_one] = validate_one
-        except ImportError:
-            # Remove addition to `sys.path`
-            sys.path.pop()
-
-            # Import will fail if the pandas installation is not inplace.
-            raise pytest.skip("pandas/scripts directory does not exist")
-
-        yield
-
-        # Teardown.
-        sys.path.pop()
-        del globals()[global_validate_one]
-
     def _import_path(self, klass=None, func=None):
         """
         Build the required import path for tests in this module.
@@ -545,27 +509,29 @@ def _import_path(self, klass=None, func=None):
         str
             Import path of specified object in this module
         """
-        base_path = 'pandas.tests.scripts.test_validate_docstrings'
+        base_path = "scripts.tests.test_validate_docstrings"
+
         if klass:
-            base_path = '.'.join([base_path, klass])
+            base_path = ".".join([base_path, klass])
+
         if func:
-            base_path = '.'.join([base_path, func])
+            base_path = ".".join([base_path, func])
 
         return base_path
 
     def test_good_class(self):
-        assert validate_one(self._import_path(  # noqa: F821
+        assert validate_one(self._import_path(
             klass='GoodDocStrings')) == 0
 
     @pytest.mark.parametrize("func", [
         'plot', 'sample', 'random_letters', 'sample_values', 'head', 'head1',
         'contains', 'mode'])
     def test_good_functions(self, func):
-        assert validate_one(self._import_path(   # noqa: F821
+        assert validate_one(self._import_path(
             klass='GoodDocStrings', func=func)) == 0
 
     def test_bad_class(self):
-        assert validate_one(self._import_path(  # noqa: F821
+        assert validate_one(self._import_path(
             klass='BadGenericDocStrings')) > 0
 
     @pytest.mark.parametrize("func", [