-
Notifications
You must be signed in to change notification settings - Fork 120
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
add tests #203
Conversation
.github/workflows/pull-request.yml
Outdated
run: | | ||
make db-migrate & | ||
npx playwright install --with-deps & | ||
make test-e2e |
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.
ты можешь сделать отдельную джобу в воркфлоу, чтобы у тебя параллельно работали тесты обычные и е2е. Плюс здесь команда слишком страшная, можно обернуть в некий make ci-e2e-tests
.
Плюс команды через амперсанд запускаются. А так должно быть?
package.json
Outdated
@@ -6,8 +6,9 @@ | |||
"scripts": { | |||
"build": "npm run build --prefix frontend && npm run build --prefix backend", | |||
"start:prod": "npm run start:prod --prefix backend", | |||
"test": "echo \"Error: no test specified\" && exit 1", | |||
"preinstall": "npm ci --prefix frontend && npm ci --prefix backend" | |||
"preinstall": "npm ci --prefix frontend && npm ci --prefix backend", |
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.
по семантике preinstall некорректно использовать для установки зависимостей. Это скорее для всяких хуков-скриптов.
Можно попробовать в виде воркспейсов тут
вот как у нас в домашках было
"workspaces": [
"./exercises/*"
],
No description provided.