diff --git a/tests/create-testing-library-rule.test.ts b/tests/create-testing-library-rule.test.ts index 43b74f70..55a9f22c 100644 --- a/tests/create-testing-library-rule.test.ts +++ b/tests/create-testing-library-rule.test.ts @@ -196,7 +196,6 @@ ruleTester.run(RULE_NAME, rule, { // cases: weird scenarios to check guard against parent nodes 'expect(element).not()', - 'expect(element).not()', // Test Cases for Queries and Aggressive Queries Reporting { diff --git a/tests/lib/rules/no-await-sync-events.test.ts b/tests/lib/rules/no-await-sync-events.test.ts index 93ff235c..5116080e 100644 --- a/tests/lib/rules/no-await-sync-events.test.ts +++ b/tests/lib/rules/no-await-sync-events.test.ts @@ -289,51 +289,6 @@ ruleTester.run(RULE_NAME, rule, { } as const) ), - // sync fireEvent methods with await operator are not valid - // when only fire-event set in eventModules - ...SUPPORTED_TESTING_FRAMEWORKS.flatMap((testingFramework) => - FIRE_EVENT_FUNCTIONS.map( - (func) => - ({ - code: ` - import { fireEvent } from '${testingFramework}'; - test('should report fireEvent.${func} sync event awaited', async() => { - await fireEvent.${func}('foo'); - }); - `, - errors: [ - { - line: 4, - column: 17, - messageId: 'noAwaitSyncEvents', - data: { name: `fireEvent.${func}` }, - }, - ], - } as const) - ) - ), - - ...USER_EVENT_SYNC_FUNCTIONS.map( - (func) => - ({ - code: ` - import userEvent from '@testing-library/user-event'; - test('should report userEvent.${func} sync event awaited', async() => { - await userEvent.${func}('foo'); - }); - `, - options: [{ eventModules: ['user-event'] }], - errors: [ - { - line: 4, - column: 17, - messageId: 'noAwaitSyncEvents', - data: { name: `userEvent.${func}` }, - }, - ], - } as const) - ), - { code: ` import userEvent from '@testing-library/user-event'; diff --git a/tests/lib/rules/no-debugging-utils.test.ts b/tests/lib/rules/no-debugging-utils.test.ts index b31861a7..c41c6423 100644 --- a/tests/lib/rules/no-debugging-utils.test.ts +++ b/tests/lib/rules/no-debugging-utils.test.ts @@ -434,19 +434,6 @@ ruleTester.run(RULE_NAME, rule, { code: ` import { screen } from '@testing-library/dom' screen.debug() - `, - errors: [ - { - line: 3, - column: 16, - messageId: 'noDebug', - }, - ], - }, - { - code: ` - import { screen } from '@testing-library/dom' - screen.logTestingPlaygroundURL() `, errors: [ { @@ -486,19 +473,6 @@ ruleTester.run(RULE_NAME, rule, { code: ` import { screen } from '@testing-library/dom' screen.logTestingPlaygroundURL() - `, - errors: [ - { - line: 3, - column: 16, - messageId: 'noDebug', - }, - ], - }, - { - code: ` - import { screen } from '@testing-library/dom' - screen.logTestingPlaygroundURL() `, options: [{ utilsToCheckFor: {} }], errors: [ @@ -509,19 +483,6 @@ ruleTester.run(RULE_NAME, rule, { }, ], }, - { - code: ` - import { screen } from '@testing-library/dom' - screen.logTestingPlaygroundURL() - `, - errors: [ - { - line: 3, - column: 16, - messageId: 'noDebug', - }, - ], - }, { settings: { 'testing-library/utils-module': 'test-utils' }, code: `// aggressive reporting disabled diff --git a/tests/lib/rules/no-wait-for-side-effects.test.ts b/tests/lib/rules/no-wait-for-side-effects.test.ts index 869dfb2b..37f9a0ac 100644 --- a/tests/lib/rules/no-wait-for-side-effects.test.ts +++ b/tests/lib/rules/no-wait-for-side-effects.test.ts @@ -299,16 +299,6 @@ ruleTester.run(RULE_NAME, rule, { settings: { 'testing-library/custom-renders': ['renderHelper'] }, code: ` import { waitFor } from '${testingFramework}'; - import { renderWrapper } from 'somewhere-else'; - await waitFor(() => { - renderWrapper() - }) - `, - }, - { - settings: { 'testing-library/custom-renders': ['renderHelper'] }, - code: ` - import { waitFor } from '${testingFramework}'; await waitFor(() => result = renderWrapper()) `, }, @@ -432,14 +422,6 @@ ruleTester.run(RULE_NAME, rule, { { code: ` import { waitFor } from '${testingFramework}'; - const { rerender } = render() - await waitFor(() => rerender()) - `, - errors: [{ line: 4, column: 29, messageId: 'noSideEffectsWaitFor' }], - } as const, - { - code: ` - import { waitFor } from '${testingFramework}'; await waitFor(() => renderHelper()) `, errors: [{ line: 3, column: 29, messageId: 'noSideEffectsWaitFor' }], diff --git a/tests/lib/rules/prefer-explicit-assert.test.ts b/tests/lib/rules/prefer-explicit-assert.test.ts index 952f770d..0b65a72f 100644 --- a/tests/lib/rules/prefer-explicit-assert.test.ts +++ b/tests/lib/rules/prefer-explicit-assert.test.ts @@ -330,14 +330,6 @@ ruleTester.run(RULE_NAME, rule, { ], } as const) ), - { - code: `getByIcon('foo')`, // custom `getBy` query extended through options - errors: [ - { - messageId: 'preferExplicitAssert', - }, - ], - }, ...COMBINED_QUERIES_METHODS.map( (queryMethod) => ({ diff --git a/tests/lib/rules/prefer-query-by-disappearance.test.ts b/tests/lib/rules/prefer-query-by-disappearance.test.ts index f6623652..32eb2f3b 100644 --- a/tests/lib/rules/prefer-query-by-disappearance.test.ts +++ b/tests/lib/rules/prefer-query-by-disappearance.test.ts @@ -219,20 +219,6 @@ ruleTester.run(RULE_NAME, rule, { code: ` import { screen, waitForElementToBeRemoved } from '${testingFramework}'; - await waitForElementToBeRemoved(() => screen.getByText("hello")) - `, - errors: [ - { - messageId: 'preferQueryByDisappearance', - line: 4, - column: 54, - }, - ], - }, - { - code: ` - import { screen, waitForElementToBeRemoved } from '@marko/testing-library'; - await waitForElementToBeRemoved(() => screen.getByText("hello")) `, errors: [ diff --git a/tests/lib/rules/prefer-screen-queries.test.ts b/tests/lib/rules/prefer-screen-queries.test.ts index 7c3b4a93..4951233b 100644 --- a/tests/lib/rules/prefer-screen-queries.test.ts +++ b/tests/lib/rules/prefer-screen-queries.test.ts @@ -278,26 +278,6 @@ ruleTester.run(RULE_NAME, rule, { } as const) ) ), - ...ALL_QUERIES_COMBINATIONS.map( - (queryMethod) => - ({ - settings: { 'testing-library/utils-module': 'test-utils' }, - code: ` - import { render } from 'test-utils' - const { ${queryMethod} } = render(foo) - ${queryMethod}()`, - errors: [ - { - line: 4, - column: 9, - messageId: 'preferScreenQueries', - data: { - name: queryMethod, - }, - }, - ], - } as const) - ), ...ALL_QUERIES_COMBINATIONS.map( (queryMethod) => ({