Ignore warnings about unused mock_folio_client

sul-dlss · May 2, 2024 · 2f3b35c · 2f3b35c
1 parent 084296f
commit 2f3b35c
Showing 1 changed file with 21 additions and 21 deletions.
diff --git a/tests/data_exports/test_gobi_exports_list.py b/tests/data_exports/test_gobi_exports_list.py
@@ -2,7 +2,7 @@
 import pymarc
 
 from libsys_airflow.plugins.data_exports.marc import gobi as gobi_transformer
-from tests.data_exports.test_marc_transformations import mock_folio_client as test_client
+from tests.data_exports.test_marc_transformations import mock_folio_client  # noqa
 
 
 def record(**kwargs):
@@ -38,7 +38,7 @@ def record(**kwargs):
     return record
 
 
-def test_with_ebook_and_print(tmp_path, mocker, test_client):
+def test_with_ebook_and_print(tmp_path, mocker, mock_folio_client):  # noqa
     file_date = "20240101"
 
     folio_result = {
@@ -58,11 +58,11 @@ def test_with_ebook_and_print(tmp_path, mocker, test_client):
     def mock_folio_get(*args):
         return folio_result
 
-    test_client.folio_get = mock_folio_get
+    mock_folio_client.folio_get = mock_folio_get
 
     mocker.patch(
         'libsys_airflow.plugins.data_exports.marc.transformer.folio_client',
-        return_value=test_client,
+        return_value=mock_folio_client,
     )
 
     marc_file = tmp_path / f"{file_date}.mrc"
@@ -88,7 +88,7 @@ def mock_folio_get(*args):
         assert fo.readline() == "1234567890123|ebook|325099\n"
 
 
-def test_with_ebook_only(tmp_path, mocker, test_client):
+def test_with_ebook_only(tmp_path, mocker, mock_folio_client):  # noqa
     file_date = "20240102"
 
     folio_result = {
@@ -104,11 +104,11 @@ def test_with_ebook_only(tmp_path, mocker, test_client):
     def mock_folio_get(*args):
         return folio_result
 
-    test_client.folio_get = mock_folio_get
+    mock_folio_client.folio_get = mock_folio_get
 
     mocker.patch(
         'libsys_airflow.plugins.data_exports.marc.transformer.folio_client',
-        return_value=test_client,
+        return_value=mock_folio_client,
     )
 
     marc_file = tmp_path / f"{file_date}.mrc"
@@ -133,12 +133,12 @@ def mock_folio_get(*args):
         assert fo.readline() == "1234567890123|ebook|325099\n"
 
 
-def test_with_no_isbn(tmp_path, mocker, test_client):
+def test_with_no_isbn(tmp_path, mocker, mock_folio_client):  # noqa
     file_date = "20240103.mrc"
 
     mocker.patch(
         'libsys_airflow.plugins.data_exports.marc.transformer.folio_client',
-        return_value=test_client,
+        return_value=mock_folio_client,
     )
 
     marc_file = tmp_path / f"{file_date}.mrc"
@@ -156,7 +156,7 @@ def test_with_no_isbn(tmp_path, mocker, test_client):
         assert fo.readline() == ""
 
 
-def test_with_print_no_electronic_holding(tmp_path, mocker, test_client):
+def test_with_print_no_electronic_holding(tmp_path, mocker, mock_folio_client):  # noqa
     file_date = "20240104"
 
     folio_result = {
@@ -176,11 +176,11 @@ def test_with_print_no_electronic_holding(tmp_path, mocker, test_client):
     def mock_folio_get(*args):
         return folio_result
 
-    test_client.folio_get = mock_folio_get
+    mock_folio_client.folio_get = mock_folio_get
 
     mocker.patch(
         'libsys_airflow.plugins.data_exports.marc.transformer.folio_client',
-        return_value=test_client,
+        return_value=mock_folio_client,
     )
 
     marc_file = tmp_path / f"{file_date}.mrc"
@@ -206,7 +206,7 @@ def mock_folio_get(*args):
         assert fo.readline() == ""
 
 
-def test_with_skipped_by_035(tmp_path, mocker, test_client):
+def test_with_skipped_by_035(tmp_path, mocker, mock_folio_client):  # noqa
     file_date = "20240105"
 
     folio_result = {
@@ -226,11 +226,11 @@ def test_with_skipped_by_035(tmp_path, mocker, test_client):
     def mock_folio_get(*args):
         return folio_result
 
-    test_client.folio_get = mock_folio_get
+    mock_folio_client.folio_get = mock_folio_get
 
     mocker.patch(
         'libsys_airflow.plugins.data_exports.marc.transformer.folio_client',
-        return_value=test_client,
+        return_value=mock_folio_client,
     )
 
     marc_file = tmp_path / f"{file_date}.mrc"
@@ -256,7 +256,7 @@ def mock_folio_get(*args):
         assert fo.readline() == ""
 
 
-def test_with_skipped_by_856(tmp_path, mocker, test_client):
+def test_with_skipped_by_856(tmp_path, mocker, mock_folio_client):  # noqa
     file_date = "20240106"
 
     folio_result = {
@@ -276,11 +276,11 @@ def test_with_skipped_by_856(tmp_path, mocker, test_client):
     def mock_folio_get(*args):
         return folio_result
 
-    test_client.folio_get = mock_folio_get
+    mock_folio_client.folio_get = mock_folio_get
 
     mocker.patch(
         'libsys_airflow.plugins.data_exports.marc.transformer.folio_client',
-        return_value=test_client,
+        return_value=mock_folio_client,
     )
 
     marc_file = tmp_path / f"{file_date}.mrc"
@@ -306,7 +306,7 @@ def mock_folio_get(*args):
         assert fo.readline() == ""
 
 
-def test_with_skipped_by_956(tmp_path, mocker, test_client):
+def test_with_skipped_by_956(tmp_path, mocker, mock_folio_client):  # noqa
     file_date = "20240107"
 
     folio_result = {
@@ -326,11 +326,11 @@ def test_with_skipped_by_956(tmp_path, mocker, test_client):
     def mock_folio_get(*args):
         return folio_result
 
-    test_client.folio_get = mock_folio_get
+    mock_folio_client.folio_get = mock_folio_get
 
     mocker.patch(
         'libsys_airflow.plugins.data_exports.marc.transformer.folio_client',
-        return_value=test_client,
+        return_value=mock_folio_client,
     )
 
     marc_file = tmp_path / f"{file_date}.mrc"