Skip to content

test: Create unit tests for react components in packages/ui/components/layout#10458

Merged
hariombalhara merged 2 commits intomainfrom
test-layout
Jul 31, 2023
Merged

test: Create unit tests for react components in packages/ui/components/layout#10458
hariombalhara merged 2 commits intomainfrom
test-layout

Conversation

@gitstart-calcom
Copy link
Copy Markdown
Contributor

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

Demo

Run

yarn test layout

image

@vercel
Copy link
Copy Markdown

vercel bot commented Jul 28, 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 31, 2023 6:22am
cal-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 31, 2023 6:22am
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 31, 2023 6:22am
3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Jul 31, 2023 6:22am
qa ⬜️ Ignored (Inspect) Jul 31, 2023 6:22am
ui ⬜️ Ignored (Inspect) Visit Preview Jul 31, 2023 6:22am

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 28, 2023

Thank you for following the naming conventions! 🙏

@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous bot commented Jul 28, 2023

🤖 Meticulous spotted visual differences in 47 of 225 screens tested: view and approve differences detected.

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 28, 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! 🙌

@gitstart-calcom gitstart-calcom marked this pull request as ready for review July 28, 2023 23:03
@gitstart-calcom gitstart-calcom requested review from a team, keithwillcode and sean-brydon July 28, 2023 23:04
@deploysentinel
Copy link
Copy Markdown

deploysentinel bot commented Jul 28, 2023

Current Playwright Test Results Summary

✅ 106 Passing - ⚠️ 3 Flaky

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

(Last updated on 07/31/2023 06:24:56am UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: a909f4e

Started: 07/31/2023 06:22:40am 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 1Initial Attempt
1.15% (3) 3 / 262 runs
failed over last 7 days
33.97% (89) 89 / 262 runs
flaked over last 7 days

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

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
FORM_SUBMITTED can submit a form and get a submission event
Retry 1Initial Attempt
1.88% (5) 5 / 266 runs
failed over last 7 days
18.80% (50) 50 / 266 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
0.40% (1) 1 / 247 run
failed over last 7 days
95.14% (235) 235 / 247 runs
flaked over last 7 days

View Detailed Build Results


@hariombalhara hariombalhara merged commit 65384d7 into main Jul 31, 2023
@hariombalhara hariombalhara deleted the test-layout branch July 31, 2023 06:37
describe("Tests for ShellSubHeading Component", () => {
test("Should render correctly the component with title and subtitle if the prop was passed", () => {
const { getByText, queryByText, rerender } = render(
<ShellSubHeading title={titleText} subtitle={subtitleText} />
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should test className as well.

sean-brydon pushed a commit that referenced this pull request Jul 31, 2023
…s/layout (#10458)

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

Labels

core area: core, team members only

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants