test: remove duplicate cases (#927)

Author: G-Rath

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
 		{

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';

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

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(<App />)
-          })
-        `,
-			},
-			{
-				settings: { 'testing-library/custom-renders': ['renderHelper'] },
-				code: `
-          import { waitFor } from '${testingFramework}';
           await waitFor(() => result = renderWrapper(<App />))
         `,
 			},
@@ -432,14 +422,6 @@ ruleTester.run(RULE_NAME, rule, {
 			{
 				code: `
         import { waitFor } from '${testingFramework}';
-        const { rerender } = render(<App />)
-        await waitFor(() => rerender(<App />))
-      `,
-				errors: [{ line: 4, column: 29, messageId: 'noSideEffectsWaitFor' }],
-			} as const,
-			{
-				code: `
-        import { waitFor } from '${testingFramework}';
         await waitFor(() => renderHelper(<App />))
       `,
 				errors: [{ line: 3, column: 29, messageId: 'noSideEffectsWaitFor' }],

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) =>
 				({

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: [

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) =>
 				({