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

[tests-only] Test with postgres:9.4 #38685

Closed
wants to merge 2 commits into from
Closed

Conversation

phil-davis
Copy link
Contributor

to see if the problem fixed in #38682 causes any tests to fail in CI

@phil-davis phil-davis self-assigned this May 3, 2021
@sonarcloud
Copy link

sonarcloud bot commented May 3, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@phil-davis
Copy link
Contributor Author

phil-davis commented May 4, 2021

Everything passes with postgres:9.4

@phil-davis
Copy link
Contributor Author

phil-davis commented May 4, 2021

With postgres:9.6
https://drone.owncloud.com/owncloud/core/29759/151/15
webUIWebdavLocks/unlock.feature:205 fails. But that is already tagged @skipOnDbPgsql - we do not expect that to pass anyway, for different reasons:
"Pgsql and Oracle database can have different sorting, so allow this scenario to be skipped on those"

And https://drone.owncloud.com/owncloud/core/29759/84/13 apiTrashbin/trashbinFilesFoldersOc10Issue23151.feature:44 failed. That is issue #23151 and the bug-demo test is known to be not reliable on postgres and Oracle.

Everything else passes. So we do not seem to have a test scenario that exposes the bug that is being fixed in #38682

@phil-davis phil-davis closed this May 4, 2021
@phil-davis phil-davis deleted the test-with-postgres-9.4 branch May 4, 2021 11:04
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.

None yet

1 participant