feat(nx-plugin): slim down default generated nx-plugin

docs/generated/manifests/packages.json

@@ -1887,7 +1887,7 @@
       "/packages/nx-plugin/generators/e2e-project": {
         "description": "Create a E2E application for a Nx Plugin.",
         "file": "generated/packages/nx-plugin/generators/e2e-project.json",
-        "hidden": true,
+        "hidden": false,
         "name": "e2e-project",
         "originalFilePath": "/packages/nx-plugin/src/generators/e2e-project/schema.json",
         "path": "/packages/nx-plugin/generators/e2e-project",

docs/generated/packages-metadata.json

@@ -1865,7 +1865,7 @@
       {
         "description": "Create a E2E application for a Nx Plugin.",
         "file": "generated/packages/nx-plugin/generators/e2e-project.json",
-        "hidden": true,
+        "hidden": false,
         "name": "e2e-project",
         "originalFilePath": "/packages/nx-plugin/src/generators/e2e-project/schema.json",
         "path": "nx-plugin/generators/e2e-project",

docs/generated/packages/nx-plugin/generators/e2e-project.json

@@ -57,9 +57,9 @@
     "presets": []
   },
   "description": "Create a E2E application for a Nx Plugin.",
-  "hidden": true,
   "implementation": "/packages/nx-plugin/src/generators/e2e-project/e2e.ts",
   "aliases": [],
+  "hidden": false,
   "path": "/packages/nx-plugin/src/generators/e2e-project/schema.json",
   "type": "generator"
 }

docs/generated/packages/nx-plugin/generators/plugin.json

@@ -69,7 +69,7 @@
         "type": "string",
         "enum": ["jest", "none"],
         "description": "Test runner to use for end to end (E2E) tests.",
-        "default": "jest"
+        "default": "none"
       },
       "standaloneConfig": {
         "description": "Split the project configuration into `<projectRoot>/project.json` rather than including it inside `workspace.json`.",
@@ -87,15 +87,9 @@
         "enum": ["tsc", "swc"],
         "default": "tsc",
         "description": "The compiler used by the build and test targets."
-      },
-      "minimal": {
-        "type": "boolean",
-        "description": "Generate the plugin with a minimal setup. This would involve not generating a default executor and generator.",
-        "default": false
       }
     },
     "required": ["name"],
-    "additionalProperties": false,
     "presets": []
   },
   "description": "Create a Nx Plugin.",

e2e/nx-plugin/src/nx-plugin.test.ts

