Skip to content

test: Create unit tests for react components in packages/ui/components/TokenHandle#10407

Merged
keithwillcode merged 2 commits intomainfrom
CALCOM-640
Jul 27, 2023
Merged

test: Create unit tests for react components in packages/ui/components/TokenHandle#10407
keithwillcode merged 2 commits intomainfrom
CALCOM-640

Conversation

@gitstart-calcom
Copy link
Copy Markdown
Contributor

@gitstart-calcom gitstart-calcom commented Jul 26, 2023

DEMO

Run

yarn test tokenHandle

image

@vercel
Copy link
Copy Markdown

vercel bot commented Jul 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 27, 2023 11:03am
cal-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 27, 2023 11:03am
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 27, 2023 11:03am
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 27, 2023 11:03am
2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Jul 27, 2023 11:03am
qa ⬜️ Ignored (Inspect) Jul 27, 2023 11:03am

@zomars zomars added the core area: core, team members only label Jul 26, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 26, 2023

Thank you for following the naming conventions! 🙏

@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous bot commented Jul 26, 2023

🤖 Meticulous spotted visual differences in 55 of 222 screens tested: view and approve differences detected.

Last updated for commit a070509. This comment will update as new commits are pushed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 26, 2023

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel bot commented Jul 26, 2023

Current Playwright Test Results Summary

✅ 105 Passing - ⚠️ 4 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 07/27/2023 11:06:04am UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: a070509

Started: 07/27/2023 11:03:06am UTC

⚠️ Flakes

📄   apps/web/playwright/login.2fa.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
2FA Tests should allow a user to enable 2FA and login using 2FA
Retry 2Retry 1Initial Attempt
1.46% (4) 4 / 274 runs
failed over last 7 days
38.69% (106) 106 / 274 runs
flaked over last 7 days

📄   packages/app-store/typeform/playwright/tests/basic.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Routing Forms Seeded Routing Form Test preview should return correct route
Retry 1Initial Attempt
0% (0) 0 / 96 runs
failed over last 7 days
16.67% (16) 16 / 96 runs
flaked over last 7 days

📄   apps/web/playwright/reschedule.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Reschedule Tests Should do a reschedule from user owner
Retry 1Initial Attempt
0% (0) 0 / 282 runs
failed over last 7 days
1.06% (3) 3 / 282 runs
flaked over last 7 days

📄   packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should be able to reschedule
Retry 1Initial Attempt
1.44% (3) 3 / 209 runs
failed over last 7 days
95.69% (200) 200 / 209 runs
flaked over last 7 days

View Detailed Build Results


global.ResizeObserver = vi.fn().mockImplementation(() => ({
observe: vi.fn(),
unobserve: vi.fn(),
disconnect: vi.fn(),
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We added it to solve the following error:
image


vi.mock("@calcom/lib/OgImages", async () => {
return {};
});
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We added it to solve the following error:
image

@gitstart-calcom gitstart-calcom marked this pull request as ready for review July 26, 2023 17:48
@gitstart-calcom gitstart-calcom requested review from a team, keithwillcode and sean-brydon July 26, 2023 17:48
Copy link
Copy Markdown
Member

@sean-brydon sean-brydon left a comment

Choose a reason for hiding this comment

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

Nice work

@keithwillcode keithwillcode merged commit e81985c into main Jul 27, 2023
@keithwillcode keithwillcode deleted the CALCOM-640 branch July 27, 2023 13:48
@keithwillcode keithwillcode added Low priority Created by Linear-GitHub Sync automated-tests area: unit tests, e2e tests, playwright labels Jul 27, 2023
@keithwillcode keithwillcode added this to the v3.2 milestone Jul 27, 2023
sean-brydon pushed a commit that referenced this pull request Jul 31, 2023
…s/TokenHandle (#10407)

Co-authored-by: gitstart-calcom <gitstart@users.noreply.github.com>
Co-authored-by: Shivam Kalra <shivamkalra98@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-tests area: unit tests, e2e tests, playwright core area: core, team members only Low priority Created by Linear-GitHub Sync

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

6 participants