Crush 🪗 core/overdrive.py and api/overdrive.py together (PP-500) (#1436)

This refactors the core/overdrive.py and api/overdrive.py files to smash them together. In the process it combines the test fixtures for overdrive from overdrive_fixture, overdrive_with_api_fixture, and overdrive_api_fixture, to just overdrive_api_fixture.
ThePalaceProject · Oct 5, 2023 · 5484028 · 5484028
1 parent 3961f6c
commit 5484028
Show file tree

Hide file tree

Showing 24 changed files with 2,818 additions and 2,999 deletions.
diff --git a/api/overdrive.py b/api/overdrive.py
diff --git a/bin/informational/overdrive-advantage-accounts b/bin/informational/overdrive-advantage-accounts
@@ -6,6 +6,6 @@ bin_dir = os.path.split(__file__)[0]
 package_dir = os.path.join(bin_dir, "..", "..")
 sys.path.append(os.path.abspath(package_dir))
 
-from core.scripts import GenerateOverdriveAdvantageAccountList
+from api.overdrive import GenerateOverdriveAdvantageAccountList
 
 GenerateOverdriveAdvantageAccountList().run()
diff --git a/bin/informational/overdrive-advantage-list b/bin/informational/overdrive-advantage-list
@@ -6,6 +6,6 @@ bin_dir = os.path.split(__file__)[0]
 package_dir = os.path.join(bin_dir, "..", "..")
 sys.path.append(os.path.abspath(package_dir))
 
-from api.overdrive import OverdriveAdvantageAccountListScript  # noqa: E402
+from api.overdrive import OverdriveAdvantageAccountListScript
 
 OverdriveAdvantageAccountListScript().run()
diff --git a/bin/repair/overdrive_bibliographic_coverage b/bin/repair/overdrive_bibliographic_coverage
@@ -6,7 +6,7 @@ import sys
 bin_dir = os.path.split(__file__)[0]
 package_dir = os.path.join(bin_dir, "..", "..")
 sys.path.append(os.path.abspath(package_dir))
-from core.overdrive import OverdriveBibliographicCoverageProvider
+from api.overdrive import OverdriveBibliographicCoverageProvider
 from core.scripts import RunCoverageProviderScript
 
 RunCoverageProviderScript(OverdriveBibliographicCoverageProvider).run()