umputun / newscope / 32271360150

81%

master: 81%

Pull #43

github

Fix panic when content extraction is disabled

Previously, main passed a nil *content.HTTPExtractor to the scheduler when
extraction.enabled was false, producing a non-nil Extractor interface with a
nil pointer inside and panicking on the first processed item.

The extractor is now kept as scheduler.Extractor and stays nil when extraction
is off. The feed processor then skips extraction and classifies articles from
the content provided by the feed, ExtractContentNow reports that extraction is
disabled, and the startup pass over items needing extraction is skipped because
nothing would ever mark those items as extracted.

27 of 32 new or added lines in 3 files covered. (84.38%)

3748 of 4620 relevant lines covered (81.13%)

27.86 hits per line