only benchmarks for first CI check

codezonediitj · Jul 5, 2024 · 45853fc · 45853fc
1 parent d9ce6a8
commit 45853fc
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -93,7 +93,7 @@ jobs:
 
       - name: Run tests
         run: |
-          python -c "import pydatastructs; pydatastructs.test(include_benchmarks=True)"
+          python -c "import pydatastructs; pydatastructs.test(only_benchmarks=True)"
 
       - name: Build Documentation
         run: |

diff --git a/pydatastructs/trees/tests/test_binary_trees.py b/pydatastructs/trees/tests/test_binary_trees.py
@@ -338,8 +338,6 @@ def _test_AVLTree(backend):
 
     assert a5.rank(-1) is None
     def test_select_rank(expected_output):
-        if backend == Backend.CPP:
-            return
         output = []
         for i in range(len(expected_output)):
             output.append(a5.select(i + 1).key)

diff --git a/pydatastructs/utils/testing_util.py b/pydatastructs/utils/testing_util.py
@@ -18,7 +18,7 @@
 
 SKIP_FILES = ['testing_util.py']
 
-def test(submodules=None, include_benchmarks=False,
+def test(submodules=None, only_benchmarks=False,
          benchmarks_size=1000, **kwargs):
     """
     Runs the library tests using pytest
@@ -52,11 +52,12 @@ def test(submodules=None, include_benchmarks=False,
                 else:
                     raise Exception("Submodule should be of type: str or module")
                 if sub in path:
-                    if not include_benchmarks:
+                    if not only_benchmarks:
                         if not 'benchmarks' in path:
                             test_files.append(path)
                     else:
-                        test_files.append(path)
+                        if 'benchmarks' in path:
+                            test_files.append(path)
                     break
     else:
         for path in glob.glob(f'{ROOT_DIR}/**/test_*.py', recursive=True):
@@ -67,11 +68,12 @@ def test(submodules=None, include_benchmarks=False,
                     break
             if skip_test:
                 continue
-            if not include_benchmarks:
+            if not only_benchmarks:
                 if not 'benchmarks' in path:
                     test_files.append(path)
             else:
-                test_files.append(path)
+                if 'benchmarks' in path:
+                    test_files.append(path)
 
     extra_args = []
     if not kwargs.get("n", False) is False: