diff --git a/pyproject.toml b/pyproject.toml index 2defdb2..225540e 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -98,6 +98,8 @@ markers = [ line-length = 120 # Remove this for a more detailed lint report output-format = "concise" +# TODO: temp ignore rule for timestamping_enricher to allow for open PR +exclude = ["src/auto_archiver/modules/timestamping_enricher/*"] [tool.ruff.lint] @@ -105,16 +107,13 @@ output-format = "concise" # See documentation for more details: https://docs.astral.sh/ruff/rules/ #extend-select = ["B"] -# Ignore unused imports as some are currently required for lazy loading -# This can be removed for a `ruff check` run which is manually reviewed -#ignore = ["F401"] - [tool.ruff.lint.per-file-ignores] # Ignore import violations in __init__.py files "__init__.py" = ["F401", "F403"] # Ignore 'useless expression' in manifest files. "__manifest__.py" = ["B018"] + [tool.ruff.format] docstring-code-format = false diff --git a/src/auto_archiver/core/orchestrator.py b/src/auto_archiver/core/orchestrator.py index 8c7d112..672994a 100644 --- a/src/auto_archiver/core/orchestrator.py +++ b/src/auto_archiver/core/orchestrator.py @@ -486,7 +486,7 @@ Here's how that would look: \n\nsteps:\n extractors:\n - [your_extractor_name_ self.setup(args) return self.feed() except Exception as e: - logger.error(e, exc_info=True) + logger.error(e) exit(1) def cleanup(self) -> None: diff --git a/tests/databases/test_api_db.py b/tests/databases/test_api_db.py index 4627425..2e87a87 100644 --- a/tests/databases/test_api_db.py +++ b/tests/databases/test_api_db.py @@ -1,6 +1,5 @@ import pytest -from auto_archiver.core import Metadata from auto_archiver.modules.api_db import AAApiDb diff --git a/tests/databases/test_gsheet_db.py b/tests/databases/test_gsheet_db.py index 6c9e585..0760c79 100644 --- a/tests/databases/test_gsheet_db.py +++ b/tests/databases/test_gsheet_db.py @@ -54,7 +54,7 @@ def mock_media(mocker): @pytest.fixture -def gsheets_db(mock_gworksheet, setup_module, mocker): +def gsheets_db(mock_gworksheet, setup_module, mocker) -> GsheetsFeederDB: mocker.patch("gspread.service_account") config: dict = { "sheet": "testsheet", diff --git a/tests/extractors/test_twitter_api_extractor.py b/tests/extractors/test_twitter_api_extractor.py index 0664f49..8b8e0d9 100644 --- a/tests/extractors/test_twitter_api_extractor.py +++ b/tests/extractors/test_twitter_api_extractor.py @@ -1,6 +1,5 @@ import os import datetime -import hashlib import pytest from pytwitter.models.media import MediaVariant @@ -10,7 +9,7 @@ from auto_archiver.modules.twitter_api_extractor import TwitterApiExtractor @pytest.mark.incremental class TestTwitterApiExtractor(TestExtractorBase): - extractor_module = "twitter_api_extractor" + extractor_module: TwitterApiExtractor = "twitter_api_extractor" config = { "bearer_tokens": [], diff --git a/tests/storages/test_gdrive_storage.py b/tests/storages/test_gdrive_storage.py index 08c516f..501bd58 100644 --- a/tests/storages/test_gdrive_storage.py +++ b/tests/storages/test_gdrive_storage.py @@ -1,10 +1,8 @@ from typing import Type import pytest -from oauth2client import service_account from auto_archiver.core import Media from auto_archiver.modules.gdrive_storage import GDriveStorage -from auto_archiver.core.metadata import Metadata from tests.storages.test_storage_base import TestStorageBase diff --git a/tests/test_implementation.py b/tests/test_implementation.py index 973d53d..e52a8d8 100644 --- a/tests/test_implementation.py +++ b/tests/test_implementation.py @@ -46,7 +46,6 @@ def autoarchiver(tmp_path, monkeypatch, request): def test_run_auto_archiver_no_args(caplog, autoarchiver): with pytest.raises(SystemExit): autoarchiver() - assert "provide at least one URL via the command line, or set up an alternative feeder" in caplog.text diff --git a/tests/test_modules.py b/tests/test_modules.py index b6018da..f672ca6 100644 --- a/tests/test_modules.py +++ b/tests/test_modules.py @@ -1,4 +1,3 @@ -import sys import pytest from auto_archiver.core.module import ModuleFactory, LazyBaseModule from auto_archiver.core.base_module import BaseModule diff --git a/tests/test_orchestrator.py b/tests/test_orchestrator.py index a79aa70..326b93d 100644 --- a/tests/test_orchestrator.py +++ b/tests/test_orchestrator.py @@ -1,5 +1,4 @@ import pytest -import sys from argparse import ArgumentParser, ArgumentTypeError from auto_archiver.core.orchestrator import ArchivingOrchestrator from auto_archiver.version import __version__