Don't run with shell=True

ga4gh · Oct 24, 2016 · a6a0853 · a6a0853
1 parent 6f5d227
commit a6a0853
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/.flake8 b/.flake8
diff --git a/.travis.yml b/.travis.yml
@@ -22,7 +22,7 @@ before_script:
 # run_tests.py runs everything under the script: tag so only put commands
 # under it that we want to run (and want to be able to run) as local tests
 script: 
-  - flake8 *.py ga4gh_common tests
+  - flake8 run_tests_dev.py setup.py ga4gh_common tests
   - nosetests tests
         --with-coverage --cover-package ga4gh_common
         --cover-inclusive --cover-min-percentage=70

diff --git a/ga4gh_common/run_tests.py b/ga4gh_common/run_tests.py
@@ -24,7 +24,7 @@ def runTests(self):
         testCommands = self.parseTestCommands()
         for command in testCommands:
             self.log('Running: "{}"'.format(command))
-            utils.runCommand(command, shell=True)
+            utils.runCommand(command)
         self.log('SUCCESS')
 
     def log(self, logStr):