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

Fix flaky test 'As editor I can add links' by using getSlateEditorAndType #5966

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

ksuess
Copy link
Member

@ksuess ksuess commented Apr 18, 2024

This should fix the failing test in https://github.com/plone/volto/actions/runs/8726201362/job/23941446900?pr=5295
Locally I could reproduce the failure with Electron and fix it like this by using cy.getSlateEditorAndType.

Copy link

netlify bot commented Apr 18, 2024

Deploy Preview for volto canceled.

Name Link
🔨 Latest commit c3fd1a3
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/6620f5c3336850000809fdb9

Copy link

netlify bot commented Apr 18, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit c3fd1a3
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/6620f5c31876c6000839fe45

Copy link
Contributor

@pnicolli pnicolli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome thanks! I did not know this existed.

@pnicolli pnicolli merged commit 4b056a8 into main Apr 18, 2024
73 checks passed
@pnicolli pnicolli deleted the fix-test-'As-editor-I-can-add-links' branch April 18, 2024 10:45
sneridagh added a commit that referenced this pull request Apr 29, 2024
* main: (49 commits)
  Update to Plone 6.0.11 (#5989)
  Defines the last 4 parameters of the `asyncConnect` function with optional (#5986)
  Flexibilize the pins for all ESlint deps, in Volto and generators (#5991)
  Release 18.0.0-alpha.29
  Release @plone/types 1.0.0-alpha.11
  fix: pass down locale to IntlProvider (#5976)
  Add Vite (client only, no SSR) build. Update Next.js 14.2.2 and Remix to 2.8.0 (#5970)
  Fix no router link in logo (#5981)
  Improve postinstall script, building all the deps (#5980)
  Better BlocksData types (#5979)
  Add missing types ts fields
  Release 18.0.0-alpha.28
  Release @plone/slate 18.0.0-alpha.12
  Release @plone/registry 1.5.6
  Improvements to the monorepo setup with utilities, especially ESLint.… (#5969)
  DEV: put nvm installation section into a separate include file (#5968)
  Bundle optimization (#5295)
  [client] Move provider to own package - Use parcel - `@plone/providers` (#5887)
  Fix flaky test 'As editor I can add links' by using getSlateEditorAndType (#5966)
  Fix rendering if ConditionalLink has no children (#5963)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants