Skip to content

DATAJDBC-188 - Ensure proper behavior of delete logic. #51

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

Closed
wants to merge 2 commits into from

Conversation

schauder
Copy link
Contributor

No behavior changed.
Added tests on various levels for deleting entities with references to other entities.

@schauder schauder force-pushed the issue/DATAJDBC-188 branch from c9dc1a0 to 83856bf Compare March 19, 2018 08:37
schauder added 2 commits May 22, 2018 14:15
No behavior changed.
Added tests on various levels for deleting entities with references to other entities.
@schauder schauder force-pushed the issue/DATAJDBC-188 branch from 83856bf to a88dc63 Compare May 22, 2018 12:25
@gregturn gregturn closed this May 23, 2018
@gregturn gregturn deleted the issue/DATAJDBC-188 branch May 23, 2018 20:17
@gregturn
Copy link
Contributor

Resolved via 0460f2f

mp911de pushed a commit that referenced this pull request Feb 21, 2022
Postgres integration tests now run either with a locally installed and started database or if no such database can be found an instance gets started in a Docker container via Testcontainers.

We prefer a database based on TestContainers. Only if this can't be obtained do we try to access a local database for Postgres.

This minimizes the risk of accessing a database during tests that is not intended for that purpose.

If the system property `spring.data.r2dbc.test.preferLocalDatabase` is set to "true" the local database is preferred.

Original pull request: #51.
mp911de added a commit that referenced this pull request Feb 21, 2022
Add author tags. Add static factory method to create an unavailable database object instance. Simplify database selection. Javadoc.

Original pull request: #51.
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.

2 participants