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.

Sign up for GitHub

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

Jump to bottom

[e2e testing] Migrate test-e2e package to use Cypress #6139

Closed
1 task done
jeffchew opened this issue May 20, 2021 · 0 comments
Closed
1 task done

[e2e testing] Migrate test-e2e package to use Cypress #6139

jeffchew opened this issue May 20, 2021 · 0 comments
Assignees
@jeffchew
Labels
dev Needs some dev work priority: high
Milestone
Sprint 21-18

Comments

@jeffchew
Copy link
Member

jeffchew commented May 20, 2021 •
edited
Loading

User Story

As a [user role below]:
Carbon for IBM.com developer

I need to:
migrate the test-e2e package from Puppeteer to Cypress

so that I can:
automate e2e testing on multiple browsers, and deploy tests on Browserstack

Additional information

Acceptance criteria

  • Migrate test-e2e to cypress
@jeffchew jeffchew added dev Needs some dev work priority: medium labels May 20, 2021
@jeffchew jeffchew mentioned this issue May 20, 2021
Closed
2 tasks
@jeffchew jeffchew self-assigned this May 20, 2021
@jeffchew jeffchew added this to the Sprint 21-11 milestone May 20, 2021
jeffchew added a commit to jeffchew/carbon-for-ibm-dotcom that referenced this issue May 24, 2021
@jeffchew
test(integration): disabling react integration bundle/build tests
b692e04 
These are showing flakiness at the moment. We will ultimately replace
with the following stories:

carbon-design-system#6139
carbon-design-system#6142
@jeffchew jeffchew mentioned this issue May 24, 2021
Merged
kodiakhq bot pushed a commit that referenced this issue May 24, 2021
@jeffchew
test(integration): disabling react integration bundle/build tests (#6163
4ec6a3d 
)

### Related Ticket(s)

No related issues

### Description

These are showing flakiness at the moment. We will ultimately replace
with the following stories:

#6139
#6142

### Changelog

**Removed**

- React integration build/bundle tests

<!-- React and Web Component deploy previews are enabled by default. -->
<!-- To enable additional available deploy previews, apply the following -->
<!-- labels for the corresponding package: -->
<!-- *** "package: services": Services -->
<!-- *** "package: utilities": Utilities -->
<!-- *** "package: styles": Carbon Expressive -->
<!-- *** "RTL": React / Web Components (RTL) -->
<!-- *** "feature flag": React / Web Components (experimental) -->
@jeffchew jeffchew mentioned this issue Jun 3, 2021
Closed
2 tasks
@jeffchew jeffchew modified the milestones: Sprint 21-18, Sprint 21-20 Aug 16, 2021
@jeffchew jeffchew mentioned this issue Sep 9, 2021
Merged
@jeffchew jeffchew modified the milestones: Sprint 21-20, Sprint 21-18 Sep 9, 2021
kodiakhq bot pushed a commit that referenced this issue Sep 10, 2021
@jeffchew
test(e2e): introduce cypress for web components & react storybook (#7096
97258d3 
)

### Related Ticket(s)

Refs #6139

### Description

This PR fully moves the existing `test-e2e` package to a dedicated cypress test setup for web components and react storybooks. A few tests have been included for `masthead`, `locale-modal`, and `leadspace`. They are largely focused on snapshot testing, though percy has been disabled for now as rendering of dynamic states are still not supported in percy for web components (see percy/cli#280).

Actual cypress tests can be created now though, and also the current tests uses the built-in cypress screenshot feature:

![image](https://user-images.githubusercontent.com/1641214/132697052-2943ccdf-81c5-450d-999e-c39e557a3d05.png)

And video that shows how the tests are run:

https://user-images.githubusercontent.com/1641214/132697174-a778c055-d3d8-4e01-877d-bd58c2aa7491.mp4

### Changelog

**New**

- Cypress testing for web components and react storybook

**Changed**

- disabled video recording by default to speed up tests

**Removed**

- Removed `test-e2e` package
@jeffchew jeffchew mentioned this issue Sep 10, 2021
Merged
kodiakhq bot pushed a commit that referenced this issue Sep 10, 2021
@jeffchew
fix(e2e): removed test-e2e percy base update and added to react e2e (…
65a7d5c 
…#7117)

### Related Ticket(s)

Refs #6139

### Description

I missed removing this workflow in Github Actions, removed and added the new one to `percy-update-base.yml`.

### Changelog

**Changed**

- Added react e2e percy update
- Removed `test-e2e` workflow in deploy-canary
@jeffchew jeffchew modified the milestones: Sprint 21-19, Sprint 21-18 Sep 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jeffchew jeffchew

Labels
dev Needs some dev work priority: high
Projects
None yet
Milestone
Sprint 21-18
Development

No branches or pull requests
2 participants
@jeffchew @RobertaJHahn