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

Added tests for auth endpoints #15

Merged
merged 1 commit into from
Dec 2, 2021
Merged

Added tests for auth endpoints #15

merged 1 commit into from
Dec 2, 2021

Conversation

V3lop5
Copy link
Collaborator

@V3lop5 V3lop5 commented Dec 1, 2021

This PR ...

adds test for authentication endpoints


Checklist

Checklist for Contributors

  • Unit tests provided?
    Unit tests are needed for new features or to reproduce the bug solved by this PR.
  • Documentation updated?
  • Unit tests successfull
  • Code smells (flake8) checked

Checklist for Reviewers

  • Read PR description
  • Check linked issue
  • Check CI/CD (build, automated tests, linting, preview build)
  • Check documentation changes
  • Add labels to PR

@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2021

Unit Test Results

  1 files  ±0    1 suites  ±0   4s ⏱️ -1s
20 tests +5  20 ✔️ +5  0 💤 ±0  0 ±0 

Results for commit b2557dd. ± Comparison against base commit 7f5fb9e.

@codecov-commenter
Copy link

Codecov Report

Merging #15 (b2557dd) into main (8d8623c) will increase coverage by 4.15%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
+ Coverage   85.43%   89.58%   +4.15%     
==========================================
  Files          55       55              
  Lines         666      701      +35     
==========================================
+ Hits          569      628      +59     
+ Misses         97       73      -24     
Flag Coverage Δ
pytest 89.58% <100.00%> (+4.15%) ⬆️

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

Impacted Files Coverage Δ
ensysmod/model/energy_source.py 100.00% <ø> (ø)
ensysmod/api/endpoints/authentication.py 100.00% <100.00%> (+20.83%) ⬆️
tests/api/test_authentication.py 100.00% <100.00%> (ø)
ensysmod/crud/base.py 69.23% <0.00%> (+12.82%) ⬆️
ensysmod/api/deps.py 100.00% <0.00%> (+33.33%) ⬆️
ensysmod/core/security.py 92.30% <0.00%> (+38.46%) ⬆️

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 8d8623c...b2557dd. Read the comment docs.

@einfachMel einfachMel merged commit f9a530c into main Dec 2, 2021
@einfachMel einfachMel deleted the feature/test-auth branch December 2, 2021 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants