test: unskip relevant tests #309

ral-facilities · Feb 8, 2022 · 786eb1a · 786eb1a
1 parent 56d613d
commit 786eb1a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 22 deletions.
diff --git a/test/search_api/endpoints/test_count_dataset_files.py b/test/search_api/endpoints/test_count_dataset_files.py
@@ -12,8 +12,6 @@ class TestSearchAPICountDatasetFilesEndpoint:
                 "{}",
                 {"count": 56},
                 id="Basic /datasets/{pid}/files/count request",
-                # Skipped because empty dict for filter doesn't work on where
-                marks=pytest.mark.skip,
             ),
             pytest.param(
                 "0-8401-1070-7",
@@ -40,12 +38,7 @@ class TestSearchAPICountDatasetFilesEndpoint:
                 id="Count dataset files with filter to return zero count",
             ),
             pytest.param(
-                "unknown pid",
-                "{}",
-                {"count": 0},
-                id="Non-existent dataset pid",
-                # Skipped because empty dict for filter doesn't work on where
-                marks=pytest.mark.skip,
+                "unknown pid", "{}", {"count": 0}, id="Non-existent dataset pid",
             ),
         ],
     )

diff --git a/test/search_api/endpoints/test_count_endpoint.py b/test/search_api/endpoints/test_count_endpoint.py
@@ -8,28 +8,16 @@ class TestSearchAPICountEndpoint:
         "endpoint_name, request_filter, expected_json",
         [
             pytest.param(
-                "datasets",
-                "{}",
-                {"count": 479},
-                id="Basic /datasets/count request",
-                # Skipped because empty dict for filter doesn't work on where
-                marks=pytest.mark.skip,
+                "datasets", "{}", {"count": 479}, id="Basic /datasets/count request",
             ),
             pytest.param(
-                "documents",
-                "{}",
-                {"count": 239},
-                id="Basic /documents/count request",
-                # Skipped because empty dict for filter doesn't work on where
-                marks=pytest.mark.skip,
+                "documents", "{}", {"count": 239}, id="Basic /documents/count request",
             ),
             pytest.param(
                 "instruments",
                 "{}",
                 {"count": 14},
                 id="Basic /instruments/count request",
-                # Skipped because empty dict for filter doesn't work on where
-                marks=pytest.mark.skip,
             ),
             pytest.param(
                 "datasets",