don't install reqs as part of test

nchammas · Feb 5, 2016 · 85afbba · 85afbba
1 parent 8b88c80
commit 85afbba
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -8,6 +8,7 @@ install:
 script:
   - "py.test ./tests/test_static.py"
   - "py.test ./tests/test_flintrock.py"
+  - "pip install -r requirements/maintainer.pip"
   - "py.test ./tests/test_pyinstaller_packaging.py"
 addons:
   artifacts:

diff --git a/requirements/maintainer.pip b/requirements/maintainer.pip
@@ -2,4 +2,4 @@
 wheel >= 0.26.0
 twine >= 1.6.4
 pypandoc >= 1.1.2
-PyInstaller >= 3.1
+PyInstaller >= 3.1.1
diff --git a/tests/test_pyinstaller_packaging.py b/tests/test_pyinstaller_packaging.py
@@ -6,11 +6,14 @@
 import pytest
 
 
+def pyinstaller_exists():
+    s = subprocess.run(['command', '-v', 'pyinstaller'])
+    return s.returncode == 0
+
+
 @pytest.mark.skipif(sys.version_info < (3, 5), reason="Python 3.5+ is required")
+@pytest.mark.skipif(not pyinstaller_exists(), reason="PyInstaller is required")
 def test_pyinstaller_packaging():
-    subprocess.run(
-        ['pip', 'install', '-r', 'requirements/maintainer.pip'],
-        check=True)
     subprocess.run(
         ['python', 'generate-standalone-package.py'],
         check=True)