-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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(unify): fixing icons and supporting router within the shared cypress commands package #18237
fix(unify): fixing icons and supporting router within the shared cypress commands package #18237
Conversation
Thanks for taking the time to open a PR!
|
1c015cf
to
231a2b2
Compare
…ess commands package
231a2b2
to
3676263
Compare
Cypress.Commands.add( | ||
'mount', | ||
<C extends Parameters<typeof mount>[0]>(comp: C, options: CyMountOptions<C> = {}) => { | ||
export const registerMountFn = ({ plugins }) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CC @lmiller1990 🎉
…into the helper directory
…ss-io/cypress into fix/icon-colors-and-shared-plugins
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The only suggetsion I'd have is to rename the spec file Foo.spec.ts
into SmokeTest.spec.ts
or mountFreamnet.spec.ts
.
The rest seems to work.
* } | ||
* } | ||
*/ | ||
* { | ||
* `.icon-light-green-500`: { | ||
* '> *[stroke].icon-light': { | ||
* stroke: resolvedColor | ||
* }, | ||
* '> *[fill].icon-light': { | ||
* fill: resolvedColor | ||
* } | ||
* } | ||
* } | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hum... but OK
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah it was wrong the first time I wrote it.
* unified-desktop-gui: (48 commits) refactor: remove nexus-decorators, add data context (#18211) fix mountFragment tests fix(unified-desktop-gui branch): initial installation on windows (#18247) fix(unify): fixing icons and supporting router within the shared cypress commands package (#18237) feat(launchpad): improve layout and design (#18238) feat(launchpad): add design tokens from Figma (#18223) fix(unify): appveyor windows issue + better icon types (#18226) fixing broken ct tests fix types remove old code chore: bump deps (#18213) feat(app): icon library supporting Cy's custom icons (#18195) feat(app): adding navigation, pages, router, and layout (#18194) chore: Update Chrome (stable) to 94.0.4606.54 (#18196) fix(launchpad): fix bugs introduced by urql/graphcache (#18193) chore: bump yarn.lock (#18204) feat: add gulp makePackage, begin to scaffold data-context (#18186) feat(launchpad): Directory Upload in Global Mode (#18165) chore(tests): fix flake in net stubbing/xhr/proxy logging tests (#18163) chore: remove nxs.gen.ts - should be created by codegen (#18184) ...
Summary
Currently, the component tests for the App aren't running and the icons don't look right within the tests. This PR fixes the local component tests for the App.
How to test
git checkout fix/icon-colors-and-shared-plugins
cd packages/app
yarn cypress:open
and the component tests should render and look correct.Changes
frontend-shared/commands/command.ts
to export a factory function where each project can inject the plugins it needs.CYPRESS_INTERNAL_CLOUD_ENV
during the remote schema stitching PR.