test: add tests for summary image removal feature

Author: ogajduse

pyproject.toml

@@ -159,6 +159,9 @@ disable_error_code = ["attr-defined", "import"]
 
 [tool.pytest.ini_options]
 pythonpath = [".", "tests"]
+filterwarnings = [
+  "ignore:Inheritance class HomeAssistantApplication from web.Application is discouraged:DeprecationWarning"
+]
 
 [tool.bumpver]
 current_version = "0.1.11"

tests/conftest.py

@@ -14,9 +14,23 @@ def get_feeds() -> list[FeedSource]:
 
 def pytest_generate_tests(metafunc: pytest.Metafunc) -> None:
     """Generate tests and fixtures."""
+    feeds = get_feeds()
     if "feed" in metafunc.fixturenames:
-        feeds = get_feeds()
         metafunc.parametrize("feed", feeds, ids=[f.name for f in feeds], indirect=True)
+    if "feed_with_image_in_summary" in metafunc.fixturenames:
+        feeds_with_image_in_summary = [
+            pytest.param(
+                feed,
+                id=feed.name,
+            )
+            for feed in feeds
+            if feed.has_images_in_summary
+        ]
+        metafunc.parametrize(
+            "feed_with_image_in_summary",
+            feeds_with_image_in_summary,
+            indirect=True,
+        )
 
 
 @pytest.fixture()
@@ -29,3 +43,11 @@ def feed(request: pytest.FixtureRequest) -> FeedSource:
 def feed_sensor(feed: FeedSource) -> FeedParserSensor:
     """Return feed sensor initialized with the local RSS feed."""
     return FeedParserSensor(**feed.sensor_config_local_feed)
+
+
+@pytest.fixture()
+def feed_with_image_in_summary(
+    request: pytest.FixtureRequest,
+) -> FeedSource:
+    """Return feed sensor with images in summary of its entries."""
+    return request.param

tests/constants.py

@@ -8,6 +8,7 @@
 TEST_FEEDS = [
     {
         "has_images": True,
+        "has_unique_images": False,
         "sensor_config": {
             "name": "CTK",
             "feed_url": "https://www.ceskenoviny.cz/sluzby/rss/cr.php",
@@ -54,6 +55,7 @@
     {
         "has_images": True,
         "all_entries_have_images": False,
+        "has_images_in_summary": True,
         "has_unique_links": False,
         "sensor_config": {
             "name": "buienradar_nl",
@@ -72,6 +74,7 @@
     },
     {
         "has_images": False,
+        "has_unique_dates": False,
         "sensor_config": {
             "name": "api_met_no_metalerts",
             "feed_url": "https://api.met.no/weatherapi/metalerts/1.1/",
@@ -85,13 +88,34 @@
         "sensor_config": {
             "name": "anp_nieuws",
             "feed_url": "https://www.omnycontent.com/d/playlist/56ccbbb7-0ff7-4482-9d99-a88800f49f6c/a49c87f6-d567-4189-8692-a8e2009eaf86/9fea2041-fccd-4fcf-8cec-a8e2009eeca2/podcast.rss",
-            "inclusions": ["title", "link", "published", "summary"],
+            "inclusions": ["title", "link", "published", "summary", "image"],
+        },
+    },
+    {
+        "has_images": True,
+        "all_entries_have_images": False,
+        "has_images_in_summary": True,
+        "all_entries_have_summary": False,
+        "sensor_config": {
+            "name": "alle_meldungen",
+            "feed_url": "https://rss.sueddeutsche.de/alles/",
+            "inclusions": ["image", "title", "link", "published", "summary"],
+            "remove_summary_image": True,
+        },
+    },
+    {
+        "has_images": True,
+        "has_unique_dates": False,
+        "sensor_config": {
+            "name": "stern_auto",
+            "feed_url": "https://www.stern.de/feed/standard/auto/",
+            "inclusions": ["image", "title", "link", "published", "summary"],
         },
     },
 ]
 
 DEFAULT_EXCLUSIONS: list[str] = []
-DEFAULT_INCLUSIONS = ["image", "title", "link", "published"]
+DEFAULT_INCLUSIONS = ["image", "title", "link", "summary", "published"]
 DATE_FORMAT = "%a, %d %b %Y %H:%M:%S UTC%z"
 
 URLS_HEADERS_REQUIRED = [

tests/data/alle_meldungen.json

@@ -0,0 +1,19 @@
+{
+    "has_images": true,
+    "all_entries_have_images": false,
+    "has_images_in_summary": true,
+    "all_entries_have_summary": false,
+    "sensor_config": {
+        "name": "alle_meldungen",
+        "feed_url": "https://rss.sueddeutsche.de/alles/",
+        "inclusions": [
+            "image",
+            "title",
+            "link",
+            "published",
+            "summary"
+        ],
+        "remove_summary_image": true
+    },
+    "download_date": "2024-02-23T21:08:09.183484+00:00"
+}

tests/data/alle_meldungen.xml

@@ -0,0 +1,16 @@
+{
+    "has_images": true,
+    "has_unique_dates": false,
+    "sensor_config": {
+        "name": "stern_auto",
+        "feed_url": "https://www.stern.de/feed/standard/auto/",
+        "inclusions": [
+            "image",
+            "title",
+            "link",
+            "published",
+            "summary"
+        ]
+    },
+    "download_date": "2024-02-23T21:08:09.183484+00:00"
+}