Remove the eslint rule that allows @ts-nocheck in test files

Now that we removed the last @ts-nocheck from the src/test directory, we
can remove that eslint rule that allows it. Ideally we shouldn't need to
add that ts comment anymore.

Author: canova

eslint.config.mjs

@@ -215,22 +215,6 @@ export default [
       // Allow require(), for example for importing JSON files.
       '@typescript-eslint/no-require-imports': 'off',
 
-      // Override the project-wide config to allow @ts-nocheck.
-      // We really just need this for our mocks.
-      '@typescript-eslint/ban-ts-comment': [
-        'error',
-        {
-          // Allow @ts-expect-error and @ts-no-check annotations with descriptions.
-          'ts-expect-error': 'allow-with-description',
-          'ts-nocheck': 'allow-with-description',
-          // Don't allow @ts-ignore because we want to be notified
-          // when the error goes away so we can remove the annotation - use
-          // @ts-expect-error instead
-          'ts-ignore': true,
-          'ts-check': false,
-        },
-      ],
-
       // Adding more errors now
       'testing-library/no-manual-cleanup': 'error',
       'testing-library/no-wait-for-snapshot': 'error',