Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace feed cypress test with equivalent pytest #464

Merged
merged 4 commits into from
Dec 6, 2022

Conversation

brianpeiris
Copy link
Member

Detailed information:

  • Deleted feed.cy.js and replaced it with an equivalent pytest in app/feed/tests/test_feed_routes.py
  • Removed obsolete feedEndpoint cypress command.

Closing issues:

List all issues the pull request solve:

Test plan (required)

  • The PR contains new PyTest unit/integration tests for any function or functional added.
  • The PR changes existing PyTest unit/integration tests to keep all tests up to date.
  • The PR does not lead to degradation in unit test coverage.
  • Project parts affected by changes in this PR was tested manually on your local (using Postman or in any other way). List everything you've tested below:
    • ran all pytests

conftest.py Show resolved Hide resolved
@codecov-commenter
Copy link

Codecov Report

Base: 66.71% // Head: 66.71% // No change to project coverage 👍

Coverage data is based on head (3c46140) compared to base (ee6427e).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #464   +/-   ##
========================================
  Coverage    66.71%   66.71%           
========================================
  Files           81       81           
  Lines         2569     2569           
  Branches       313      313           
========================================
  Hits          1714     1714           
  Misses         828      828           
  Partials        27       27           
Impacted Files Coverage Δ
conftest.py 100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@danmash danmash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I'm glad we were figured out with lrf_data.

app/feed/tests/test_feed_routes.py Show resolved Hide resolved
@danmash danmash merged commit 7ed6cc4 into ClimateMind:develop Dec 6, 2022
@danmash danmash mentioned this pull request Dec 6, 2022
3 tasks
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.

Replace cypress feed.cy.js with PyTest integration tests.
3 participants