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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PI-2616 Don't attempt to drop schema when using TestContainers #4373

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

marcus-bcl
Copy link
Contributor

This resolves connection errors at the end of the integration tests, and it shaves ~1 minute off the build time for each project - which is a nice saving

  • Before:
SPRING_PROFILES_ACTIVE=integration-test,oracle ./gradlew clean approved-premises-and-delius:integrationTest --no-build-cache
...
BUILD SUCCESSFUL in 3m 41s
89 actionable tasks: 26 executed, 63 up-to-date
  • After:
SPRING_PROFILES_ACTIVE=integration-test,oracle ./gradlew clean approved-premises-and-delius:integrationTest --no-build-cache
...
BUILD SUCCESSFUL in 2m 46s
89 actionable tasks: 26 executed, 63 up-to-date

This resolves connection errors at the end of the integration tests, and it shaves ~1 minute off the build time for each project - which is a nice saving

* Before:
```
SPRING_PROFILES_ACTIVE=integration-test,oracle ./gradlew clean approved-premises-and-delius:integrationTest --no-build-cache
...
BUILD SUCCESSFUL in 3m 41s
89 actionable tasks: 26 executed, 63 up-to-date
```

* After:
```
SPRING_PROFILES_ACTIVE=integration-test,oracle ./gradlew clean approved-premises-and-delius:integrationTest --no-build-cache
...
BUILD SUCCESSFUL in 2m 46s
89 actionable tasks: 26 executed, 63 up-to-date
```
@marcus-bcl marcus-bcl requested a review from a team as a code owner October 31, 2024 15:08
@github-actions github-actions bot added the bug Something isn't working label Oct 31, 2024
@marcus-bcl marcus-bcl enabled auto-merge (squash) October 31, 2024 15:26
@marcus-bcl marcus-bcl merged commit ad989e5 into main Oct 31, 2024
64 checks passed
@marcus-bcl marcus-bcl deleted the PI-2616-drop-oracle branch October 31, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants