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

EW-1019: TSP Testing #5375

Draft
wants to merge 41 commits into
base: main
Choose a base branch
from
Draft

EW-1019: TSP Testing #5375

wants to merge 41 commits into from
+935 −697

Conversation

mkreuzkam-cap
Copy link
Contributor

@mkreuzkam-cap mkreuzkam-cap commented Dec 4, 2024 •
edited
Loading

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.

mkreuzkam-cap and others added 30 commits December 4, 2024 10:26
@mkreuzkam-cap
Add empty line.
0ce5351
@mkreuzkam-cap
log token
7bbabe7
@mkreuzkam-cap
Remove pLimit temporarily for testing.
fcd8d6d
@mkreuzkam-cap
Manual batching of data.
49bfdda
@mkreuzkam-cap
Merge branch 'main' into EW-1019
628d8c3
@mkreuzkam-cap
Fix array size.
397dff6
@mkreuzkam-cap
Better logging.
e4548fc
@mkreuzkam-cap
Merge branch 'main' into EW-1019
634049c
@mkreuzkam-cap
Even more logs!
0f7acca
@mkreuzkam-cap
Logging of errors.
1c342a0
@mkreuzkam-cap
Use bulk operations for sync. (Very WIP!!!)
a9669a5
@SimoneRadtke-Cap
EW-1019 Fix Linter warnings
3633cc0
@SimoneRadtke-Cap
EW-1019 Fix linter warnings
ac863a7
@mkreuzkam-cap
Use bulk operations for migration.
551f0e6
@mkreuzkam-cap
Merge branch 'main' into EW-1019
3c805b4
@SimoneRadtke-Cap
EW-1019 Add docker compose file to gitignore
79c6a7c
@mkreuzkam-cap
fix linter warning
18217e6
@mkreuzkam-cap
Bulk migration for students.
643c843
@mkreuzkam-cap
Merge branch 'main' into EW-1019
0a38d25
@mkreuzkam-cap
Use configService directly and don't read values in constructor.
a04d22d
@mkreuzkam-cap
Add batching.
c16a7bf
@mkreuzkam-cap
Merge branch 'main' into EW-1019
92e1b49
@mkreuzkam-cap
remove console log, add comments
7dc774f
@mkreuzkam-cap
Clean up migration and move into service class.
2587c90
@mkreuzkam-cap
Clean up loggables.
470329c
@SimoneRadtke-Cap
EW.1019 Add saveAll to account-idm-service
4a163ae
@mkreuzkam-cap
adjust config var usage.
29534a2
@mkreuzkam-cap
adjust tests (wip)
8d19ebf
@mkreuzkam-cap
Fix findMany test in accoundIdm service.
a34c3a8
@mkreuzkam-cap
Fix tsp sync strategy test.
1b0627b
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
61.8% Coverage on New Code (required ≥ 80%)
58 Uncovered Lines on New Code (required ≤ 0)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mkreuzkam-cap @SimoneRadtke-Cap