From 8a603ffeb69a70a3f988965c42cac3528a6bdd7b Mon Sep 17 00:00:00 2001 From: Matyas Selmeci Date: Tue, 1 Oct 2024 14:12:35 -0500 Subject: [PATCH] Comment out tests that depend on having a certain number of caches The namespaces JSON does not list Pelican caches (on purpose); as we convert more I2 caches, we drop below the number of caches the tests are expecting. Until we come up with some better tests, comment out these ones to avoid unnecessary failures. --- src/tests/test_stashcache.py | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/tests/test_stashcache.py b/src/tests/test_stashcache.py index 1318dd318..768430216 100644 --- a/src/tests/test_stashcache.py +++ b/src/tests/test_stashcache.py @@ -326,11 +326,11 @@ def validate_namespace_schema(ns): if credgen["base_path"]: assert isinstance(credgen["base_path"], str) - def test_caches(self, caches): - # Have a reasonable number of caches - assert len(caches) > 20 - for cache in caches: - self.validate_cache_schema(cache) + # def test_caches(self, caches): + # # Have a reasonable number of caches + # assert len(caches) > 20 + # for cache in caches: + # self.validate_cache_schema(cache) def test_namespaces(self, namespaces): # Have a reasonable number of namespaces @@ -375,7 +375,7 @@ def test_testvo_public_namespace(self, namespaces): assert ns["readhttps"] is False assert ns["usetokenonread"] is False assert TEST_SC_ORIGIN in ns["writebackhost"] - assert len(ns["caches"]) > 10 + # assert len(ns["caches"]) > 10 assert len(ns["origins"]) == 2 assert ns["credential_generation"] is None assert len(ns["scitokens"]) == 1 @@ -394,7 +394,7 @@ def test_testvo_namespace(self, namespaces): assert ns["usetokenonread"] is True assert TEST_ORIGIN_AUTH2000 in ns["writebackhost"] assert TEST_ORIGIN_AUTH2000 in ns["dirlisthost"] - assert len(ns["caches"]) > 10 + # assert len(ns["caches"]) > 10 assert len(ns["origins"]) == 1 credgen = ns["credential_generation"] assert credgen["base_path"] == TEST_BASEPATH