From cbf2c4369a9feb7bc248356984103a2c7b91f579 Mon Sep 17 00:00:00 2001
From: Jack Hsu <jack.hsu@gmail.com>
Date: Fri, 2 Feb 2024 12:40:01 -0500
Subject: [PATCH] chore(misc): re-enable convert-to-monorepo test to test
 without Nx plugins for now

---
 e2e/nx-misc/src/workspace-legacy.test.ts | 49 ++++++++++++++++++++++++
 e2e/nx-misc/src/workspace.test.ts        | 37 ++++--------------
 2 files changed, 56 insertions(+), 30 deletions(-)
 create mode 100644 e2e/nx-misc/src/workspace-legacy.test.ts

diff --git a/e2e/nx-misc/src/workspace-legacy.test.ts b/e2e/nx-misc/src/workspace-legacy.test.ts
new file mode 100644
index 0000000000000..5f080812c5f73
--- /dev/null
+++ b/e2e/nx-misc/src/workspace-legacy.test.ts
@@ -0,0 +1,49 @@
+import {
+  checkFilesExist,
+  cleanupProject,
+  newProject,
+  runCLI,
+  runE2ETests,
+  uniq,
+} from '@nx/e2e/utils';
+
+let proj: string;
+
+describe('@nx/workspace:convert-to-monorepo', () => {
+  beforeEach(() => {
+    proj = newProject({ packages: ['@nx/react', '@nx/js'] });
+  });
+
+  afterEach(() => cleanupProject());
+
+  it('should convert a standalone webpack and jest react project to a monorepo (legacy)', async () => {
+    const reactApp = uniq('reactapp');
+    runCLI(
+      `generate @nx/react:app ${reactApp} --rootProject=true --bundler=webpack --unitTestRunner=jest --e2eTestRunner=cypress --no-interactive`,
+      {
+        env: {
+          NX_ADD_PLUGINS: 'false',
+        },
+      }
+    );
+
+    runCLI('generate @nx/workspace:convert-to-monorepo --no-interactive', {
+      env: {
+        NX_ADD_PLUGINS: 'false',
+      },
+    });
+
+    checkFilesExist(
+      `apps/${reactApp}/src/main.tsx`,
+      `apps/e2e/cypress.config.ts`
+    );
+
+    expect(() => runCLI(`build ${reactApp}`)).not.toThrow();
+    expect(() => runCLI(`test ${reactApp}`)).not.toThrow();
+    expect(() => runCLI(`lint ${reactApp}`)).not.toThrow();
+    expect(() => runCLI(`lint e2e`)).not.toThrow();
+    if (runE2ETests()) {
+      expect(() => runCLI(`e2e e2e`)).not.toThrow();
+    }
+  });
+});
diff --git a/e2e/nx-misc/src/workspace.test.ts b/e2e/nx-misc/src/workspace.test.ts
index 6edb060ece66a..8290c2fb784d8 100644
--- a/e2e/nx-misc/src/workspace.test.ts
+++ b/e2e/nx-misc/src/workspace.test.ts
@@ -1,18 +1,18 @@
 import {
   checkFilesExist,
-  newProject,
-  readJson,
   cleanupProject,
-  runCLI,
-  uniq,
-  updateFile,
-  readFile,
   exists,
-  tmpProjPath,
   getPackageManagerCommand,
   getSelectedPackageManager,
+  newProject,
+  readFile,
+  readJson,
+  runCLI,
   runCommand,
   runE2ETests,
+  tmpProjPath,
+  uniq,
+  updateFile,
 } from '@nx/e2e/utils';
 import { join } from 'path';
 
@@ -25,29 +25,6 @@ describe('@nx/workspace:convert-to-monorepo', () => {
 
   afterEach(() => cleanupProject());
 
-  // TODO(crystal, @jaysoo): Investigate why this test is failing
-  xit('should convert a standalone webpack and jest react project to a monorepo', async () => {
-    const reactApp = uniq('reactapp');
-    runCLI(
-      `generate @nx/react:app ${reactApp} --rootProject=true --bundler=webpack --unitTestRunner=jest --e2eTestRunner=cypress --no-interactive`
-    );
-
-    runCLI('generate @nx/workspace:convert-to-monorepo --no-interactive');
-
-    checkFilesExist(
-      `apps/${reactApp}/src/main.tsx`,
-      `apps/e2e/cypress.config.ts`
-    );
-
-    expect(() => runCLI(`build ${reactApp}`)).not.toThrow();
-    expect(() => runCLI(`test ${reactApp}`)).not.toThrow();
-    expect(() => runCLI(`lint ${reactApp}`)).not.toThrow();
-    expect(() => runCLI(`lint e2e`)).not.toThrow();
-    if (runE2ETests()) {
-      expect(() => runCLI(`e2e e2e`)).not.toThrow();
-    }
-  });
-
   it('should be convert a standalone vite and playwright react project to a monorepo', async () => {
     const reactApp = uniq('reactapp');
     runCLI(