From a16e0ac669da0cc99a8349a9088fcb38c2697f52 Mon Sep 17 00:00:00 2001 From: Thomas Roberts Date: Fri, 25 Feb 2022 13:32:19 +0000 Subject: [PATCH] Remove period from eslint rule disables --- .../block-editor/src/components/link-control/test/index.js | 4 ++-- packages/components/src/modal/index.js | 2 +- packages/components/src/utils/hooks/stories/use-cx.js | 2 +- packages/e2e-tests/specs/editor/blocks/navigation.test.js | 2 +- .../e2e-tests/specs/editor/various/reusable-blocks.test.js | 2 +- test/integration/full-content/full-content.test.js | 4 ++-- test/native/helpers.js | 2 +- 7 files changed, 9 insertions(+), 9 deletions(-) diff --git a/packages/block-editor/src/components/link-control/test/index.js b/packages/block-editor/src/components/link-control/test/index.js index 2c2f807eba6c5..6bbf2ea47b8ca 100644 --- a/packages/block-editor/src/components/link-control/test/index.js +++ b/packages/block-editor/src/components/link-control/test/index.js @@ -1633,7 +1633,7 @@ describe( 'Selecting links', () => { ); // We should have highlighted the first item using the keyboard - // eslint-disable-next-line jest/no-conditional-expect. + // eslint-disable-next-line jest/no-conditional-expect expect( selectedSearchResultElement ).toEqual( secondSearchSuggestion ); @@ -1648,7 +1648,7 @@ describe( 'Selecting links', () => { ); // We should be back to highlighting the first search result again - // eslint-disable-next-line jest/no-conditional-expect. + // eslint-disable-next-line jest/no-conditional-expect expect( selectedSearchResultElement ).toEqual( firstSearchSuggestion ); diff --git a/packages/components/src/modal/index.js b/packages/components/src/modal/index.js index f415f1a5f2cf2..f1af581ec5727 100644 --- a/packages/components/src/modal/index.js +++ b/packages/components/src/modal/index.js @@ -105,7 +105,7 @@ function Modal( props, forwardedRef ) { } return createPortal( - // eslint-disable-next-line jsx-a11y/no-static-element-interactions. + // eslint-disable-next-line jsx-a11y/no-static-element-interactions
{ const ExampleWithUseMemoWrong = ( { serializedStyles, children } ) => { const cx = useCx(); // Wrong: using 'useMemo' without adding 'cx' to the dependency list. - // eslint-disable-next-line react-hooks/exhaustive-deps. + // eslint-disable-next-line react-hooks/exhaustive-deps const classes = useMemo( () => cx( serializedStyles ), [ serializedStyles, ] ); diff --git a/packages/e2e-tests/specs/editor/blocks/navigation.test.js b/packages/e2e-tests/specs/editor/blocks/navigation.test.js index 2a7b3760094e9..d60574ae771cb 100644 --- a/packages/e2e-tests/specs/editor/blocks/navigation.test.js +++ b/packages/e2e-tests/specs/editor/blocks/navigation.test.js @@ -222,7 +222,7 @@ async function waitForBlock( blockName ) { } // Disable reason - these tests are to be re-written. -// eslint-disable-next-line jest/no-disabled-tests. +// eslint-disable-next-line jest/no-disabled-tests describe( 'Navigation', () => { const contributorUsername = uniqueId( 'contributoruser_' ); let contributorPassword; diff --git a/packages/e2e-tests/specs/editor/various/reusable-blocks.test.js b/packages/e2e-tests/specs/editor/various/reusable-blocks.test.js index 5ef1baf975806..16f8347b0fea5 100644 --- a/packages/e2e-tests/specs/editor/various/reusable-blocks.test.js +++ b/packages/e2e-tests/specs/editor/various/reusable-blocks.test.js @@ -311,7 +311,7 @@ describe( 'Reusable blocks', () => { await page.keyboard.type( ' modified' ); // Wait for async mode to dispatch the update. - // eslint-disable-next-line no-restricted-syntax. + // eslint-disable-next-line no-restricted-syntax await page.waitForTimeout( 1000 ); // Check that the content of the second reusable block has been updated. diff --git a/test/integration/full-content/full-content.test.js b/test/integration/full-content/full-content.test.js index b2e09a328d379..8ff389ccd99be 100644 --- a/test/integration/full-content/full-content.test.js +++ b/test/integration/full-content/full-content.test.js @@ -12,7 +12,7 @@ import { getBlockTypes, parse, serialize, - unstable__bootstrapServerSideBlockDefinitions, // eslint-disable-line camelcase. + unstable__bootstrapServerSideBlockDefinitions, // eslint-disable-line camelcase } from '@wordpress/blocks'; import { parse as grammarParse } from '@wordpress/block-serialization-default-parser'; import { @@ -82,7 +82,7 @@ describe( 'full post content fixture', () => { } blockBasenames.forEach( ( basename ) => { - // eslint-disable-next-line jest/valid-title. + // eslint-disable-next-line jest/valid-title it( basename, () => { const { filename: htmlFixtureFileName, diff --git a/test/native/helpers.js b/test/native/helpers.js index 7c7d8707ed4fc..3a2386d46bf37 100644 --- a/test/native/helpers.js +++ b/test/native/helpers.js @@ -14,7 +14,7 @@ import { // Editor component is not exposed in the pacakge because is meant to be consumed // internally, however we require it for rendering the editor in integration tests, // for this reason it's imported with path access. -// eslint-disable-next-line no-restricted-syntax. +// eslint-disable-next-line no-restricted-syntax import Editor from '@wordpress/edit-post/src/editor'; // Set up the mocks for getting the HTML output of the editor.