-
Notifications
You must be signed in to change notification settings - Fork 171
Switch fluentui-react-native to pnpm mode #3984
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
Conversation
…ode due to a failure to resolve virtual packages
|
non blocking comment, how do we feel about having the webdriverio tests in FURN at all? I think the main usefulness from those tests may have now moved to our internal tests, and in FURN it feels like we've spent more time keeping them alive than not. Some more context: one day I'd love to move Fluent Tester to use React Native Storybook, which would then use it's own testing suite and/or not use Appium. Have lots of ideas like this I'll probably never do though :) |
Mind expanding a bit on this? "I think the main usefulness from those tests may have now moved to our internal tests"? |
…imitations of 4.x and fix e2e
AFAIK, we're running Appium / webdriverio tests internally, and getting lots of value testing our features. However, the same E2E test suite in FURN has been disabled for android and iOS (not sure of Mac?) due to flakiness. React Native Core also switched away from Appium to Maestro due to flakiness in the open source repo. Given that most of the team is also working product side and not in this repo, I feel like the test suite has been more difficult to maintain which is part of why half the platforms ended up getting disabled instead of fixed - it's the fast unblock that we haven't prioritized to go back and fix. Well, at least not till Jason just did :). And as always, I might be biased, feel free to disagree with any of this |
I think that's fair - I'm not too familiar with how much this repo gets iterated on. I'm also surprised to hear that the E2E testing has been difficult to maintain. Is there any specific issues that have come up? Is it when we try to update Webdriverio or appium dependencies? |
At least from my perspective, it's usually something like "Wait why is CI failing? Nothing changed. Wait why are the Android E2E test failing? Is anyone using this on Android? Do I fix this or disable it to unblock <RN upgrade / bugfix / publish pipeline / etc>? And then never having time / priority to go back and fix it." |
Platforms Impacted
Description of changes
This moves the repo to use pnpm as the nodeLinker setting for yarn. Most of the work was done at the end of the summer but appium failed to run correctly. The parts of the change are:
Verification
Will be monitoring the pipelines to ensure they run correctly.