Skip to content

Conversation

@EtienneLt
Copy link
Collaborator

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • A PR or issue has been opened in all impacted repositories (if any)

Does this PR already have an issue describing the problem?
no

What kind of change does this PR introduce?
cleaning test

What is the current behavior?
for tests using springboot, spring is launched at each test in one class

What is the new behavior (if this is a feature change)?
launch spring only once in a test class

Does this PR introduce a breaking change or deprecate an API?

  • Yes
  • No

Signed-off-by: Etienne LESOT <[email protected]>
@EtienneLt EtienneLt self-assigned this Dec 5, 2025
@antoinebhs
Copy link
Contributor

antoinebhs commented Dec 5, 2025

This improves CI speed :
50s vs >3m on local computer

Signed-off-by: Etienne LESOT <[email protected]>
Signed-off-by: Etienne LESOT <[email protected]>
Signed-off-by: Etienne LESOT <[email protected]>
Signed-off-by: Etienne LESOT <[email protected]>
Signed-off-by: Etienne LESOT <[email protected]>
Copy link
Contributor

@antoinebhs antoinebhs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code OK

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 8, 2025

@EtienneLt EtienneLt merged commit 7ff0d92 into main Dec 8, 2025
5 checks passed
@EtienneLt EtienneLt deleted the clean-test-to-improve-speed branch December 8, 2025 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants