Skip to content

Comments

Development: Make sure ES schema is reset before test runs#3067

Merged
smatting merged 3 commits intodevelopfrom
smatting/dev-schema-updates
Feb 9, 2023
Merged

Development: Make sure ES schema is reset before test runs#3067
smatting merged 3 commits intodevelopfrom
smatting/dev-schema-updates

Conversation

@smatting
Copy link
Contributor

@smatting smatting commented Feb 9, 2023

Follow up to #2791

This PR updates local development test setup:

  • the ES schema is always reset before any local integration tests are run
  • all DB schemas are migrated when you run any test suite (was only the DB for the test suite's service before). This is most likely what we want.

This PR also removes am unhelpful warning in the -schema binaries.

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Feb 9, 2023
@smatting smatting force-pushed the smatting/dev-schema-updates branch from 67e9e74 to e309106 Compare February 9, 2023 10:55
@smatting smatting merged commit d7cd311 into develop Feb 9, 2023
@smatting smatting deleted the smatting/dev-schema-updates branch February 9, 2023 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants