fix: Properly cleanup multilingual glossaries in tests

Author: BriannaDelgado

tests/conftest.py

@@ -206,7 +206,7 @@ def translator_with_random_auth_key_and_proxy(server):
 
 
 @pytest.fixture
-def cleanup_matching_glossaries(translator):
+def cleanup_matching_glossaries(deepl_client):
     """
     Fixture function to remove all glossaries from the server matching the
     given predicate. Can be used, for example, to remove all glossaries with a
@@ -221,11 +221,11 @@ def test_example(cleanup_matching_glossaries):
     """
 
     def do_cleanup(predicate: Callable[[deepl.GlossaryInfo], bool]):
-        glossaries = translator.list_glossaries()
+        glossaries = deepl_client.list_multilingual_glossaries()
         for glossary in glossaries:
             if predicate(glossary):
                 try:
-                    translator.delete_glossary(glossary)
+                    deepl_client.delete_multilingual_glossary(glossary)
                 except deepl.DeepLException:
                     pass
 

tests/test_multilingual_glossary.py

@@ -140,9 +140,7 @@ def test_glossary_name_update(
         with pytest.raises(ValueError):
             deepl_client.update_multilingual_glossary_name(glossary, "")
     finally:
-        cleanup_matching_glossaries(
-            lambda glossary: glossary.name == glossary_name
-        )
+        cleanup_matching_glossaries(lambda glossary: glossary.name == new_name)
 
 
 def test_glossary_dictionary_replace(