-
Notifications
You must be signed in to change notification settings - Fork 0
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
fix: issues identified by sonarqube #157
base: main
Are you sure you want to change the base?
Conversation
cgawron
commented
Oct 4, 2024
- increase memory limit for backend
- use snackbar for error messages in client
- increase memory limit for backend - use snackbar for error messages in client
Test Results2 files 2 suites 0s ⏱️ Results for commit 0b8504b. ♻️ This comment has been updated with latest results. |
- update dependencies & `yarn.lock`
- fix basic test cases
Test Summary - Run #181
❌ Some tests failed!
|
- fix environment variables for tests
Test Summary - Run #182
❌ Some tests failed!
|
- since authentication is handled externally, the `password` attribute is no longer needed
Test Summary - Run #183
❌ Some tests failed!
|
- set time zone to `Europe/Berlin` for cypress tests
Test Summary - Run #184
🎉 All tests passed! |
- fix configuration to enable coverage in cypress tests
Test Summary - Run #185
🎉 All tests passed! |
- add additional cypress tests
Test Summary - Run #186
❌ Some tests failed!
|
- upgrade axios - remove node-sass
Test Summary - Run #187
❌ Some tests failed!
|
- fixed various issues spotted by sonarqube
Test Summary - Run #188
❌ Some tests failed!
|
- fixed various issues spotted by sonarqube
Test Summary - Run #189
❌ Some tests failed!
|
- fixed various issues spotted by sonarqube
- remove sonarqube token from properties file (will be provided via GH secret) - add additional coverage paths
Test Summary - Run #190
❌ Some tests failed!
|
- remove (unused) logic to skip steps
- add cypress tests to improve coverage
Test Summary - Run #191
❌ Some tests failed!
|
- exclude backend from coverage for now
Test Summary - Run #192
❌ Some tests failed!
|
- add cypress tests to improve coverage
- remove unused `ChooseTime` component
Test Summary - Run #193
❌ Some tests failed!
|
- exclude client helper services from coverage for now
Test Summary - Run #194
❌ Some tests failed!
|