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.

Sign up for GitHub

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

Jump to bottom

Return a dummy /versions response in tests #316

Merged

DMRobertson merged 1 commit into main from dmr/versions-response-in-tests

Sep 28, 2023

Contributor

DMRobertson commented Sep 28, 2023 •

edited

Loading

Otherwise every integration test logs a warning that isn't useful, since #286:

=== RUN   TestBackfillInviteDoesntCorruptState
Note: tests require a postgres install accessible to the current user
createdb: error: database creation failed: ERROR:  database "syncv3_test" already exists
12:29:34 WRN Could not contact upstream homeserver. Is SYNCV3_SERVER set correctly? error="/versions returned HTTP 404" dest=http://127.0.0.1:45461


          Return a dummy /versions response in tests

1079ec6

Otherwise every integration test logs a warning that isn't useful.

kegsay approved these changes

View reviewed changes

DMRobertson merged commit 0d59f01 into main

6 of 7 checks passed

DMRobertson deleted the dmr/versions-response-in-tests branch

September 28, 2023 11:49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet