-
Notifications
You must be signed in to change notification settings - Fork 985
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
fix double @@ in component-test-watch
and don't do yarn install
for component tests
#18521
Conversation
Jenkins BuildsClick to see older builds (8)
|
075af09
to
11b6dcb
Compare
I remember there will be a problem if skip |
Thanks @yqrashawn was this issue CI specific or reproducible locally also? |
#15071 (comment) |
Thanks @yqrashawn! Yeah i think since component tests are the last stage in CI It seems safe to get rid of To be super sure I also run |
11b6dcb
to
315f062
Compare
77% of end-end tests have passed
Failed tests (9)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (2)Click to expandClass TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (37)Click to expandClass TestActivityMultipleDevicePRTwo:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePR:
|
315f062
to
a92d4f0
Compare
Summary
This PR contains tiny fixes for the
Makefile
status: ready