Development: Enable parallel server test execution
#6796
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DominikRemo
changed the title
Development: Enable parallel Unit TestingDevelopment: Enable parallel Testing
DominikRemo
changed the title
Development: Enable parallel TestingDevelopment: Enable parallel test execution
Note: Since this change affects the test execution order, some tests might fail. I commit this for asserting that the tests actually run (fail) faster with parallel execution enabled on bamboo and github actions. The tests will be fixed in another commit
src/main/java/de/tum/in/www1/artemis/service/scheduled/NotificationScheduleService.java
Outdated
Show resolved
Hide resolved
…-testing' into development/enable-parallel-unit-testing
Strohgelaender
previously approved these changes
Sep 21, 2023
terlan98
reviewed
Sep 21, 2023
src/test/java/de/tum/in/www1/artemis/notification/NotificationResourceIntegrationTest.java
Show resolved
Hide resolved
terlan98
previously approved these changes
Sep 21, 2023
DominikRemo
dismissed stale reviews from terlan98 and Strohgelaender
via
b8b389d
DominikRemo
requested review from
Strohgelaender,
MaximilianAnzinger and
terlan98
MaximilianAnzinger
previously approved these changes
Sep 21, 2023
terlan98
previously approved these changes
Sep 21, 2023
Strohgelaender
previously approved these changes
Sep 22, 2023
DominikRemo
dismissed stale reviews from Strohgelaender, terlan98, and MaximilianAnzinger
via
e042540
github-actions
bot
added
the
client
krusche
changed the title
Development: Enable parallel test executionDevelopment: Enable parallel server test execution
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checklist
General
Server
Motivation and Context
Fast tests 🏎️
Improves server-test execution time from up to 30 min 🚗 => down to 12 min 🏎️
Description
Enables parallel testing with JUnit 5 JUnit Docs: 2.20. Parallel Execution.
Implements a new JUnit 5 extension and custom Logback appender for logging in parallel tests.
Fixes several flaky tests (e.g.: in classes LongFeedbackResourceIntegrationTest, NotificationScheduleServiceTest and ProgrammingExerciseScheduleServiceTest)
Re-organized tests into different groups for better balancing.
We used a custom plan that only executes the server tests. Besides only running the server tests, the Server Tests Only plan is similar to the Artemis Test plan
Steps for Testing
Test changes only
Review Progress
Code Review