Convert the Bibliotheca integration into a package

The Bibliotheca integration was five flat bibliotheca_* modules sitting
directly in integration/license/, while every other integration
(overdrive/, boundless/, opds/) is a package. Restructure it to match.

The five modules become bibliotheca/{api,importer,purchase_record_importer,
circulation_updater,scripts}.py, and the 1295-line api.py is split along
the same seams the siblings use: settings.py (as in overdrive and
boundless), parser.py (as in boundless), and coverage.py (as in
overdrive).

Splitting the parsers out exposed a cycle: BibliothecaAPI constructs the
parsers, but ErrorParser, EventParser and PatronCirculationParser reached
back for BibliothecaAPI.SERVICE_NAME and .ARGUMENT_TIME_FORMAT. Add a
constants.py holding those values and point the parsers at it, mirroring
how boundless/parser.py and overdrive/coverage.py already avoid the same
cycle. BibliothecaAPI keeps both class attributes, so nothing that reads
them off the API changes.

Also drop EventParser.EVENT_SOURCE and EventParser.SET_DELIVERY_MECHANISM_AT.
Both were unread: EVENT_SOURCE has no readers anywhere, and the two sites
that read SET_DELIVERY_MECHANISM_AT do so off a BaseCirculationAPI, which
EventParser is not. Removing them lets parser.py stop importing the
circulation API layer entirely. BibliothecaAPI.SET_DELIVERY_MECHANISM_AT
is a different attribute on a different class and is untouched.

Mirror the layout in tests/, moving the fixture into a conftest.py as
boundless does. TestBibliographicCoverageProvider subclassed
TestBibliothecaAPI without using anything from it, which made pytest
re-run all 22 API tests a second time; it no longer does. All 38 declared
tests are preserved, with test_internal_formats moving to
TestItemListParser, the class it actually exercises.

Author: jonathangreen

bin/bibliotheca_circulation_update

@@ -1,6 +1,6 @@
 #!/usr/bin/env python
 """Manually kick off the Bibliotheca circulation update for one or all collections."""
-from palace.manager.integration.license.bibliotheca_scripts import (
+from palace.manager.integration.license.bibliotheca.scripts import (
     CirculationUpdateCollection,
 )
 

bin/bibliotheca_import

@@ -1,7 +1,7 @@
 #!/usr/bin/env python
 """Manually kick off the Bibliotheca event import for one or all collections."""
 
-from palace.manager.integration.license.bibliotheca_scripts import (
+from palace.manager.integration.license.bibliotheca.scripts import (
     ImportEventCollection,
 )
 

bin/bibliotheca_purchase_record_import

@@ -1,7 +1,7 @@
 #!/usr/bin/env python
 """Manually kick off the Bibliotheca purchase record import for one or all collections."""
 
-from palace.manager.integration.license.bibliotheca_scripts import (
+from palace.manager.integration.license.bibliotheca.scripts import (
     ImportPurchaseRecordCollection,
 )
 

bin/informational/bibliotheca-raw-bibliographic

@@ -1,7 +1,7 @@
 #!/usr/bin/env python3
 from xml.dom import minidom
 
-from palace.manager.integration.license.bibliotheca import BibliothecaAPI
+from palace.manager.integration.license.bibliotheca.api import BibliothecaAPI
 from palace.manager.scripts.input import IdentifierInputScript
 from palace.manager.sqlalchemy.model.collection import Collection
 

bin/informational/bibliotheca-raw-patron-status

@@ -2,7 +2,7 @@
 import sys
 from xml.dom import minidom
 
-from palace.manager.integration.license.bibliotheca import BibliothecaAPI
+from palace.manager.integration.license.bibliotheca.api import BibliothecaAPI
 from palace.manager.scripts.base import Script
 from palace.manager.sqlalchemy.model.collection import Collection
 from palace.manager.sqlalchemy.model.patron import Patron

bin/repair/bibliotheca_bibliographic_coverage

@@ -2,7 +2,7 @@
 """Make sure all Bibliotheca books have bibliographic coverage."""
 
 
-from palace.manager.integration.license.bibliotheca import (
+from palace.manager.integration.license.bibliotheca.coverage import (
     BibliothecaBibliographicCoverageProvider,
 )
 from palace.manager.scripts.coverage_provider import RunCollectionCoverageProviderScript

src/palace/manager/celery/tasks/bibliotheca.py

@@ -32,15 +32,15 @@
 from palace.manager.celery.importer import workflow_lock_guard
 from palace.manager.celery.task import Task
 from palace.manager.celery.utils import ModelNotFoundError, load_from_id, signature_with
-from palace.manager.integration.license.bibliotheca import BibliothecaAPI
-from palace.manager.integration.license.bibliotheca_circulation_updater import (
+from palace.manager.integration.license.bibliotheca.api import BibliothecaAPI
+from palace.manager.integration.license.bibliotheca.circulation_updater import (
     BibliothecaCirculationUpdater,
 )
-from palace.manager.integration.license.bibliotheca_importer import (
+from palace.manager.integration.license.bibliotheca.importer import (
     EVENT_IMPORT_OVERLAP,
     BibliothecaEventImporter,
 )
-from palace.manager.integration.license.bibliotheca_purchase_record_importer import (
+from palace.manager.integration.license.bibliotheca.purchase_record_importer import (
     DEFAULT_PURCHASE_RECORD_START_TIME,
     PURCHASE_RECORD_SERVICE_NAME,
     BibliothecaPurchaseRecordImporter,