test: stop the command e2e suite mutating the shared playground

Author: danielroe

packages/nuxt-cli/src/commands/docs.ts

@@ -13,7 +13,7 @@ import { logger } from '../utils/logger'
 import{resolveRootDir}from'../utils/paths'
 import{withSpinner}from'../utils/spinner'
 import{isInteractive}from'../utils/stdout'
-import{rootDirArgs}from'./_shared'
+import{cwdArgs}from'./_shared'
 
 constMAX_RESULTS=8
 
@@ -28,15 +28,15 @@ export default defineCommand({
 description: 'Words to search the documentation for',
 required: false,
 },
- ...rootDirArgs,
+ ...cwdArgs,
 open: {
 type: 'boolean',
 description: 'Open the best match in a browser',
 default: true,
 },
 },
 asyncrun(ctx){
-constquery=ctx.args._.slice(0,!ctx.args._.includes('--') ? undefined : ctx.args._.indexOf('--')).join(' ').trim()
+constquery=ctx.args._.join(' ').trim()
 if(!query){
 returnvisit(DOCS_PATH,ctx.args.open)
 }

packages/nuxt-cli/test/e2e/commands.spec.ts

@@ -10,9 +10,9 @@ import { getPort } from 'get-port-please'
 import{isWindows}from'std-env'
 import{x}from'tinyexec'
 import{describe,expect,it}from'vitest'
-import{fetchWithPolling}from'../utils'
+import{createPlaygroundFixture,fetchWithPolling}from'../utils'
 
-constfixtureDir=fileURLToPath(newURL('../../../../playground',import.meta.url))
+constfixtureDir=awaitcreatePlaygroundFixture('commands')
 constnuxi=fileURLToPath(newURL('../../bin/nuxi.mjs',import.meta.url))
 
 describe('commands',()=>{

packages/nuxt-cli/test/unit/help.spec.ts

@@ -253,17 +253,17 @@ describe('help', () => {
 expect(awaitusage(commands.docs,main)).toMatchInlineSnapshot(`
  "Search or open the Nuxt documentation (nuxt docs v0.0.0)
 
- USAGE nuxt docs [OPTIONS] [QUERY] [ROOTDIR]
+ USAGE nuxt docs [OPTIONS] [QUERY]
 
  ARGUMENTS
 
- QUERY Words to search the documentation for 
- ROOTDIR The root directory of your Nuxt project (default: .)
+ QUERY Words to search the documentation for
 
  OPTIONS
 
- --open Open the best match in a browser (Default: true)
- --no-open 
+ --cwd=<directory> Specify the root directory of your Nuxt project (Default: .)
+ --open Open the best match in a browser (Default: true) 
+ --no-open 
  "
  `)
 })

packages/nuxt-cli/test/utils/index.ts

@@ -28,25 +28,25 @@ export async function fetchWithPolling(url: string, options: RequestInit = {}, t
 }
 
 /**
- * Clone the shared `dev` fixture into a project private to a single spec file.
+ * Copy a fixture into a project private to a single spec file.
  *
- * The dev e2e specs each wipe `.nuxt` and take the dev lock inside their
- * fixture, so pointing more than one of them at `fixtures/dev` makes them race
- * whenever vitest runs the files in parallel.
+ * The e2e specs build, clear and take locks inside their fixture, so pointing
+ * more than one of them at the same directory makes them race whenever vitest
+ * runs the files in parallel.
  *
  * Installed packages are linked in one by one rather than by symlinking
  * `node_modules` wholesale, so that the caches tooling writes underneath it
  * (`node_modules/.cache/vite`) belong to a single spec.
  */
-exportasyncfunctioncreateDevFixture(name: string): Promise<string>{
-constsource=fileURLToPath(newURL('../fixtures/dev',import.meta.url))
+asyncfunctioncreateFixture(source: string,name: string): Promise<string>{
 consttarget=fileURLToPath(newURL(`../fixtures/.tmp-${name}`,import.meta.url))
 
 awaitrm(target,{recursive: true,force: true})
 awaitcp(source,target,{
 recursive: true,
-filter: entry=>!entry.includes('node_modules')&&!entry.includes(`${sep}.nuxt`),
+filter: entry=>!entry.includes('node_modules')&&!entry.includes(`${sep}.nuxt`)&&!entry.includes(`${sep}.output`),
 })
+
 constsourceModules=join(source,'node_modules')
 consttargetModules=join(target,'node_modules')
 awaitmkdir(targetModules,{recursive: true})
@@ -59,3 +59,13 @@ export async function createDevFixture(name: string): Promise<string> {
 
 returntarget
 }
+
+/** Copy of the shared `dev` fixture, private to a single spec file. */
+exportfunctioncreateDevFixture(name: string): Promise<string>{
+returncreateFixture(fileURLToPath(newURL('../fixtures/dev',import.meta.url)),name)
+}
+
+/** Copy of the workspace playground, private to a single spec file. */
+exportfunctioncreatePlaygroundFixture(name: string): Promise<string>{
+returncreateFixture(fileURLToPath(newURL('../../../../playground',import.meta.url)),name)
+}