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
qe: use one db per test on cockroachdb #3597
Conversation
This is the recommended approach.
f6413a8
to
32ee6b9
Compare
821b7e7
to
3d37b56
Compare
@@ -30,6 +30,7 @@ mod metrics { | |||
Sqlite => assert_eq!(total_queries, 9), | |||
SqlServer(_) => assert_eq!(total_queries, 15), | |||
MongoDb(_) => assert_eq!(total_queries, 5), | |||
ConnectorVersion::CockroachDb => assert_eq!(total_queries, 10), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't dug to the bottom of this, but it would make sense for it to be from DROP DATABASE + CREATE DATABASE on test initialization becoming 1 query (instead of 2).
@@ -10,6 +10,7 @@ on: | |||
- 'LICENSE' | |||
- 'CODEOWNERS' | |||
- 'renovate.json' | |||
- 'introspection-engine/**' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is an interesting addition - avoiding running the QE tests on IE only changes I assume?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, not reflected but it seems like a small win.
/shrug |
And move to GH actions.
See individual commits for details.