Clean up some test projects. (#5124)

* Make the ts-command-line-test project a unit test.

* Clean up api-documenter-scenarios.

* Extract scenarios test helpers.

* Remove extraneous copyrights.

* Enable eslint for api-documenter-test.

* Add verbose logging around finding the lint config file.

* Clean up the API Extractor tests.

* fixup! Make the ts-command-line-test project a unit test.

* Update repo README.

* fixup! Clean up api-documenter-scenarios.

* Clean up build for localization-plugin-test-* projects.

Author: iclanton

README.md

@@ -186,12 +186,12 @@ These GitHub repositories provide supplementary resources for Rush Stack:
 | [/build-tests/package-extractor-test-02](./build-tests/package-extractor-test-02/) | This project is used by tests in the @rushstack/package-extractor package. |
 | [/build-tests/package-extractor-test-03](./build-tests/package-extractor-test-03/) | This project is used by tests in the @rushstack/package-extractor package. |
 | [/build-tests/package-extractor-test-04](./build-tests/package-extractor-test-04/) | This project is used by tests in the @rushstack/package-extractor package. |
+| [/build-tests/run-scenarios-helpers](./build-tests/run-scenarios-helpers/) | Helpers for the *-scenarios test projects. |
 | [/build-tests/rush-amazon-s3-build-cache-plugin-integration-test](./build-tests/rush-amazon-s3-build-cache-plugin-integration-test/) | Tests connecting to an amazon S3 endpoint |
 | [/build-tests/rush-lib-declaration-paths-test](./build-tests/rush-lib-declaration-paths-test/) | This project ensures all of the paths in rush-lib/lib/... have imports that resolve correctly. If this project builds, all `lib/**/*.d.ts` files in the `@microsoft/rush-lib` package are valid. |
 | [/build-tests/rush-project-change-analyzer-test](./build-tests/rush-project-change-analyzer-test/) | This is an example project that uses rush-lib's ProjectChangeAnalyzer to  |
 | [/build-tests/rush-redis-cobuild-plugin-integration-test](./build-tests/rush-redis-cobuild-plugin-integration-test/) | Tests connecting to an redis server |
 | [/build-tests/set-webpack-public-path-plugin-test](./build-tests/set-webpack-public-path-plugin-test/) | Building this project tests the set-webpack-public-path-plugin |
-| [/build-tests/ts-command-line-test](./build-tests/ts-command-line-test/) | Building this project is a regression test for ts-command-line |
 | [/eslint/local-eslint-config](./eslint/local-eslint-config/) | An ESLint configuration consumed projects inside the rushstack repo. |
 | [/libraries/rush-themed-ui](./libraries/rush-themed-ui/) | Rush Component Library: a set of themed components for rush projects |
 | [/libraries/rushell](./libraries/rushell/) | Execute shell commands using a consistent syntax on every platform |

build-tests/api-documenter-scenarios/.eslintrc.js

@@ -0,0 +1,12 @@
+// This is a workaround for https://github.com/eslint/eslint/issues/3458
+require('local-node-rig/profiles/default/includes/eslint/patch/modern-module-resolution');
+// This is a workaround for https://github.com/microsoft/rushstack/issues/3021
+require('local-node-rig/profiles/default/includes/eslint/patch/custom-config-package-names');
+
+module.exports = {
+  extends: [
+    'local-node-rig/profiles/default/includes/eslint/profile/node-trusted-tool',
+    'local-node-rig/profiles/default/includes/eslint/mixins/friendly-locals'
+  ],
+  parserOptions: { tsconfigRootDir: __dirname }
+};

build-tests/api-documenter-scenarios/build.js

@@ -0,0 +1,38 @@
+{
+  "$schema": "https://developer.microsoft.com/json-schemas/heft/v0/heft.schema.json",
+
+  "extends": "local-node-rig/profiles/default/config/heft.json",
+
+  "phasesByName": {
+    "build": {
+      "tasksByName": {
+        "copy-dts": {
+          "taskPlugin": {
+            "pluginPackage": "@rushstack/heft",
+            "pluginName": "copy-files-plugin",
+            "options": {
+              "copyOperations": [
+                {
+                  "sourcePath": "src",
+                  "destinationFolders": ["lib"],
+                  "includeGlobs": ["**/*.d.ts"]
+                }
+              ]
+            }
+          }
+        },
+
+        "run-scenarios": {
+          "taskDependencies": ["typescript"],
+          "taskPlugin": {
+            "pluginPackage": "@rushstack/heft",
+            "pluginName": "run-script-plugin",
+            "options": {
+              "scriptPath": "./lib/runScenarios.js"
+            }
+          }
+        }
+      }
+    }
+  }
+}

build-tests/api-documenter-scenarios/config/build-config.json

@@ -0,0 +1,7 @@
+{
+  // The "rig.json" file directs tools to look for their config files in an external package.
+  // Documentation for this system: https://www.npmjs.com/package/@rushstack/rig-package
+  "$schema": "https://developer.microsoft.com/json-schemas/rig-package/rig.schema.json",
+
+  "rigPackageName": "local-node-rig"
+}

build-tests/api-documenter-scenarios/config/heft.json

@@ -5,17 +5,15 @@
   "private": true,
   "typings": "dist/internal/some-fake-file.d.ts",
   "scripts": {
-    "build": "node build.js",
-    "_phase:build": "node build.js"
+    "build": "heft build --clean",
+    "_phase:build": "heft build --clean"
   },
   "devDependencies": {
     "@microsoft/api-documenter": "workspace:*",
     "@microsoft/api-extractor": "workspace:*",
-    "@microsoft/teams-js": "1.3.0-beta.4",
+    "@rushstack/heft": "workspace:*",
     "@rushstack/node-core-library": "workspace:*",
-    "@types/jest": "29.2.5",
-    "@types/node": "18.17.15",
-    "fs-extra": "~11.3.0",
-    "typescript": "~5.4.2"
+    "local-node-rig": "workspace:*",
+    "run-scenarios-helpers": "workspace:*"
   }
 }

