Make and test that sites/ tests directory passes mypy (#1221)

This directory contains currently-maintained test files so we should care for it as much as the other test directories.
Parsl · Aug 27, 2019 · e0fcd7fffce8107018e935f55d2c261511a5bc44 · e0fcd7f
1 parent 6e77e8e
commit e0fcd7fffce8107018e935f55d2c261511a5bc44
Unified Split

Showing with 9 additions and 2 deletions.

+1 −0 .travis.yml

+6 −0 mypy.ini

+2 −2 parsl/tests/sites/test_multinode_mpi.py
diff --git a/.travis.yml b/.travis.yml
@@ -39,6 +39,7 @@ script:
     # parsl
     - MYPYPATH=$(pwd)/mypy-stubs mypy parsl/tests/configs/
     - MYPYPATH=$(pwd)/mypy-stubs mypy parsl/tests/test*/
+    - MYPYPATH=$(pwd)/mypy-stubs mypy parsl/tests/sites/
 
       # do this before any testing, but not in-between tests
     - rm -f .coverage

diff --git a/mypy.ini b/mypy.ini
@@ -10,9 +10,15 @@ ignore_missing_imports = True
 [mypy-sqlalchemy_utils.*]
 ignore_missing_imports = True
 
+[mypy-_pytest.*]
+ignore_missing_imports = True
+
 [mypy-pytest.*]
 ignore_missing_imports = True
 
+[mypy-pytest_forked.*]
+ignore_missing_imports = True
+
 [mypy-dill.*]
 ignore_missing_imports = True
 

diff --git a/parsl/tests/sites/test_multinode_mpi.py b/parsl/tests/sites/test_multinode_mpi.py
@@ -65,5 +65,5 @@ def bash_mpi_app(stdout=None, stderr=None):
                          stderr="parsl.{0}.err".format(i))
         items.extend([x])
 
-    for i in items:
-        print(i.result())
+    for f in items:
+        print(f.result())