Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(react-drawer): add e2e for base drawer components #29029

Conversation

marcosmoura
Copy link
Contributor

@marcosmoura marcosmoura commented Aug 30, 2023

Add missing Cypress tests for Drawer, DrawerInline and DrawerOverlay components.

@marcosmoura marcosmoura added the Component: Drawer The Fluent v9 Drawer component label Aug 30, 2023
@marcosmoura marcosmoura self-assigned this Aug 30, 2023
@marcosmoura marcosmoura requested review from a team as code owners August 30, 2023 17:44
@marcosmoura marcosmoura mentioned this pull request Aug 30, 2023
46 tasks
@fabricteam
Copy link
Collaborator

fabricteam commented Aug 30, 2023

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
69.517 kB
19.64 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
208.067 kB
59.345 kB
react-components
react-components: FluentProvider & webLightTheme
40.713 kB
13.509 kB
react-portal-compat
PortalCompatProvider
6.541 kB
2.227 kB
🤖 This report was generated against 6363e77de4ed560ea8dbd53f5eb0511b65f19b77

@fabricteam
Copy link
Collaborator

fabricteam commented Aug 30, 2023

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
InfoButton mount 15 13 5000 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 580 614 5000
Button mount 309 296 5000
Field mount 1086 1087 5000
FluentProvider mount 678 684 5000
FluentProviderWithTheme mount 79 79 10
FluentProviderWithTheme virtual-rerender 67 73 10
FluentProviderWithTheme virtual-rerender-with-unmount 78 78 10
InfoButton mount 15 13 5000 Possible regression
MakeStyles mount 848 843 50000
Persona mount 1682 1692 5000
SpinButton mount 1393 1382 5000

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 30, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit b79b672:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@size-auditor
Copy link

size-auditor bot commented Aug 30, 2023

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 6363e77de4ed560ea8dbd53f5eb0511b65f19b77 (build)

@marcosmoura marcosmoura enabled auto-merge (squash) August 30, 2023 18:21
@tudorpopams tudorpopams self-requested a review August 31, 2023 12:15
* master:
  refactor(react-drawer): change DrawerHeaderTitle slot creation while keeping the same API (microsoft#29042)
  test(react-drawer): add render tests for drawer subcomponents (microsoft#29043)
  Grouped vertical bar chart - Component tests (microsoft#29031)
  docs: add Fluent UI Insights EP06 to README (microsoft#29051)
  chore: use swc-plugin-de-indent-template-literal for consoles (microsoft#29040)
  chore: adds swc-plugin-de-indent-template-literal (microsoft#29037)
  feat(react-jsx-runtime): v9 packages to use importSource (microsoft#28959)
@marcosmoura marcosmoura merged commit c6a8055 into microsoft:master Sep 5, 2023
21 checks passed
@marcosmoura marcosmoura deleted the test/add-e2e-tests-for-drawer-base-components branch September 5, 2023 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Drawer The Fluent v9 Drawer component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants