test(qwik-nx): update e2e

Author: dmitry-stepanenko

e2e/qwik-nx-e2e/tests/application-basic-behavior.suite.ts

@@ -21,7 +21,7 @@ import { names } from '@nx/devkit';
 export function testApplicationBasicBehavior(generator: 'app' | 'preset') {
   const isPreset = generator === 'preset';
   const appPathPrefix = isPreset ? 'apps/' : '';
-  const libPathPrefix = isPreset ? 'libs/' : '';
+  const libPathPrefix = '';
   const devServerPort = 4212;
   const previewServerPort = 4232;
   describe(`Basic behavior with ${generator} generator`, () => {
@@ -77,7 +77,7 @@ export function testApplicationBasicBehavior(generator: 'app' | 'preset') {
       'should create qwik-nx',
       async () => {
         const result = await runNxCommandAsync(`build ${project}`);
-        expect(result.stdout).toContain(
+        expect(stripConsoleColors(result.stdout)).toContain(
           `Running type check for the "${project}"..`
         );
         expect(
@@ -91,7 +91,7 @@ export function testApplicationBasicBehavior(generator: 'app' | 'preset') {
             .join('')
             .replace(/\n|\s/g, '')
         );
-        expect(result.stdout).toContain(
+        expect(stripConsoleColors(result.stdout)).toContain(
           `Successfully ran target build for project ${project}`
         );
         expect(() =>
@@ -167,7 +167,7 @@ export function testApplicationBasicBehavior(generator: 'app' | 'preset') {
       'unit tests should pass in the created app',
       async () => {
         const result = await runNxCommandAsync(`test ${project}`);
-        expect(result.stdout).toContain(
+        expect(stripConsoleColors(result.stdout)).toContain(
           `Successfully ran target test for project ${project}`
         );
       },
@@ -186,7 +186,7 @@ export function testApplicationBasicBehavior(generator: 'app' | 'preset') {
         const result = await runNxCommandAsync(`test ${project}`, {
           silenceError: true,
         });
-        expect(result.stdout).toContain(
+        expect(stripConsoleColors(result.stdout)).toContain(
           `Running target test for project ${project} failed`
         );
       },
@@ -211,7 +211,7 @@ export function testApplicationBasicBehavior(generator: 'app' | 'preset') {
             .join('')
             .replace(/\n|\s/g, '')
         );
-        expect(result.stdout).toContain(
+        expect(stripConsoleColors(result.stdout)).toContain(
           `Successfully ran target build for project ${project}`
         );
       },

e2e/qwik-nx-e2e/tests/qwik-nx-cloudflare.spec.ts

@@ -11,6 +11,7 @@ import {
   killPort,
   killPorts,
   DEFAULT_E2E_TIMEOUT,
+  stripConsoleColors,
 } from '@qwikifiers/e2e/utils';
 
 const CLOUDFLARE_PREVIEW_PORT = 4173;
@@ -49,7 +50,7 @@ describe('qwik nx cloudflare generator', () => {
       'should be able to successfully build the application',
       async () => {
         const result = await runNxCommandAsync(`build-cloudflare ${project}`);
-        expect(result.stdout).toContain(
+        expect(stripConsoleColors(result.stdout)).toContain(
           `Successfully ran target build for project ${project}`
         );
         expect(() =>

e2e/qwik-nx-e2e/tests/qwik-nx-netlify.spec.ts

@@ -11,6 +11,7 @@ import {
   killPort,
   killPorts,
   DEFAULT_E2E_TIMEOUT,
+  stripConsoleColors,
 } from '@qwikifiers/e2e/utils';
 
 const NETLIFY_PREVIEW_PORT = 8888;
@@ -41,7 +42,7 @@ describe('qwik nx netlify generator', () => {
       'should be able to successfully build the application',
       async () => {
         const result = await runNxCommandAsync(`build-netlify ${project}`);
-        expect(result.stdout).toContain(
+        expect(stripConsoleColors(result.stdout)).toContain(
           `Successfully ran target build for project ${project}`
         );
         expect(() =>

e2e/qwik-nx-e2e/tests/storybook.spec.ts

@@ -36,7 +36,7 @@ describe('qwikNxVite plugin e2e', () => {
     describe('Applying storybook for existing application', () => {
       beforeAll(async () => {
         await runNxCommandAsync(
-          `generate qwik-nx:app ${appProject} --e2eTestRunner=none --no-interactive`
+          `generate qwik-nx:app --directory=apps/${appProject} --e2eTestRunner=none --no-interactive`
         );
         await runNxCommandAsync(
           `generate qwik-nx:storybook-configuration ${appProject} --no-interactive`
@@ -48,7 +48,7 @@ describe('qwikNxVite plugin e2e', () => {
     describe('Applying storybook for existing library', () => {
       beforeAll(async () => {
         await runNxCommandAsync(
-          `generate qwik-nx:library ${libProject} --no-interactive`
+          `generate qwik-nx:library --directory=libs/${libProject} --no-interactive`
         );
         await runNxCommandAsync(
           `generate qwik-nx:storybook-configuration ${libProject} --no-interactive`
@@ -61,7 +61,7 @@ describe('qwikNxVite plugin e2e', () => {
     describe('Generating a new library with storybook configuration', () => {
       beforeAll(async () => {
         await runNxCommandAsync(
-          `generate qwik-nx:library ${secondLibProject} --storybookConfiguration=true --no-interactive`
+          `generate qwik-nx:library --directory=libs/${secondLibProject} --storybookConfiguration=true --no-interactive`
         );
         await addAdditionalStories(secondLibProject);
       }, DEFAULT_E2E_TIMEOUT);

e2e/utils/index.ts

@@ -10,6 +10,7 @@ import * as treeKill from 'tree-kill';
 import {
   detectPackageManager,
   getPackageManagerCommand,
+  PackageManager,
 } from 'nx/src/utils/package-manager';
 
 export const DEFAULT_E2E_TIMEOUT = 3 * 60 * 1000;
@@ -26,7 +27,7 @@ function getAdditionalPackageManagerCommands(): {
   createWorkspace: string;
   runNx: string;
 } {
-  const pm = detectPackageManager();
+  const pm = detectPackageManager() as Exclude<PackageManager, 'bun'>;
   const [npmMajorVersion] = execSync(`npm -v`).toString().split('.');
   const publishedVersion = execSync('npm view nx version');
   switch (pm) {

packages/qwik-nx/src/generators/application/files/vite.config.ts__template__

@@ -6,6 +6,7 @@ import { qwikNxVite } from 'qwik-nx/plugins';
 
 export default defineConfig({
   cacheDir: '<%= offsetFromRoot %>node_modules/.vite/<%= projectRoot %>',
+  root: '<%= projectRoot %>',
   plugins: [
     qwikNxVite(),
     qwikCity(), 

packages/qwik-nx/src/generators/preset/generator.ts

@@ -1,13 +1,17 @@
-import { Tree } from '@nx/devkit';
+import { joinPathFragments, Tree } from '@nx/devkit';
 import { QwikWorkspacePresetGeneratorSchema } from './schema';
 
 export default async function (
   tree: Tree,
   options: QwikWorkspacePresetGeneratorSchema
 ) {
-  options.name = options.qwikAppName ?? options.name;
+  if (options.qwikAppName.split('/').length === 1) {
+    options.directory = joinPathFragments('apps', options.qwikAppName);
+  } else {
+    options.directory = options.qwikAppName;
+  }
   options.style = options.qwikAppStyle ?? options.style;
   return await import('../application/generator').then(({ appGenerator }) =>
-    appGenerator(tree, { ...options, directory: 'apps' })
+    appGenerator(tree, options)
   );
 }

packages/qwik-nx/src/generators/preset/schema.d.ts

@@ -2,10 +2,9 @@ import { Linter } from '@nx/eslint';
 
 export interface QwikWorkspacePresetGeneratorSchema
   extends QwikAppGeneratorSchema {
-  name: string;
   qwikAppName: string;
   tags?: string;
-  directory?: string;
+  directory: string;
 
   style: 'css' | 'scss' | 'styl' | 'less' | 'none';
   qwikAppStyle: 'css' | 'scss' | 'styl' | 'less' | 'none';

packages/qwik-nx/src/generators/storybook-configuration/generator.ts

@@ -9,6 +9,7 @@ import {
   offsetFromRoot,
   ProjectConfiguration,
   readProjectConfiguration,
+  runTasksInSerial,
   Tree,
 } from '@nx/devkit';
 import { Linter } from '@nx/eslint';
@@ -84,7 +85,8 @@ export async function storybookConfigurationGenerator(
 
   const { oldFormat } = await getStorybookVersion();
 
-  await configurationGenerator(tree, {
+  const tasks: GeneratorCallback[] = [];
+  const addStorybookCallback = await configurationGenerator(tree, {
     storybook7UiFramework: '@storybook/html-webpack5',
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-ignore
@@ -98,11 +100,14 @@ export async function storybookConfigurationGenerator(
     configureCypress: false,
     project: projectConfig.name!,
   });
+  tasks.push(addStorybookCallback);
 
   addFiles(tree, normalizedOptions, projectConfig);
   await formatFiles(tree);
 
-  return addStorybookDependencies(tree);
+  const addStorybookDependenciesCallback = await addStorybookDependencies(tree);
+  tasks.push(addStorybookDependenciesCallback);
+  return runTasksInSerial(...tasks);
 }
 
 async function addStorybookDependencies(

packages/qwik-nx/src/utils/add-common-qwik-dependencies.spec.ts

@@ -32,23 +32,4 @@ describe('init generator', () => {
     const { devDependencies } = readJson(appTree, 'package.json');
     expect(devDependencies['@builder.io/qwik']).toBe('my-version');
   });
-
-  it('should override existing versions if they are listed in "unsupportedPackageVersions" 1', async () => {
-    updateJson(appTree, 'package.json', (json) => {
-      (json.devDependencies ??= {})['vite'] = '5.1.1';
-      return json;
-    });
-    await addCommonQwikDependencies(appTree);
-    const { devDependencies } = readJson(appTree, 'package.json');
-    expect(devDependencies['vite']).toBe('~5.2.0');
-  });
-  it('should not override existing versions if they are listed in "unsupportedPackageVersions" and have proper version', async () => {
-    updateJson(appTree, 'package.json', (json) => {
-      (json.devDependencies ??= {})['vite'] = '5.2.1';
-      return json;
-    });
-    await addCommonQwikDependencies(appTree);
-    const { devDependencies } = readJson(appTree, 'package.json');
-    expect(devDependencies['vite']).toBe('5.2.1');
-  });
 });