Skip to content
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

Tests: Initial e2e tests setup #332

Merged
merged 7 commits into from
Aug 20, 2020
Merged

Tests: Initial e2e tests setup #332

merged 7 commits into from
Aug 20, 2020

Conversation

sorovlad
Copy link

@sorovlad sorovlad commented Aug 13, 2020

Configuration for start e2e tests on local console, editors or travise CI.
Added command yarn test:e2e to run this tests.
Added two examples "add trigger" and "navigate to add trigger's page".

Description:
Check users' scenarios use chromium and real API. This helps to check that front and api are working together correctly after updates.
To run locally, you need to install docker.
You don't need to start API and webpack-dev-server himself, jest runs them after lunch.

Add "Add trigger" test
Add "Navigate to Add trigger page" test
@sorovlad sorovlad marked this pull request as draft August 13, 2020 10:07
@sorovlad sorovlad changed the title Tests: Init config for integration tests Tests: Init configuration integration tests Aug 13, 2020
@sorovlad sorovlad changed the title Tests: Init configuration integration tests Tests: Initial integration tests setup Aug 13, 2020
@@ -18,7 +18,7 @@ export default function NewTagBadge(props: Props): React.Node {
return (
<div className={cn({ tag: true, removeable: onRemove, focused: focus })}>
{onClick ? (
<button type="button" onClick={onClick} className={cn("title", "clickable")}>
<button type="button" onClick={onClick} className={cn("title", "clickable")} data-tid="New Tag">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Пробелы в названии - это ок?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not/
Код выглядит немного чище, чем с разделителем дефисом.
Но это вскусовщина. Если считаешь лучше другая нотация, не проблема поменять.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Если ни на что не влияет, то норм

@sorovlad sorovlad marked this pull request as ready for review August 14, 2020 10:07
@sorovlad sorovlad changed the title Tests: Initial integration tests setup Tests: Initial e2e tests setup Aug 14, 2020
@sorovlad sorovlad merged commit 4134810 into feature/typescript Aug 20, 2020
@sorovlad sorovlad deleted the tests/e2e branch August 20, 2020 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants