fix(bundling): react-standalone take into account bundler (#14573)

Author: mandarini

e2e/workspace-create/src/create-nx-workspace.test.ts

@@ -32,7 +32,7 @@ describe('create-nx-workspace', () => {
     checkFilesExist('project.json');
   });
 
-  it('should create a workspace with a single react app at the root', () => {
+  it('should create a workspace with a single react app with vite at the root', () => {
     const wsName = uniq('react');
 
     runCreateWorkspace(wsName, {
@@ -45,6 +45,23 @@ describe('create-nx-workspace', () => {
 
     checkFilesExist('package.json');
     checkFilesExist('project.json');
+    checkFilesExist('vite.config.ts');
+  });
+
+  it('should create a workspace with a single react app with webpack at the root', () => {
+    const wsName = uniq('react');
+
+    runCreateWorkspace(wsName, {
+      preset: 'react-standalone',
+      appName: wsName,
+      style: 'css',
+      packageManager,
+      bundler: 'webpack',
+    });
+
+    checkFilesExist('package.json');
+    checkFilesExist('project.json');
+    checkFilesExist('webpack.config.js');
   });
 
   it('should be able to create an empty workspace built for apps', () => {

packages/create-nx-workspace/bin/create-nx-workspace.ts

@@ -241,6 +241,7 @@ async function main(parsedArgs: yargs.Arguments<Arguments>) {
     skipGit,
     commit,
     framework,
+    bundler,
   } = parsedArgs;
 
   output.log({
@@ -266,6 +267,7 @@ async function main(parsedArgs: yargs.Arguments<Arguments>) {
       nxCloud,
       defaultBase,
       framework,
+      bundler,
     }
   );
 

packages/workspace/src/generators/new/generate-preset.ts

@@ -73,6 +73,7 @@ export function generatePreset(host: Tree, opts: NormalizedSchema) {
       opts.linter ? `--linter=${opts.linter}` : null,
       opts.npmScope ? `--npmScope=${opts.npmScope}` : `--npmScope=${opts.name}`,
       opts.preset ? `--preset=${opts.preset}` : null,
+      opts.bundler ? `--bundler=${opts.bundler}` : null,
       opts.framework ? `--framework=${opts.framework}` : null,
       opts.packageManager ? `--packageManager=${opts.packageManager}` : null,
       parsedArgs.interactive ? '--interactive=true' : '--interactive=false',

packages/workspace/src/generators/new/new.ts

@@ -26,6 +26,7 @@ interface Schema {
   defaultBase: string;
   framework?: string;
   linter?: Linter;
+  bundler?: 'vite' | 'webpack';
   packageManager?: PackageManager;
 }
 

packages/workspace/src/generators/preset/preset.ts

@@ -71,7 +71,7 @@ async function createPreset(tree: Tree, options: Schema) {
       rootProject: true,
       bundler: options.bundler ?? 'vite',
       e2eTestRunner: 'cypress',
-      unitTestRunner: 'vitest',
+      unitTestRunner: options.bundler === 'vite' ? 'vitest' : 'jest',
     });
   } else if (options.preset === Preset.NextJs) {
     const { applicationGenerator: nextApplicationGenerator } = require('@nrwl' +