-
Notifications
You must be signed in to change notification settings - Fork 24
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: add migrate db seed test #2048
Conversation
Based on #1807
I think we should remove the enginetype from the matrix here, it doesn鈥檛 bring any value I can see (it's tested in other tests and that would reduce the number of jobs here) |
|
||
set -eux | ||
|
||
yarn prisma db pull --print |
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.
confirmation that db push
worked?
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.
It's a copy paste from your script, so probably
|
||
# generate not needed, it will be done by db push | ||
# yarn prisma generate | ||
yarn prisma db push --force-reset |
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.
if force-reset
does not work for whatever reason, the test would possibly also succeed if the seeding fails, correct?
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.
Well here it's about preparing the db and I assume test.sh runs after with
#!/bin/sh
set -eux
yarn prisma db pull --print
yarn prisma db seed
yarn test
This should fail
Based on and replaces #1807
closes #1806
@janpio this adds 12 jobs 馃槰