From 786eb1a83bfaeddac18ed83076a6cfadcf83abf3 Mon Sep 17 00:00:00 2001
From: Viktor Bozhinov <viktor_bozinov1@hotmail.com>
Date: Tue, 8 Feb 2022 14:49:03 +0000
Subject: [PATCH] test: unskip relevant tests #309

---
 .../endpoints/test_count_dataset_files.py        |  9 +--------
 test/search_api/endpoints/test_count_endpoint.py | 16 ++--------------
 2 files changed, 3 insertions(+), 22 deletions(-)

diff --git a/test/search_api/endpoints/test_count_dataset_files.py b/test/search_api/endpoints/test_count_dataset_files.py
index 32749a1f..1f475fa6 100644
--- 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
index 2e41d18c..a89ec03d 100644
--- 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",