Skip to content

Fix test failures in samples #2138

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

Merged
merged 2 commits into from
Sep 20, 2022
Merged

Conversation

vpavic
Copy link
Contributor

@vpavic vpavic commented Sep 16, 2022

This PR addresses two recently surfaced issues with samples:

  1. Spring Security configuration used by spring-session-sample-boot-mongodb-traditional stopped working which resulted in test failures as randomly generated password was used.

    • failure first observed on Sep 9th CI run
    • fixed in 674e76d by removing Spring Security configuration in favor of Spring Boot's auto-configuration
  2. Selenium tests in several Spring Boot based samples started failing with ClassNotFoundException due to incompatible versions of dependencies being present on the classpath.

    • failure first observed on Sep 16th CI run
    • fixed in be1172b by aligning dependencies across samples using Spring Boot's BOM

This commit aligns dependency versions used across different samples application by leveraging Spring Boot's BOM, rather than manually managing dependency versions for samples that are not Spring Boot based.
@rwinch rwinch added this to the 3.0.0-M3 milestone Sep 20, 2022
@rwinch rwinch self-assigned this Sep 20, 2022
@rwinch rwinch added type: bug A general bug in: build An issue in the build labels Sep 20, 2022
@rwinch rwinch merged commit cb6f5c9 into spring-projects:main Sep 20, 2022
@vpavic vpavic deleted the fix-samples branch September 20, 2022 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: build An issue in the build type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants