fix(node): Ensure docker file can be generated when nest framework is…

… set

closes: #17343

e2e/node/src/__snapshots__/node.test.ts.snap

@@ -0,0 +1,29 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Node Applications should generate a nest application with docker 1`] = `
+"# This file is generated by Nx.
+#
+# Build the docker image with \`npx nx docker-build nestapp3120610\`.
+# Tip: Modify "docker-build" options in project.json to change docker build args.
+#
+# Run the container with \`docker run -p 3000:3000 -t nestapp3120610\`.
+FROM docker.io/node:lts-alpine
+
+ENV HOST=0.0.0.0
+ENV PORT=3000
+
+WORKDIR /app
+
+RUN addgroup --system nestapp3120610 && \\
+          adduser --system -G nestapp3120610 nestapp3120610
+
+COPY dist/nestapp3120610 nestapp3120610/
+RUN chown -R nestapp3120610:nestapp3120610 .
+
+# You can remove this install step if you build with \`--bundle\` option.
+# The bundled output will include external dependencies.
+RUN npm --prefix nestapp3120610 --omit=dev -f install
+
+CMD [ "node", "nestapp3120610" ]
+"
+`;

e2e/node/src/node.test.ts

@@ -23,7 +23,7 @@ import {
   updateFile,
   updateJson,
 } from '@nx/e2e/utils';
-import { exec, execSync } from 'child_process';
+import { execSync } from 'child_process';
 import * as http from 'http';
 import { getLockFileName } from '@nx/js';
 import { satisfies } from 'semver';
@@ -58,7 +58,7 @@ describe('Node Applications', () => {
   beforeAll(() => {
     originalEnvPort = process.env.PORT;
     newProject({
-      packages: ['@nx/node', '@nx/express', '@nx/nest'],
+      packages: ['@nx/node', '@nx/express', '@nx/nest', '@nx/webpack'],
     });
   });
 
@@ -340,6 +340,19 @@ module.exports = {
     await promisifiedTreeKill(p.pid, 'SIGKILL');
   }, 120000);
 
+  it('should generate a nest application with docker', async () => {
+    const nestapp = uniq('nestapp');
+
+    runCLI(
+      `generate @nx/node:app ${nestapp} --project-name-and-root-format=as-provided --bundler=webpack --framework=nest --docker`
+    );
+
+    checkFilesExist(`${nestapp}/Dockerfile`);
+
+    const dockerFile = readFile(`${nestapp}/Dockerfile`);
+    expect(dockerFile).toMatchSnapshot();
+  });
+
   // TODO(crystal, @ndcunningham): how do we handle this now?
   // Revisit when NxWebpackPlugin({}) outputFilename is working.
   xit('should be able to run ESM applications', async () => {

packages/node/src/generators/application/application.ts

@@ -396,9 +396,19 @@ export async function applicationGeneratorInternal(tree: Tree, schema: Schema) {
       ...options,
       skipFormat: true,
     });
+    tasks.push(nestTasks);
+
+    if (options.docker) {
+      const dockerTask = await setupDockerGenerator(tree, {
+        ...options,
+        project: options.name,
+        skipFormat: true,
+      });
+      tasks.push(dockerTask);
+    }
     return runTasksInSerial(
       ...[
-        nestTasks,
+        ...tasks,
         () => {
           logShowProjectCommand(options.name);
         },