Skip to content

test(web-components/buttons): add ui tests#395

Closed
amir78729 wants to merge 27 commits intomainfrom
test/ui/buttons
Closed

test(web-components/buttons): add ui tests#395
amir78729 wants to merge 27 commits intomainfrom
test/ui/buttons

Conversation

@amir78729
Copy link
Member

No description provided.

@amir78729 amir78729 added test Testing our design system package: web-components labels Mar 19, 2025
@amir78729 amir78729 self-assigned this Mar 19, 2025
@Tap30 Tap30 deleted a comment from github-actions bot Mar 25, 2025
@Tap30 Tap30 deleted a comment from github-actions bot Mar 25, 2025
@Tap30 Tap30 deleted a comment from github-actions bot Mar 25, 2025
@amir78729 amir78729 marked this pull request as draft March 25, 2025 14:18
@github-actions
Copy link
Contributor

Caution

Playwright visual snapshot differences were detected.
You can download the report file.

To approve the snapshot changes and update the snapshots, please comment:

/approve-snapshots

@Tap30 Tap30 deleted a comment from github-actions bot Mar 25, 2025
@github-actions
Copy link
Contributor

Caution

Playwright visual snapshot differences were detected.
You can download the report file.

To approve the snapshot changes and update the snapshots, please comment:

/approve-snapshots

@amir78729
Copy link
Member Author

/approve-snapshots

@amir78729
Copy link
Member Author

/approve-snapshots

@amir78729
Copy link
Member Author

/approve-snapshots

@amir78729
Copy link
Member Author

/approve-snapshots

- name: 🗄️ archive artifacts
uses: actions/upload-artifact@v4
id: artifact-upload
if: always()
Copy link
Member Author

Choose a reason for hiding this comment

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

if: not cancelled

types:
- created
- edited
- deleted
Copy link
Member Author

Choose a reason for hiding this comment

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

delete is not neccessery

jobs:
update-snapshots:
name: Update Snapshots
if: ${{ github.event.issue.pull_request && contains(github.event.comment.body, '/approve-snapshots') }}
Copy link
Member Author

Choose a reason for hiding this comment

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

change to isEqual

Comment on lines +15 to +17
permissions:
contents: write
pull-requests: write
Copy link
Member Author

Choose a reason for hiding this comment

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

move to top

uses: google/wireit@setup-github-actions-caching/v2

- name: 📸 update snapshots
run: pnpm exec playwright test --update-snapshots
Copy link
Member Author

Choose a reason for hiding this comment

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

create a script in pakcage.json without caching in wireit

@Tap30 Tap30 deleted a comment from github-actions bot Mar 31, 2025
@Tap30 Tap30 deleted a comment from github-actions bot Mar 31, 2025
@amir78729
Copy link
Member Author

/approve-snapshots

@github-actions
Copy link
Contributor

⏳ Updating snapshots. Click here to see the status.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 5, 2025

Caution

Playwright visual snapshot differences were detected.
You can download the report file.

To approve the snapshot changes and update the snapshots, please comment:

/approve-snapshots

@amir78729
Copy link
Member Author

/approve-snapshots

@github-actions
Copy link
Contributor

github-actions bot commented Apr 5, 2025

/approve-snapshots

⏳ Updating snapshots. Click here to see the status.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 5, 2025

Caution

Playwright visual snapshot differences were detected.
You can download the report file.

To approve the snapshot changes and update the snapshots, please comment:

/approve-snapshots

@Tap30 Tap30 deleted a comment from github-actions bot Apr 7, 2025
@Tap30 Tap30 deleted a comment from github-actions bot Apr 7, 2025
@mimshins mimshins closed this May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: web-components test Testing our design system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants