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.

Sign up for GitHub

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

Jump to bottom

Add Tests for Flyout and Popup Launch #11102

Open
Tracked by #11422
chiaramooney opened this issue Jan 18, 2023 · 2 comments
Open
Tracked by #11422

Add Tests for Flyout and Popup Launch #11102

chiaramooney opened this issue Jan 18, 2023 · 2 comments
Labels
Area: Flyout Area: Paper Area: Popup Area: Tests enhancement Old Architecture Broad category for issues that apply to the RN "old" architecture of Cxx Modules + Paper Workstream: Test Coverage Expand and update our new architecture test coverage.
Milestone
Backlog

Comments

@chiaramooney
Copy link
Contributor

Summary

We recently had a regression in the behavior of the Flyout component, where the component fired a RedBox error upon attempt to launch the Flyout. This was found during the manual validation of gallery for the v71 release.

Currently, we do not have automated testing to check that Popups and Flyout can successfully launch. These should be added to our test set.

Motivation

Prevent behavior regressions.

Basic Example

No response

Open Questions

No response
@ghost ghost added the Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) label Jan 18, 2023
@chrisglein chrisglein added Area: Tests Area: Flyout Area: Popup and removed Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) labels Jan 19, 2023
@chrisglein chrisglein added this to the 0.72 milestone Jan 19, 2023
@chiaramooney chiaramooney modified the milestones: 0.72, 0.73 May 5, 2023
@chrisglein chrisglein mentioned this issue Jul 21, 2023
Open
15 tasks
@TatianaKapos
Copy link
Contributor

manually validated for 73! would be nice to have these for 74 but not necessary :)

@TatianaKapos TatianaKapos modified the milestones: 0.73, 0.74 Dec 8, 2023
@chrisglein chrisglein modified the milestones: 0.74, Next Apr 9, 2024
@chiaramooney chiaramooney mentioned this issue May 17, 2024
Open
@TatianaKapos
Copy link
Contributor

Costing as 4 days since I'm not sure if our current test structure will support opening and closing another windows. This will require some testing/research on the capabilities of WebDriverIO

@microsoft-github-policy-service microsoft-github-policy-service bot added the Old Architecture Broad category for issues that apply to the RN "old" architecture of Cxx Modules + Paper label May 22, 2024
This was referenced Jul 19, 2024
Merged
Closed
@chrisglein chrisglein modified the milestones: Next, Backlog Sep 12, 2024
@chiaramooney chiaramooney added the Workstream: Test Coverage Expand and update our new architecture test coverage. label Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area: Flyout Area: Paper Area: Popup Area: Tests enhancement Old Architecture Broad category for issues that apply to the RN "old" architecture of Cxx Modules + Paper Workstream: Test Coverage Expand and update our new architecture test coverage.
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
4 participants
@chrisglein @YajurG @chiaramooney @TatianaKapos