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.

Sign up for GitHub

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

Jump to bottom

BC-7871 - Use test api client in api tests #5413

Merged
merged 39 commits into from
Jan 13, 2025
Merged

BC-7871 - Use test api client in api tests #5413

merged 39 commits into from
Jan 13, 2025

Conversation

bischofmax
Copy link
Contributor

Description

Links to Tickets or other pull requests

Approval for review

  • DEV: If api was changed - generate-client:server was executed in vue frontend and changes were tested and put in a PR with the same branch name.
  • QA: In addition to review, the code has been manually tested (if manual testing is possible)
  • All points were discussed with the ticket creator, support-team or product owner. The code upholds all quality guidelines from the PR-template.

@bischofmax bischofmax self-assigned this Jan 6, 2025
bischofmax and others added 27 commits January 6, 2025 14:53
@bischofmax
Use test api client for sharing create token and submission api test
45692b9
@bischofmax
Use test api client in task copy api spec
40059ef
@bischofmax
Use test api client in dashboard api spec
746429e
@SevenWaysDP
Refactor user-me API tests to use TestApiClient and improve authentic…
d4c59f6 
…ation handling
@bergatco
BC-7871 - Use test api client in task-finished api test
3f030a4
@SevenWaysDP
Refactor JWT and CurrentUser decorators tests to use TestApiClient fo…
bffd812 
…r improved authentication handling
@SevenWaysDP
Merge branch 'BC-7871' of https://github.com/hpi-schul-cloud/schulclo…
2c42be1 
…ud-server into BC-7871
@bischofmax
Use test api client in admin api students spec
38bad43
@bischofmax
Merge branch 'BC-7871' of github.com:hpi-schul-cloud/schulcloud-serve…
a88745c 
…r into BC-7871
@bischofmax
Merge branch 'main' into BC-7871
4b2de19
@bergatco
BC-7871 - Use test api client in admin-api-teachers api test
64647c8
@bischofmax
Remove override of xapikey guard
f5a93b9
@bischofmax
Fix imports
897679f
@SevenWaysDP
Add SCJwtTestFactory for JWT handling in tests
cfeaeb0
@SevenWaysDP
WIP test
58c4c7d
@bischofmax
Use domain for audience and issuer
0299f26
@bischofmax
Use new auth in complete list files
e680269
@bischofmax
Use test client in copy files
d408c02
@SevenWaysDP
Refactor JWT handling in tests: replace SCJwtTestFactory with JwtAuth…
186b381 
…enticationFactory and remove unused JwtValidationAdapter
@bischofmax
Use test api client in delete file spec
e433716
@bischofmax
Merge branch 'BC-7871' of github.com:hpi-schul-cloud/schulcloud-serve…
2c57cb2 
…r into BC-7871
@bischofmax
Use test api client for download upload spec
811ce2e
@bischofmax
Use test api client in rename file spec
68cd63b
@bischofmax
Use api client in restore files
7f44a29
@bischofmax
Use test api client in fwu learning content test
a318c9c
@bergatco
BC-7871 - Use test api clien tin files-storage-preview api tests
0387fef
@bergatco
BC-7871 - improve test structure of files-storage-preview api tests
b18e38b
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@bischofmax bischofmax merged commit 201c998 into main Jan 13, 2025
76 checks passed
@bischofmax bischofmax deleted the BC-7871 branch January 13, 2025 13:53
HKayed pushed a commit that referenced this pull request Jan 30, 2025
@bischofmax @HKayed
BC-7871 - Test authentication in api tests by removing guard override (…
60ab9f2 
…#5413)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SevenWaysDP SevenWaysDP SevenWaysDP approved these changes

@bergatco bergatco bergatco approved these changes

Assignees

@bischofmax bischofmax

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bischofmax @SevenWaysDP @bergatco