eslint-plugin: classify stories as examples not tests (#20681)

change/@fluentui-eslint-plugin-5733e325-0c8b-4b50-ad12-d52a70e3d28b.json

@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Classify .stories.tsx as examples not tests",
+  "packageName": "@fluentui/eslint-plugin",
+  "email": "elcraig@microsoft.com",
+  "dependentChangeType": "none"
+}

packages/eslint-plugin/src/configs/react.js

@@ -374,12 +374,22 @@ const getOverrides = () => [
   },
   {
     // Example overrides
-    files: '**/*.Example.tsx',
+    files: '**/*.{Example,stories}.tsx',
     rules: {
       'no-alert': 'off',
       'no-console': 'off',
     },
   },
+  {
+    files: '**/*.stories.tsx',
+    rules: {
+      // allow arrow functions in stories for now (may want to change this later since using
+      // constantly-mutating functions can be an anti-pattern which we may not want to demonstrate
+      // in our converged components docs; it happened to be allowed starting out because .stories
+      // files were being linted as tests)
+      'react/jsx-no-bind': 'off',
+    },
+  },
   {
     // Docs overrides (excluding examples)
     files: [...configHelpers.docsFiles],

packages/eslint-plugin/src/utils/configHelpers.js

@@ -5,11 +5,10 @@ const path = require('path');
 const jju = require('jju');
 
 const testFiles = [
-  '**/*{.,-}test.{ts,tsx}',
-  '**/*.stories.tsx',
-  '**/{common,test,tests,stories}/**',
+  '**/*{.,-}{test,spec}.{ts,tsx}',
+  '**/{test,tests}/**',
   '**/testUtilities.{ts,tsx}',
-  '**/common/isConformant.{ts,tsx}',
+  '**/common/{isConformant,snapshotSerializers}.{ts,tsx}',
 ];
 
 const docsFiles = ['**/*Page.tsx', '**/{docs,demo}/**', '**/*.doc.{ts,tsx}'];
@@ -47,8 +46,16 @@ module.exports = {
   /** Files for build configuration */
   configFiles,
 
-  /** Files which may reference devDependencies: tests, docs (excluding examples), config/build */
-  devDependenciesFiles: [...testFiles, ...docsFiles, ...configFiles],
+  /**
+   * Files which may reference `devDependencies`:
+   * - tests
+   * - docs (excluding v8 examples)
+   * - config/build
+   * - stories, for now
+   *   - may need to reconsider for converged components depending on website approach
+   *   - the stories suffix is also used for screener stories in `vr-tests`
+   */
+  devDependenciesFiles: [...testFiles, ...docsFiles, ...configFiles, '**/*.stories.tsx'],
 
   /**
    * Whether linting is running in context of lint-staged (which should disable rules requiring