build-tests/api-documenter-scenarios/config/rig.json

@@ -1,46 +1,60 @@
+// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
+// See LICENSE in the project root for license information.
 import { Extractor } from '@microsoft/api-extractor';
 
 /** @public */
+// eslint-disable-next-line @typescript-eslint/no-namespace
 export namespace Namespace1 {
   /** @public */
   export class Class3 {
     /** Some prop. */
+    // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
     someProp: number;
 
     /** Some method. */
+    // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
     someMethod(x: boolean | string): void {}
 
     /** Some overload. */
+    // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
     someOverload(x: boolean): void;
 
     /** Some overload. */
+    // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
     someOverload(x: string): void;
 
+    // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
     someOverload(x: boolean | string): void {}
   }
 }
 
 /** @public */
 export class Class2<T> extends Namespace1.Class3 {
   /** A second prop. */
+  // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
   secondProp: boolean | string;
 
   /** A third prop. */
+  // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
   thirdProp: T;
 
   /** Some method. Overrides `Class3.someMethod`. */
+  // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
   someMethod(x: boolean): void {}
 }
 
 /** @public */
 export class Class1 extends Class2<number> {
   /** A second prop. Overrides `Class2.secondProp`. */
+  // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
   secondProp: boolean;
 
   /** A fourth prop */
+  // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
   fourthProp: number;
 
   /** Some overload. Overrides `Class3.someOverload`. */
+  // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
   someOverload(x: boolean | string): void {}
 }
 
@@ -72,6 +86,7 @@ export interface IExtendsMultipleInterfaces extends IInterface1, IInterface2 {
 }
 
 class UnexportedClass {
+  // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
   someProp: number;
 }
 
@@ -86,14 +101,17 @@ export class ExtendsUnexportedClass extends UnexportedClass {}
  * @public
  */
 export class ExtendsAnonymousClass extends class {
+  // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
   someProp: number;
 } {}
 
 /**
  * Some class-like variable.
  * @public
  */
+// eslint-disable-next-line @typescript-eslint/typedef
 export const ClassLikeVariable = class {
+  // eslint-disable-next-line @typescript-eslint/explicit-member-accessibility
   someProp: number;
 };
 
@@ -107,6 +125,7 @@ export class ExtendsClassLikeVariable extends ClassLikeVariable {}
  * Some interface-like type alias.
  * @public
  */
+// eslint-disable-next-line @typescript-eslint/consistent-type-definitions
 export type IInterfaceLikeTypeAlias = {
   someProp: number;
 };