Skip to content

test: Create unit tests for react components in packages/ui/components/form/TimezoneSelect#10765

Merged
keithwillcode merged 37 commits intomainfrom
test-timezoneSelect
Jan 27, 2024
Merged

test: Create unit tests for react components in packages/ui/components/form/TimezoneSelect#10765
keithwillcode merged 37 commits intomainfrom
test-timezoneSelect

Conversation

@gitstart-calcom
Copy link
Copy Markdown
Contributor

@gitstart-calcom gitstart-calcom commented Aug 15, 2023

command to run these tests: yarn test timezone

image

obs: the props that are passed to the component, have a specific and unique effect in the CSS, so the assertions are made with the hasClass

@vercel
Copy link
Copy Markdown

vercel bot commented Aug 15, 2023

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

Name Status Preview Comments Updated (UTC)
ai ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 26, 2024 9:52pm
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 26, 2024 9:52pm
dev ❌ Failed (Inspect) Jan 26, 2024 9:52pm
5 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jan 26, 2024 9:52pm
cal-demo ⬜️ Ignored (Inspect) Visit Preview Jan 26, 2024 9:52pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Jan 26, 2024 9:52pm
qa ⬜️ Ignored (Inspect) Visit Preview Jan 26, 2024 9:52pm
ui ⬜️ Ignored (Inspect) Visit Preview Jan 26, 2024 9:52pm

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 15, 2023

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 15, 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 Aug 15, 2023

Current Playwright Test Results Summary

✅ 90 Passing - ⚠️ 1 Flaky

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

(Last updated on 01/27/2024 10:56:55am UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 28e5640

Started: 01/27/2024 10:53:01am UTC

⚠️ Flakes

📄   apps/web/playwright/booking/availability.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Check availability tab in a event-type Check availability in event type
Retry 1Initial Attempt
20.32% (51) 51 / 251 runs
failed over last 7 days
41.43% (104) 104 / 251 runs
flaked over last 7 days

View Detailed Build Results


@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous bot commented Aug 15, 2023

🤖 Meticulous spotted visual differences in 51 of 209 screens tested: view and approve differences detected.

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

@gitstart-calcom gitstart-calcom marked this pull request as ready for review August 15, 2023 05:16
@PeerRich PeerRich added the Low priority Created by Linear-GitHub Sync label Aug 21, 2023
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Aug 28, 2023

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added ❗️ migrations contains migration files ❗️ .env changes contains changes to env variables labels Sep 1, 2023
@gitstart-calcom gitstart-calcom changed the base branch from main to 1182-refactor-app-list-my-settings-conferencing-and-installed-app-conferencing September 1, 2023 22:35
@gitstart-calcom gitstart-calcom changed the base branch from 1182-refactor-app-list-my-settings-conferencing-and-installed-app-conferencing to main September 1, 2023 22:35
keithwillcode
keithwillcode previously approved these changes Jan 20, 2024
@keithwillcode
Copy link
Copy Markdown
Contributor

@zomars we are getting heap issues here in the unit tests. Any recs based on recent changes you’ve made around this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Created by Linear-GitHub Sync 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.

9 participants