chore: improve error message

packages/browser/src/node/pool.ts

@@ -134,7 +134,7 @@ export function createBrowserPool(vitest: Vitest): ProcessPool {
       await project._initBrowserProvider()
 
       if (!project.browser) {
-        throw new TypeError(`The browser server was not initialized for the ${project.name} project. This is a bug in Vitest. Please, open a new issue with reproduction.`)
+        throw new TypeError(`The browser server was not initialized${project.name ? ` for the "${project.name}" project` : ''}. This is a bug in Vitest. Please, open a new issue with reproduction.`)
       }
       await executeTests(method, project, files)
     }

packages/vitest/src/node/core.ts

@@ -276,15 +276,9 @@ export class Vitest {
     const projects = await this.resolveWorkspace(cliOptions)
     this.resolvedProjects = projects
     this.projects = projects
-    // const filters = toArray(resolved.project).map(s => wildcardPatternToRegExp(s))
-    // if (filters.length > 0) {
-    //   this.projects = this.projects.filter(p =>
-    //     filters.some(pattern => pattern.test(p.name)),
-    //   )
     if (!this.projects.length) {
       throw new Error(`No projects matched the filter "${toArray(resolved.project).join('", "')}".`)
     }
-    // }
     if (!this.coreWorkspaceProject) {
       this.coreWorkspaceProject = TestProject._createBasicProject(this)
     }
@@ -404,7 +398,7 @@ export class Vitest {
     // user doesn't have a workspace config, return default project
     if (!workspaceConfigPath) {
       // user can filter projects with --project flag, `getDefaultTestProject`
-      // return the project only if it matches the filter
+      // returns the project only if it matches the filter
       const project = getDefaultTestProject(this)
       if (!project) {
         return []

packages/vitest/src/node/workspace/resolveWorkspace.ts

@@ -101,7 +101,13 @@ export async function resolveWorkspace(
 
   // pretty rare case - the glob didn't match anything and there are no inline configs
   if (!projectPromises.length) {
-    throw new Error(`No projects were found. Make sure your configuration is correct. The workspace: ${JSON.stringify(workspaceDefinition)}`)
+    throw new Error(
+      [
+        'No projects were found. Make sure your configuration is correct. ',
+        vitest.config.project.length ? `The filter matched no projects: ${vitest.config.project.map(p => p.toString()).join(', ')}. ` : '',
+        `The workspace: ${JSON.stringify(workspaceDefinition, null, 4)}.`,
+      ].join(''),
+    )
   }
 
   const resolvedProjectsPromises = await Promise.allSettled(projectPromises)
@@ -445,7 +451,10 @@ export function getDefaultTestProject(vitest: Vitest): TestProject | null {
   if (!filter.length) {
     return project
   }
-  const hasProjects = getPotentialProjectNames(project).some(p => filter.some(pattern => pattern.test(p)))
+  // check for the project name and browser names
+  const hasProjects = getPotentialProjectNames(project).some(p =>
+    filter.some(pattern => pattern.test(p)),
+  )
   if (hasProjects) {
     return project
   }

test/config/fixtures/workspace/config-empty/vitest.config.js

@@ -0,0 +1,3 @@
+import { defineConfig } from 'vitest/config';
+
+export default defineConfig({})

test/config/test/workspace.test.ts

@@ -134,3 +134,24 @@ it('correctly inherits the root config', async () => {
   expect(stderr).toBe('')
   expect(stdout).toContain('repro.test.js > importing a virtual module')
 })
+
+it('fails if workspace is empty', async () => {
+  const { stderr } = await runVitest({
+    workspace: [],
+  })
+  expect(stderr).toContain('No projects were found. Make sure your configuration is correct. The workspace: [].')
+})
+
+it('fails if workspace is filtered by the project', async () => {
+  const { stderr } = await runVitest({
+    project: 'non-existing',
+    root: 'fixtures/workspace/config-empty',
+    config: './vitest.config.js',
+    workspace: [
+      './vitest.config.js',
+    ],
+  })
+  expect(stderr).toContain(`No projects were found. Make sure your configuration is correct. The filter matched no projects: /^non-existing$/i. The workspace: [
+    "./vitest.config.js"
+].`)
+})