did small cleanup and added 2 test cases from GitHub issues

mbdevpl · Dec 10, 2017 · e69379b · e69379b
1 parent 0eb17f0
commit e69379b
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 2 deletions.
diff --git a/open_fortran_parser/main.py b/open_fortran_parser/main.py
@@ -56,7 +56,8 @@ def main(args=None, namespace=None):
         return
 
     if args.cleanup_dependencies:
-        cleanup_old_dependencies(OUTDATED_DEPENDENCIES, DEPENDENCIES_PATH)
+        cleanup_old_dependencies(OUTDATED_DEPENDENCIES, DEV_DEPENDENCIES_PATH)
+        return
 
     if not args.input:
         parser.print_help(sys.stderr)

diff --git a/test/examples/problematic_write.f90 b/test/examples/problematic_write.f90
@@ -0,0 +1,7 @@
+
+PROGRAM problematic_if
+
+! IF (printlev_loc >=2)  WRITE(numout,*),'tau_outflow, coeff_rel = ', tau_outflow, coeff_rel
+  IF (printlev_loc >=2)  WRITE(numout,*) 'tau_outflow, coeff_rel = ', tau_outflow, coeff_rel
+
+END
diff --git a/test/examples/simple_if.f90 b/test/examples/simple_if.f90
@@ -0,0 +1,8 @@
+
+PROGRAM simple_if
+
+IF (map_pft_format .AND. .NOT. impveg) THEN
+  CONTINUE
+ENDIF
+
+END
diff --git a/test/test_parser_wrapper.py b/test/test_parser_wrapper.py
@@ -88,7 +88,7 @@ def test_generate_xml(self):
                 with self.subTest(input_path=input_path, verbosity=verbosity):
                     output_path = pathlib.Path(results_path, input_path.name + '.xml')
                     process = execute_parser(input_path, output_path, verbosity)
-                    self.assertEqual(process.returncode, 0)
+                    self.assertEqual(process.returncode, 0, process)
                     self.assertTrue(output_path.exists())
 
     @unittest.skipUnless(os.environ.get('TEST_LONG'), 'skipping long test')