Skip to content

Browser test: Migrate a CallComposite test to be hermetic, and fix it#2154

Merged
prprabhu-ms merged 21 commits intomainfrom
prprabhu/deflake-menu-injection
Aug 10, 2022
Merged

Browser test: Migrate a CallComposite test to be hermetic, and fix it#2154
prprabhu-ms merged 21 commits intomainfrom
prprabhu/deflake-menu-injection

Conversation

@prprabhu-ms
Copy link
Copy Markdown
Contributor

@prprabhu-ms prprabhu-ms commented Aug 4, 2022

This PR primarily migrates a live CallComposite test to be hermetic, but also fixes a few bugs found in the processes.

  • The test was actually not testing for the right thing.
    • The snapshots did not have the injected items
    • The test for beta/mobile waited for the wrong selector (the one that would be there if no menu items were injected)
  • The test was passing because... CallComposite behavior was also broken. It did not pass through the Contoso callback for injecting menu items.

Includes some small quality of life fixes to the browser test setup.

Why

This is one of the more flaky tests around:
image

Comment thread packages/react-composites/tests/browser/common/utils.ts
Comment thread packages/react-composites/playwright.config.common.ts Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 4, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 4, 2022

Chat bundle size is increased❗.

  • Current size: 5185328
  • Base size: 5185327
  • Diff size: 1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 4, 2022

Calling bundle size is increased❗.

  • Current size: 5052870
  • Base size: 5052739
  • Diff size: 131

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 4, 2022

CallWithChat bundle size is increased❗.

  • Current size: 5431845
  • Base size: 5431714
  • Diff size: 131

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 4, 2022

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "ui change" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 4, 2022

Failed to pass the composite UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "ui change" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 4, 2022

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "ui change" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 5, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 5, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 5, 2022

@prprabhu-ms prprabhu-ms enabled auto-merge (squash) August 5, 2022 18:50
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 5, 2022

Comment thread packages/react-composites/tests/browser/call/app/BaseApp.tsx
@prprabhu-ms prprabhu-ms changed the title Browser test: Migrate a CallComposite test to be hermetic, and fix it. Browser test: Migrate a CallComposite test to be hermetic, and fix it Aug 8, 2022
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 8, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 8, 2022

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "ui change" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "ui change" label to the PR for updating the snapshot.

@prprabhu-ms prprabhu-ms enabled auto-merge (squash) August 10, 2022 11:23
@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@prprabhu-ms prprabhu-ms merged commit 2d505f7 into main Aug 10, 2022
@prprabhu-ms prprabhu-ms deleted the prprabhu/deflake-menu-injection branch August 10, 2022 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants