test(ingestion): update test for removed Ampel risk-level check
Ampel system removed – all enabled feeds are now processed regardless of risk_level. Updated test to verify feeds with any risk_level are processed instead of blocked. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
6192f8e527
commit
0a9c0b10d6
1 changed files with 14 additions and 9 deletions
|
|
@ -98,11 +98,11 @@ class TestIngestion(unittest.TestCase):
|
||||||
|
|
||||||
@patch("backend.app.ingestion.extract_article")
|
@patch("backend.app.ingestion.extract_article")
|
||||||
@patch("backend.app.ingestion.feedparser.parse")
|
@patch("backend.app.ingestion.feedparser.parse")
|
||||||
def test_ingestion_blocks_non_green_source(self, mock_parse, mock_extract_article) -> None:
|
def test_ingestion_processes_any_enabled_source(self, mock_parse, mock_extract_article) -> None:
|
||||||
# Re-create source/feed with yellow risk to verify enforcement
|
# Ampel/risk-level system removed – all enabled feeds are processed regardless of risk_level
|
||||||
source_id = create_source(
|
source_id = create_source(
|
||||||
SourceCreate(
|
SourceCreate(
|
||||||
name="Blocked Source",
|
name="Any Risk Source",
|
||||||
base_url="https://example.net",
|
base_url="https://example.net",
|
||||||
terms_url="https://example.net/terms",
|
terms_url="https://example.net/terms",
|
||||||
license_name="custom",
|
license_name="custom",
|
||||||
|
|
@ -112,20 +112,25 @@ class TestIngestion(unittest.TestCase):
|
||||||
last_reviewed_at="2026-02-18T00:00:00Z",
|
last_reviewed_at="2026-02-18T00:00:00Z",
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
blocked_feed_id = create_feed(
|
feed_id = create_feed(
|
||||||
FeedCreate(
|
FeedCreate(
|
||||||
name="Blocked Feed",
|
name="Any Risk Feed",
|
||||||
url="https://example.net/feed.xml",
|
url="https://example.net/feed.xml",
|
||||||
source_id=source_id,
|
source_id=source_id,
|
||||||
is_enabled=True,
|
is_enabled=True,
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
|
||||||
stats = run_ingestion(feed_id=blocked_feed_id)
|
mock_parse.return_value = type("FP", (), {"entries": [], "etag": None, "modified": None})()
|
||||||
|
mock_extract_article.return_value = type("E", (), {
|
||||||
|
"title": None, "author": None, "summary": None, "content_text": None,
|
||||||
|
"canonical_url": None, "images": [], "press_contact": None,
|
||||||
|
})()
|
||||||
|
|
||||||
|
stats = run_ingestion(feed_id=feed_id)
|
||||||
self.assertEqual(stats.status, "success")
|
self.assertEqual(stats.status, "success")
|
||||||
self.assertEqual(stats.articles_upserted, 0)
|
# Feed was processed (feedparser was called), even with yellow risk_level
|
||||||
mock_parse.assert_not_called()
|
mock_parse.assert_called_once()
|
||||||
mock_extract_article.assert_not_called()
|
|
||||||
|
|
||||||
@patch("backend.app.ingestion.extract_article")
|
@patch("backend.app.ingestion.extract_article")
|
||||||
@patch("backend.app.ingestion.feedparser.parse")
|
@patch("backend.app.ingestion.feedparser.parse")
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue