Skip to content
This repository was archived by the owner on Jun 13, 2025. It is now read-only.

Conversation

@seer-by-sentry
Copy link
Contributor

This PR adds tests for #945

Commits:

  • Add unit tests for SentimentAnalyzer class

@codecov-notifications
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@codecov
Copy link

codecov bot commented Oct 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.25%. Comparing base (9beb3f2) to head (5aabfbd).
Report is 3 commits behind head on launchweek-sentiment.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                  Coverage Diff                  @@
##           launchweek-sentiment     #946   +/-   ##
=====================================================
  Coverage                 96.24%   96.25%           
=====================================================
  Files                       824      825    +1     
  Lines                     19008    19033   +25     
=====================================================
+ Hits                      18295    18320   +25     
  Misses                      713      713           
Flag Coverage Δ
unit 92.51% <ø> (+0.01%) ⬆️
unit-latest-uploader 92.51% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rohitvinnakota-codecov rohitvinnakota-codecov marked this pull request as ready for review October 30, 2024 16:23
@rohitvinnakota-codecov rohitvinnakota-codecov merged commit fa4ca76 into launchweek-sentiment Oct 30, 2024
16 checks passed
@rohitvinnakota-codecov rohitvinnakota-codecov deleted the ai_tests_for_pr945_1730304131 branch October 30, 2024 16:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants