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.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: define a InMemoryConfiguration that can be shared #6871

Merged
merged 1 commit into from Mar 8, 2024

Conversation

jgiovaresco
Copy link
Contributor

@jgiovaresco jgiovaresco commented Mar 8, 2024

Description

Each rest module was defining its own InMemoryConfiguration. It was a pain to introduce new InMemory implementation because we had to add the declaration in 3 files.
Now, there is only one InMemoryConfiguration located next to in-memory alternatives. This configuration is packaged in test artifact and can be referenced in Spring configuration in each rest module.

Additional context


馃摎聽聽View the storybook of this branch here

@jgiovaresco jgiovaresco requested a review from a team as a code owner March 8, 2024 08:56
Each rest module was defining its own InMemoryConfiguration. It was a
pain when we introduce new InMemory implementation because we had to
add the declaration in 3 files.
Now there is only one InMemoryConfiguration located next to in memory
alternatives. This configuration is packaged in test artifact and can be
referenced in Spring configuration in each rest module.
Copy link
Contributor

@jourdiw jourdiw left a comment

Choose a reason for hiding this comment

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

A sanity saver 馃檶

@jgiovaresco jgiovaresco merged commit e8df8f0 into master Mar 8, 2024
28 checks passed
@jgiovaresco jgiovaresco deleted the simply-spring-init-in-test branch March 8, 2024 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants