-
Notifications
You must be signed in to change notification settings - Fork 18
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
test: Add sipi auth tests for knora.json (NO-Ticket) #2711
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #2711 +/- ##
===========================================
+ Coverage 17.89% 82.38% +64.48%
===========================================
Files 277 279 +2
Lines 28855 28936 +81
===========================================
+ Hits 5165 23838 +18673
+ Misses 23690 5098 -18592
☔ View full report in Codecov by Sentry. |
3e29f1b
to
3ebf0bb
Compare
3ebf0bb
to
bcec5ab
Compare
suite("Endpoint /{prefix}/{identifier}/knora.json")( | ||
suite("Given the user is unauthorized")( | ||
suite("And given a .info file exists in Sipi")( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just wonder if this test description could/should be in one suite, since there is only on test here?
Pull Request Checklist
Task Description/Number
Issue Number: DEV-
Basic Requirements
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
Does this PR introduce a breaking change?
Does this PR change client-test-data?