Skip to content

(test) (fix) fix failing tests, update newsspaper.hot/google trends rss feed#665

Open
BRNMan wants to merge 2 commits intoAndyTheFactory:masterfrom
BRNMan:local-testing-fix
Open

(test) (fix) fix failing tests, update newsspaper.hot/google trends rss feed#665
BRNMan wants to merge 2 commits intoAndyTheFactory:masterfrom
BRNMan:local-testing-fix

Conversation

@BRNMan
Copy link

@BRNMan BRNMan commented Mar 24, 2025

Related Issues

N/A

Proposed Changes:

A few tests were failing due to websites updating or going down. I fixed them and added an updated link to google trends for newspaper.hot().

How did you test it?

Ran the tests again. Ran newspaper.hot() and checked output.

Notes for the reviewer

I didn't fix the gnews test because it confused me and discovered an issue with category parsing that popped up in test_source which I will document under issues.

Checklist

  • I have updated the related issue with new insights and changes
  • I added unit tests and updated the docstrings
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.
  • I documented my code
  • I ran pre-commit hooks and fixed any issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant