We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
We’re moving to React 18. WP 6.2 onwards is using that already. So we are not developing and testing with the correct React version.
Moving to React 18 is not compatible with our tests. We’re going to remove Enzyme in favour of Testing Library: https://testing-library.com/
We’re going to create a feature branch. Where we will:
Notes:
@yoast/components
yoast-components
Check this list for each package:
The content you are editing has changed. Please copy your edits and refresh the page.
components/tests/YoastModalTest.js
js/tests/indexation.test.js
js/tests/components/AnalysisCheckList.test.js
js/tests/components/PrePublish.test.js
js/tests/components/SEMrushCountrySelector.test.js
js/tests/components/SEMrushKeyphrasesTable.test.js
js/tests/components/SEMrushRelatedKeyphrasesModal.test.js
js/tests/components/WincherKeyphrasesTable.test.js
js/tests/components/WincherTableRow.test.js
js/tests/settings/search.test.js
replacement-variable-editor/tests/ReplacementVariableEditorTest.js
search-metadata-previews/tests/SnippetEditorTest.js
search-metadata-previews/tests/SnippetPreviewTest.js
packages/components
components/tests/ScreenReaderShortcutTest.js
The text was updated successfully, but these errors were encountered:
Got blocked with JS package and react testing library. Also need to address components package, it might be related.
Sorry, something went wrong.
igorschoester
vraja-pro
Successfully merging a pull request may close this issue.
We’re moving to React 18. WP 6.2 onwards is using that already.
So we are not developing and testing with the correct React version.
Moving to React 18 is not compatible with our tests. We’re going to remove Enzyme in favour of Testing Library: https://testing-library.com/
We’re going to create a feature branch. Where we will:
Notes:
@yoast/components
package because we do not expect to add components to this package anymore.yoast-components
package because that is in the process of being removed.Packages tasks
Check this list for each package:
Packages, see tasks above
Tests to migrate
js/tests/indexation.test.js
Indexation (progress) #20668js/tests/components/AnalysisCheckList.test.js
Used in the editor' pre-publish and document sidebars #20669js/tests/components/PrePublish.test.js
Used in the editor #20670js/tests/components/SEMrushCountrySelector.test.js
,js/tests/components/SEMrushKeyphrasesTable.test.js
andjs/tests/components/SEMrushRelatedKeyphrasesModal.test.js
#20671js/tests/components/WincherKeyphrasesTable.test.js
andjs/tests/components/WincherTableRow.test.js
#20672replacement-variable-editor/tests/ReplacementVariableEditorTest.js
#20675search-metadata-previews/tests/SnippetEditorTest.js
andsearch-metadata-previews/tests/SnippetPreviewTest.js
#20676packages/components
#20677Tests to remove
Final checklist
The text was updated successfully, but these errors were encountered: