remove Archiver factory

Author: CaptainOfHacks

ted_sws/notice_packager/adapters/archiver.py

@@ -35,17 +35,6 @@ def process_archive(self, archive_name: Path, files: List[Path], mode: str) -> P
         """
 
 
-class ArchiverFactory:
-    @classmethod
-    def get_archiver(cls, archive_format=ARCHIVE_DEFAULT_FORMAT):
-        """Factory Method to return the needed Archiver, based on archive format"""
-        archivers = {
-            "zip": ZipArchiver
-        }
-
-        return archivers[archive_format]()
-
-
 class ZipArchiver(ArchiverABC):
     def process_archive(self, archive_name: Path, files: List[Path], mode: str = ARCHIVE_MODE) -> Path:
         with ZipFile(archive_name, mode=mode, compression=ARCHIVE_ZIP_COMPRESSION) as archive:

ted_sws/notice_packager/services/notice_packager.py

@@ -21,7 +21,7 @@
 from ted_sws.notice_metadata_processor.model.metadata import ExtractedMetadata
 from ted_sws.notice_metadata_processor.services.xml_manifestation_metadata_extractor import \
     XMLManifestationMetadataExtractor
-from ted_sws.notice_packager.adapters.archiver import ArchiverFactory, ARCHIVE_ZIP_FORMAT
+from ted_sws.notice_packager.adapters.archiver import ZipArchiver
 from ted_sws.notice_packager.adapters.template_generator import TemplateGenerator
 from ted_sws.notice_packager.model.metadata import ACTION_CREATE
 from ted_sws.notice_packager.services.metadata_transformer import MetadataTransformer
@@ -115,7 +115,7 @@ def get_archive_name(self) -> str:
         return archive_name
 
     def pack(self) -> str:
-        archiver = ArchiverFactory.get_archiver(ARCHIVE_ZIP_FORMAT)
+        archiver = ZipArchiver()
         archive_path = self.tmp_dir_path / self.get_archive_name()
         package_path = archiver.process_archive(archive_path, self.files)
         raw_archive_content = package_path.read_bytes()