-
Notifications
You must be signed in to change notification settings - Fork 8
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
KBC-587 parallel tests #448
base: master
Are you sure you want to change the base?
Conversation
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.
imho ok, uvidime teda ci to je teda naozaj nejaky bug ked lokalne mi ten test presiel
Jak pustit: |
634296e
to
988cdbf
Compare
e2ec48f
to
8ee1853
Compare
Změnil sem ten PR, parallelní testy mají vlastní namespace, je přesunutá jde to pustit takto: je předtím potřeba vyexportovat envy postup by byl asi takový:
|
Změnil sem ten PR a asi bysme to měli probrat na team callu, takže vracám na review @keboola/team-connection a dávám PR jako draft. |
JIRA: https://keboola.atlassian.net/browse/KBC-587
parallelní testy mají vlastní namespace, je jako example přesunutá BucketsTest do tests/Parallel/Backend/Common
jde to pustit takto: je předtím potřeba vyexportovat envy source ./set-env.sh
docker-compose run --rm dev vendor/bin/paratest tests/Parallel/Backend/Common -p 16 --functional
postup by byl asi takový:
nasetujeme projekt pro každý backend
začnem přesouvat testy do parallel, když to jede přesunout pokud nějaký test padá nechat v původní test classe a vyřešit v budoucnu, moc sa stým nedrbat.