Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

test: fixed test case that reset entire mock server instead of only requests#786

Merged
olavloite merged 1 commit intomainfrom
issue-754
Mar 21, 2022
Merged

test: fixed test case that reset entire mock server instead of only requests#786
olavloite merged 1 commit intomainfrom
issue-754

Conversation

@olavloite
Copy link
Copy Markdown
Collaborator

The PgNumericPreparedStatementTest reset the entire mock server after each test, instead of only clearing the requests on the mock server. This would also clear all sessions on the mock server, which could cause a Session not found error to occur occasionally.

Fixes #754

@olavloite olavloite requested review from a team and ansh0l March 14, 2022 09:22
@product-auto-label product-auto-label Bot added the api: spanner Issues related to the googleapis/java-spanner-jdbc API. label Mar 14, 2022
@olavloite olavloite merged commit 30023bf into main Mar 21, 2022
@olavloite olavloite deleted the issue-754 branch March 21, 2022 07:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spanner Issues related to the googleapis/java-spanner-jdbc API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

spanner.jdbc.PgNumericPreparedStatementTest: many tests failed

2 participants