Run cache tests separately and last

 They modify globals and impact other runs in same process, so run
 them separately in Ci and restore empty globals at end of tests.
 Run them last (renamed with zzzz)

Signed-off-by: Philippe Ombredanne <[email protected]>

Author: pombredanne

.travis.yml

@@ -14,7 +14,8 @@ env:
     - TEST_SUITE="$PYTEST --ignore=tests/scancode --ignore=tests/extractcode --ignore=tests/licensedcode --ignore=tests/cluecode --ignore=tests/packagedcode"
     - TEST_SUITE="$PYTEST tests/scancode"
     - TEST_SUITE="$PYTEST tests/extractcode"
-    - TEST_SUITE="$PYTEST tests/licensedcode"
+    - TEST_SUITE="$PYTEST tests/licensedcode --ignore=tests/licensedcode/test_zzzz_cache.py"
+    - TEST_SUITE="$PYTEST tests/licensedcode/test_zzzz_cache.py"
     - TEST_SUITE="$PYTEST tests/cluecode"
     - TEST_SUITE="$PYTEST tests/packagedcode"
     - TEST_SUITE="./etc/release/release.sh"

appveyor.yml

@@ -5,7 +5,8 @@ environment:
         - TEST_SUITE: "py.test -n 2 -vvs --ignore=tests/scancode --ignore=tests/extractcode --ignore=tests/licensedcode --ignore=tests/cluecode --ignore=tests/packagedcode"
         - TEST_SUITE: "py.test -n 2 -vvs tests/scancode"
         - TEST_SUITE: "py.test -n 2 -vvs tests/extractcode"
-        - TEST_SUITE: "py.test -n 2 -s tests/licensedcode"
+        - TEST_SUITE: "py.test -n 2 -s tests/licensedcode --ignore=tests/licensedcode/test_zzzz_cache.py"
+        - TEST_SUITE: "py.test -s tests/licensedcode/test_zzzz_cache.py"
         - TEST_SUITE: "py.test -n 2 -s tests/cluecode"
         - TEST_SUITE: "py.test -n 2 -s tests/packagedcode"
 

tests/licensedcode/test_cache.py tests/licensedcode/test_zzzz_cache.pytests/licensedcode/test_cache.py renamed to tests/licensedcode/test_zzzz_cache.py

@@ -206,6 +206,12 @@ def test_build_index(self):
         assert tree_before == open(checksum_file).read()
         assert os.path.exists(cache_file)
 
+        # reset tests caches
+        cache._LICENSE_SYMBOLS_BY_SPDX_KEY = {}
+        cache._LICENSES_BY_KEY_INDEX = None
+        cache._UNKNOWN_SPDX_SYMBOL = None
+        cache._LICENSES_BY_KEY = None
+
     def test__load_index(self):
         cache_dir = self.get_temp_dir('index_cache')
 
@@ -232,3 +238,9 @@ def test__load_index(self):
 
         idx2 = cache.load_index(cache_file)
         assert idx1.to_dict(True) == idx2.to_dict(True)
+
+        # reset tests caches
+        cache._LICENSE_SYMBOLS_BY_SPDX_KEY = {}
+        cache._LICENSES_BY_KEY_INDEX = None
+        cache._UNKNOWN_SPDX_SYMBOL = None
+        cache._LICENSES_BY_KEY = None