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

Run visual tests also againts local servers #1186

Merged
merged 16 commits into from
Jan 2, 2024
Merged

Conversation

literat
Copy link
Collaborator

@literat literat commented Dec 13, 2023

Description

Introducing a new package spirit-common where are stored shared features, scripts and code.

E2E test can be now run against production URLs of development ones. Just set env variable CI to true for production urls.

Additional context

The visual regression testing will be only run when the e2e label is used in PR.

Issue reference

https://jira.lmc.cz/browse/DS-1093


Before submitting the PR, please make sure you do the following

  • Read the Contributing Guidelines.
  • Follow the PR Title/Commit Message Convention.
  • Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
  • Ideally, include relevant tests that fail without this PR but pass with it.

Copy link

netlify bot commented Dec 13, 2023

Deploy Preview for spirit-design-system-storybook canceled.

Name Link
🔨 Latest commit ecf4b82
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-storybook/deploys/659416fdceceed00080bf54f

@github-actions github-actions bot added the feature New feature or request label Dec 13, 2023
Copy link

netlify bot commented Dec 13, 2023

Deploy Preview for spirit-design-system-demo canceled.

Name Link
🔨 Latest commit ecf4b82
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-demo/deploys/659416fd81bd4f0008a94680

Copy link

netlify bot commented Dec 13, 2023

Deploy Preview for spirit-design-system-validations canceled.

Name Link
🔨 Latest commit ecf4b82
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-validations/deploys/659416fdef66e30008f03b73

Copy link

netlify bot commented Dec 13, 2023

Deploy Preview for spirit-design-system-react canceled.

Name Link
🔨 Latest commit ecf4b82
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-react/deploys/659416fd2784a10008aad7f6

@literat literat marked this pull request as ready for review December 14, 2023 10:35
Copy link
Contributor

Coverage Status

coverage: 70.462% (-9.2%) from 79.629%
when pulling 10c00b6 on feat/visual-test-locally
into 4ddf584 on main.

@coveralls
Copy link

coveralls commented Dec 14, 2023

Coverage Status

coverage: 78.872% (+8.4%) from 70.462%
when pulling ecf4b82 on feat/visual-test-locally
into 8acf2e7 on main.

@literat literat added the e2e label Dec 14, 2023
Copy link
Contributor

@adamkudrna adamkudrna left a comment

Choose a reason for hiding this comment

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

Nice!

packages/common/package.json Show resolved Hide resolved
packages/common/package.json Show resolved Hide resolved
packages/web/vite.config.ts Show resolved Hide resolved
tests/e2e/demo-homepages.spec.ts Outdated Show resolved Hide resolved
@literat literat requested a review from adamkudrna December 15, 2023 13:08
.github/workflows/e2e.yml Show resolved Hide resolved
packages/common/constants/environments.ts Show resolved Hide resolved
tests/e2e/demo-homepages.spec.ts Outdated Show resolved Hide resolved
Copy link
Member

@crishpeen crishpeen left a comment

Choose a reason for hiding this comment

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

So I tested it and it doesn't work for me. :(

Error: page.goto: net::ERR_CONNECTION_REFUSED at http://localhost:5172/
    =========================== logs ===========================
    navigating to "http://localhost:5172/", waiting until "load"

Probably the problem with the network URL.

@literat literat force-pushed the feat/visual-test-locally branch from 37707e1 to f9e4b16 Compare January 2, 2024 13:29
packages/common/constants/servers.ts Show resolved Hide resolved
packages/common/LICENSE.md Outdated Show resolved Hide resolved
@literat literat force-pushed the feat/visual-test-locally branch from f9e4b16 to ecf4b82 Compare January 2, 2024 14:00
@literat literat merged commit bcaf99d into main Jan 2, 2024
20 checks passed
@literat literat deleted the feat/visual-test-locally branch January 2, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants