Merge pull request #111 from NREL/SkipAPIForNow

Cleanups, also skip files in the testfiles/API folder
NREL · Jul 9, 2024 · b4477c8 · b4477c8
2 parents 515377f + 2f1ad3c
commit b4477c8
Show file tree

Hide file tree

Showing 14 changed files with 181 additions and 1,319 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -10,7 +10,7 @@ jobs:
   unit_tests:
     strategy:
       matrix:
-        os: [ windows-latest, macos-11, ubuntu-22.04 ]  # macos-12 is not playing well with Tk
+        os: [ windows-latest, macos-latest, ubuntu-22.04 ]  # macos-12 is not playing well with Tk
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8  # v2

diff --git a/energyplus_regressions/__init__.py b/energyplus_regressions/__init__.py
@@ -1,2 +1,2 @@
 NAME = 'energyplus_regressions'
-VERSION = '2.0.6'
+VERSION = '2.0.7'
diff --git a/energyplus_regressions/build_files_to_run.py b/energyplus_regressions/build_files_to_run.py
diff --git a/energyplus_regressions/builds/base.py b/energyplus_regressions/builds/base.py
@@ -58,12 +58,13 @@ def get_idfs_in_dir(idf_dir: Path) -> Set[Path]:
         ]
 
         def should_keep(file_path):
-            should_ignore = False
             for i in known_ignore_list:
                 if i in str(file_path):
-                    should_ignore = True
-                    break
-            return not should_ignore
+                    return False
+                if 'API' in file_path.parts:
+                    # the testfiles/API directory is for Python API calls, we aren't doing that here, yet.
+                    return False
+            return True
 
         filtered_list = filter(should_keep, all_idfs_relative_path)
         return set(filtered_list)