@@ -2,7 +2,6 @@ import { ProjectConfiguration } from '@nrwl/devkit';
 import {
   checkFilesExist,
   expectTestsPass,
-  isNotWindows,
   killPorts,
   newProject,
   readJson,
@@ -12,8 +11,6 @@ import {
   uniq,
   updateFile,
   createFile,
-  readFile,
-  removeFile,
   cleanupProject,
   runCommand,
   getPackageManagerCommand,
@@ -39,33 +36,16 @@ describe('Nx Plugin', () => {
     const lintResults = runCLI(`lint ${plugin}`);
     expect(lintResults).toContain('All files pass linting.');
 
-    expectTestsPass(await runCLIAsync(`test ${plugin}`));
-
     const buildResults = runCLI(`build ${plugin}`);
     expect(buildResults).toContain('Done compiling TypeScript files');
     checkFilesExist(
       `dist/libs/${plugin}/package.json`,
-      `dist/libs/${plugin}/generators.json`,
-      `dist/libs/${plugin}/executors.json`,
-      `dist/libs/${plugin}/src/index.js`,
-      `dist/libs/${plugin}/src/generators/${plugin}/schema.json`,
-      `dist/libs/${plugin}/src/generators/${plugin}/schema.d.ts`,
-      `dist/libs/${plugin}/src/generators/${plugin}/generator.js`,
-      `dist/libs/${plugin}/src/generators/${plugin}/files/src/index.ts__template__`,
-      `dist/libs/${plugin}/src/executors/build/executor.js`,
-      `dist/libs/${plugin}/src/executors/build/schema.d.ts`,
-      `dist/libs/${plugin}/src/executors/build/schema.json`
+      `dist/libs/${plugin}/src/index.js`
     );
     const project = readJson(`libs/${plugin}/project.json`);
     expect(project).toMatchObject({
       tags: [],
     });
-    const e2eProject = readJson(`apps/${plugin}-e2e/project.json`);
-
-    expect(e2eProject).toMatchObject({
-      tags: [],
-      implicitDependencies: [`${plugin}`],
-    });
   }, 90000);
 
   // the test invoke ensureNxProject, which points to @nrwl/workspace collection
@@ -75,7 +55,9 @@ describe('Nx Plugin', () => {
   // TODO: Re-enable this to work with pnpm
   it(`should run the plugin's e2e tests`, async () => {
     const plugin = uniq('plugin-name');
-    runCLI(`generate @nrwl/nx-plugin:plugin ${plugin} --linter=eslint`);
+    runCLI(
+      `generate @nrwl/nx-plugin:plugin ${plugin} --linter=eslint --e2eTestRunner jest`
+    );
     const e2eResults = runCLI(`e2e ${plugin}-e2e`);
     expect(e2eResults).toContain('Successfully ran target e2e');
     expect(await killPorts()).toBeTruthy();
@@ -195,7 +177,9 @@ describe('Nx Plugin', () => {
     const plugin = uniq('plugin');
     const goodGenerator = uniq('good-generator');
     const goodExecutor = uniq('good-executor');
+    const badExecutorBadImplPath = uniq('bad-executor');
     const goodMigration = uniq('good-migration');
+    const badFactoryPath = uniq('bad-generator');
     const badMigrationVersion = uniq('bad-version');
     const missingMigrationVersion = uniq('missing-version');
 
@@ -207,10 +191,18 @@ describe('Nx Plugin', () => {
       `generate @nrwl/nx-plugin:generator ${goodGenerator} --project=${plugin}`
     );
 
+    runCLI(
+      `generate @nrwl/nx-plugin:generator ${badFactoryPath} --project=${plugin}`
+    );
+
     runCLI(
       `generate @nrwl/nx-plugin:executor ${goodExecutor} --project=${plugin}`
     );
 
+    runCLI(
+      `generate @nrwl/nx-plugin:executor ${badExecutorBadImplPath} --project=${plugin}`
+    );
+
     runCLI(
       `generate @nrwl/nx-plugin:migration ${badMigrationVersion} --project=${plugin} --packageVersion="invalid"`
     );
@@ -226,16 +218,19 @@ describe('Nx Plugin', () => {
     updateFile(`libs/${plugin}/generators.json`, (f) => {
       const json = JSON.parse(f);
       // @proj/plugin:plugin has an invalid implementation path
-      json.generators[plugin].factory = `./generators/${plugin}/bad-path`;
+      json.generators[
+        badFactoryPath
+      ].factory = `./generators/${plugin}/bad-path`;
       // @proj/plugin:non-existant has a missing implementation path amd schema
       json.generators['non-existant-generator'] = {};
       return JSON.stringify(json);
     });
 
     updateFile(`libs/${plugin}/executors.json`, (f) => {
       const json = JSON.parse(f);
-      // @proj/plugin:build has an invalid implementation path
-      json.executors['build'].implementation = './executors/build/bad-path';
+      // @proj/plugin:badExecutorBadImplPath has an invalid implementation path
+      json.executors[badExecutorBadImplPath].implementation =
+        './executors/bad-path';
       // @proj/plugin:non-existant has a missing implementation path amd schema
       json.executors['non-existant-executor'] = {};
       return JSON.stringify(json);
@@ -249,7 +244,7 @@ describe('Nx Plugin', () => {
 
     const results = runCLI(`lint ${plugin}`, { silenceError: true });
     expect(results).toContain(
-      `${plugin}: Implementation path should point to a valid file`
+      `${badFactoryPath}: Implementation path should point to a valid file`
     );
     expect(results).toContain(
       `non-existant-generator: Missing required property - \`schema\``
@@ -260,7 +255,7 @@ describe('Nx Plugin', () => {
     expect(results).not.toContain(goodGenerator);
 
     expect(results).toContain(
-      `build: Implementation path should point to a valid file`
+      `${badExecutorBadImplPath}: Implementation path should point to a valid file`
     );
     expect(results).toContain(
       `non-existant-executor: Missing required property - \`schema\``
@@ -392,7 +387,7 @@ describe('Nx Plugin', () => {
     it('should create a plugin in the specified directory', () => {
       const plugin = uniq('plugin');
       runCLI(
-        `generate @nrwl/nx-plugin:plugin ${plugin} --linter=eslint --directory subdir `
+        `generate @nrwl/nx-plugin:plugin ${plugin} --linter=eslint --directory subdir --e2eTestRunner=jest`
       );
       checkFilesExist(`libs/subdir/${plugin}/package.json`);
       const pluginProject = readProjectConfig(`subdir-${plugin}`);

packages/nx-plugin/generators.json

@@ -11,8 +11,7 @@
     "e2e-project": {
       "factory": "./src/generators/e2e-project/e2e",
       "schema": "./src/generators/e2e-project/schema.json",
-      "description": "Create a E2E application for a Nx Plugin.",
-      "hidden": true
+      "description": "Create a E2E application for a Nx Plugin."
     },
     "migration": {
       "factory": "./src/generators/migration/migration",
@@ -51,8 +50,7 @@
     "e2e-project": {
       "factory": "./src/generators/e2e-project/e2e#e2eProjectSchematic",
       "schema": "./src/generators/e2e-project/schema.json",
-      "description": "Create a E2E application for a Nx Plugin.",
-      "hidden": true
+      "description": "Create a E2E application for a Nx Plugin."
     },
     "migration": {
       "factory": "./src/generators/migration/migration#migrationSchematic",

packages/nx-plugin/src/generators/e2e-project/e2e.spec.ts

@@ -140,24 +140,6 @@ describe('NxPlugin e2e-project Generator', () => {
     expect(tree.exists('apps/my-plugin-e2e/jest.config.ts')).toBeTruthy();
   });
 
-  it('should not generate tests when minimal flag is passed', async () => {
-    // ARRANGE & ACT
-    await e2eProjectGenerator(tree, {
-      pluginName: 'my-plugin',
-      pluginOutputPath: `dist/libs/my-plugin`,
-      npmPackageName: '@proj/my-plugin',
-      minimal: true,
-    });
-
-    const { root } = readProjectConfiguration(tree, 'my-plugin-e2e');
-
-    // ASSERT
-
-    expect(
-      tree.read(joinPathFragments(root, 'tests/my-plugin.spec.ts'), 'utf-8')
-    ).not.toContain("it('should create ");
-  });
-
   it('should setup the eslint builder', async () => {
     await e2eProjectGenerator(tree, {
       pluginName: 'my-plugin',

packages/nx-plugin/src/generators/e2e-project/e2e.ts

@@ -46,7 +46,6 @@ function normalizeOptions(host: Tree, options: Schema): NormalizedSchema {
 
   return {
     ...options,
-    minimal: options.minimal ?? false,
     projectName,
     linter: options.linter ?? Linter.EsLint,
     pluginPropertyName,

packages/nx-plugin/src/generators/e2e-project/files/tests/__pluginName__.spec.ts__tmpl__

@@ -3,6 +3,7 @@ import {
   ensureNxProject,
   readJson,
   runNxCommandAsync,
+  runNxCommand,
   uniq,
 } from '@nrwl/nx-plugin/testing';
 
@@ -23,39 +24,14 @@ describe('<%= pluginName %> e2e', () => {
     runNxCommandAsync('reset');
   });
 
-  <% if(!minimal) { %>
 
-  it('should create <%= pluginName %>', async () => {
-    const project = uniq('<%= pluginName %>');
-    await runNxCommandAsync(
-      `generate <%=npmPackageName%>:<%= pluginName %> ${project}`
-    );
-    const result = await runNxCommandAsync(`build ${project}`);
-    expect(result.stdout).toContain('Executor ran');
-  }, 120000);
-
-  describe('--directory', () => {
-    it('should create src in the specified directory', async () => {
-      const project = uniq('<%= pluginName %>');
-      await runNxCommandAsync(
-        `generate <%=npmPackageName%>:<%= pluginName %> ${project} --directory subdir`
-      );
-      expect(() =>
-        checkFilesExist(`libs/subdir/${project}/src/index.ts`)
-      ).not.toThrow();
-    }, 120000);
-  });
-
-  describe('--tags', () => {
-    it('should add tags to the project', async () => {
-      const projectName = uniq('<%= pluginName %>');
-      ensureNxProject('<%= npmPackageName %>', '<%= pluginOutputPath %>');
-      await runNxCommandAsync(
-        `generate <%=npmPackageName%>:<%= pluginName %> ${projectName} --tags e2etag,e2ePackage`
-      );
-      const project = readJson(`libs/${projectName}/project.json`);
-      expect(project.tags).toEqual(['e2etag', 'e2ePackage']);
-    }, 120000);
-  });
-  <% } %>
+  // Add some tests here to check that your plugin functionality works as expected.
+  // A sample test is included below to give you some ideas.
+  //
+  // it('should be able to build generated projects', async () => {
+  //   const name = uniq('proj')
+  //   await runNxCommandAsync(`generate <%= npmPackageName %>:{my-generator} --name ${name}`);
+  //   expect(() => runNxCommand('build ${proj}')).not.toThrow();
+  //   expect(() => checkFilesExist([`dist/${name}/index.js`])).not.toThrow();
+  // });
 });

packages/nx-plugin/src/generators/e2e-project/schema.d.ts

@@ -6,7 +6,6 @@ export interface Schema {
   projectDirectory?: string;
   pluginOutputPath?: string;
   jestConfig?: string;
-  minimal?: boolean;
   linter?: Linter;
   skipFormat?: boolean;
   rootProject?: boolean;

packages/nx-plugin/src/generators/executor/executor.ts

@@ -14,6 +14,7 @@ import type { Tree } from '@nrwl/devkit';
 import type { Schema } from './schema';
 import * as path from 'path';
 import { PackageJson } from 'nx/src/utils/package-json';
+import pluginLintCheckGenerator from '../lint-checks/generator';
 
 interface NormalizedSchema extends Schema {
   fileName: string;
@@ -70,25 +71,32 @@ function addHasherFiles(host: Tree, options: NormalizedSchema) {
   }
 }
 
-function createExecutorsJson(host: Tree, options: NormalizedSchema) {
+export async function createExecutorsJson(
+  host: Tree,
+  projectRoot: string,
+  projectName: string
+) {
   updateJson<PackageJson>(
     host,
-    joinPathFragments(options.projectRoot, 'package.json'),
+    joinPathFragments(projectRoot, 'package.json'),
     (json) => {
       json.executors ??= './executors.json';
       return json;
     }
   );
   writeJson<ExecutorsJson>(
     host,
-    joinPathFragments(options.projectRoot, 'executors.json'),
+    joinPathFragments(projectRoot, 'executors.json'),
     {
       executors: {},
     }
   );
+  await pluginLintCheckGenerator(host, {
+    projectName,
+  });
 }
 
-function updateExecutorJson(host: Tree, options: NormalizedSchema) {
+async function updateExecutorJson(host: Tree, options: NormalizedSchema) {
   const packageJson = readJson<PackageJson>(
     host,
     joinPathFragments(options.projectRoot, 'package.json')
@@ -102,7 +110,7 @@ function updateExecutorJson(host: Tree, options: NormalizedSchema) {
     executorsPath = joinPathFragments(options.projectRoot, 'executors.json');
   }
   if (!host.exists(executorsPath)) {
-    createExecutorsJson(host, options);
+    await createExecutorsJson(host, options.projectRoot, options.name);
   }
 
   return updateJson(host, executorsPath, (json) => {
@@ -158,7 +166,7 @@ export async function executorGenerator(host: Tree, schema: Schema) {
     addHasherFiles(host, options);
   }
 
-  updateExecutorJson(host, options);
+  await updateExecutorJson(host, options);
 }
 
 export default executorGenerator;

packages/nx-plugin/src/generators/generator/generator.spec.ts

@@ -179,8 +179,6 @@ describe('NxPlugin Generator Generator', () => {
         'libs/my-plugin/generators.json'
       );
 
-      console.log(generatorJson.generators['preset']);
-
       expect(
         generatorJson.generators['preset']['x-use-standalone-layout']
       ).toEqual(true);