summaryrefslogtreecommitdiff
path: root/app/src/main/java/org/terst/feedreader/readability/ExtractionFailure.kt
diff options
context:
space:
mode:
authorPeter Stone <thepeterstone@gmail.com>2026-08-24 07:58:47 +0000
committerPeter Stone <thepeterstone@gmail.com>2026-08-24 07:58:47 +0000
commit209da9e628112a5a25ac4519370cfdc072462547 (patch)
treeb88d207bc2f1dafe9a04d4621e9fb9ccd2e3906b /app/src/main/java/org/terst/feedreader/readability/ExtractionFailure.kt
parentda942624a4abd82797f563cf8c57f1124c5de435 (diff)
Let filter rules be scoped to one feed; fix coverage-gate report path bug
The data model and FilterRuleEngine already supported a per-feed scope (FilterRuleEntity.feedId), but Settings' "Add rule" form never let a user set it -- only a hardcoded migration had ever created one. Add a FeedScopePicker so a rule can target "All feeds" (unchanged default) or one specific feed, and show each existing rule's scope in the list. Added the first real unit tests in this repo (FilterRuleEngineTest), covering global vs. feed-scoped matching and mute-wins-over-highlight priority -- the exact behavior this UI gap left unexercised outside one migration-seeded row. First real jacoco coverage number: 2.31%. While verifying, found the pre-commit hook's coverage step was silently a no-op: it pointed at app/build/reports/jacoco/report.xml, but the jacoco setup that landed here uses the standard Gradle JacocoReport task, which writes to reports/jacoco/jacocoTestReport/jacocoTestReport.xml instead. Fixed the path so the gate actually reads real numbers now. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017GMEkHeqKz6FLkmizowBTK
Diffstat (limited to 'app/src/main/java/org/terst/feedreader/readability/ExtractionFailure.kt')
0 files changed, 0 insertions, 0 deletions