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

Add test data and read all endpoints for all psql models #29

Merged
merged 4 commits into from
Dec 19, 2021

Conversation

AndreasPB
Copy link
Owner

@AndreasPB AndreasPB added the enhancement New feature or request label Dec 18, 2021
@AndreasPB AndreasPB self-assigned this Dec 18, 2021
@codecov-commenter
Copy link

codecov-commenter commented Dec 18, 2021

Codecov Report

Merging #29 (7e7b0fa) into master (70d45f8) will increase coverage by 10.29%.
The diff coverage is 91.22%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #29       +/-   ##
===========================================
+ Coverage   68.32%   78.61%   +10.29%     
===========================================
  Files          23       25        +2     
  Lines         682      898      +216     
  Branches       13       13               
===========================================
+ Hits          466      706      +240     
+ Misses        213      189       -24     
  Partials        3        3               
Impacted Files Coverage Δ
backend/app/main.py 70.00% <45.45%> (-2.14%) ⬇️
backend/app/db/psql_test_data.py 90.69% <90.69%> (ø)
backend/app/routers/test_psql.py 93.75% <93.75%> (ø)
backend/app/db/crud.py 50.99% <96.55%> (+15.33%) ⬆️
backend/app/routers/attendance.py 78.78% <100.00%> (+13.27%) ⬆️
backend/app/routers/course.py 78.78% <100.00%> (+13.27%) ⬆️
backend/app/routers/course_lesson.py 80.00% <100.00%> (+18.09%) ⬆️
backend/app/routers/lecturer.py 78.78% <100.00%> (+13.27%) ⬆️
backend/app/routers/lecturer_studentclass.py 80.00% <100.00%> (+18.09%) ⬆️
backend/app/routers/lesson.py 78.78% <100.00%> (+13.27%) ⬆️
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 70d45f8...7e7b0fa. Read the comment docs.

@AndreasPB AndreasPB force-pushed the feature/add_test_data_for_psql_and_tests branch 7 times, most recently from 888cde3 to edfb3c5 Compare December 19, 2021 01:55
@AndreasPB AndreasPB force-pushed the feature/add_test_data_for_psql_and_tests branch from edfb3c5 to c9b3131 Compare December 19, 2021 02:23
@Primdahl26 Primdahl26 self-requested a review December 19, 2021 02:31
Copy link
Collaborator

@Primdahl26 Primdahl26 left a comment

Choose a reason for hiding this comment

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

:shipit:

@AndreasPB AndreasPB force-pushed the feature/add_test_data_for_psql_and_tests branch from c9b3131 to 7e7b0fa Compare December 19, 2021 02:31
@AndreasPB AndreasPB merged commit a7362c1 into master Dec 19, 2021
This was linked to issues Dec 19, 2021
@AndreasPB AndreasPB deleted the feature/add_test_data_for_psql_and_tests branch December 23, 2021 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mock data for PostgreSQL
3 participants