diff --git a/tests/data_exports/test_gobi_exports_list.py b/tests/data_exports/test_gobi_exports_list.py index 8931d3c6..bb57143e 100644 --- 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"