fix(browser): don't print the deprecation notice in node_modules (#8779)

Author: sheremet-va

packages/browser/src/node/plugins/pluginContext.ts

@@ -25,7 +25,7 @@ export default function BrowserContext(globalServer: ParentBrowserProject): Plug
         return VIRTUAL_ID_CONTEXT
       }
       if (id === DEPRECATED_ID_CONTEXT) {
-        if (importer) {
+        if (importer && !importer.includes('/node_modules/')) {
           globalServer.vitest.logger.deprecate(
             `${importer} tries to load a deprecated "${id}" module. `
             + `This import will stop working in the next major version. `

pnpm-lock.yaml

@@ -3,7 +3,7 @@ import { page, userEvent } from 'vitest/browser'
 import Blog from '../../src/blog-app/blog'
 
 test('renders blog posts', async () => {
-  const screen = page.render(<Blog />)
+  const screen = await page.render(<Blog />)
 
   await expect.element(screen.getByRole('heading', { name: 'Blog' })).toBeInTheDocument()
 

test/browser/fixtures/locators/blog.test.tsx

@@ -43,7 +43,7 @@
     "react-dom": "^19.1.1",
     "url": "^0.11.4",
     "vitest": "workspace:*",
-    "vitest-browser-react": "^0.3.0",
+    "vitest-browser-react": "^2.0.0",
     "ws": "catalog:"
   }
 }

test/browser/package.json

@@ -1,27 +1,15 @@
-import { expect, test, vi } from 'vitest'
+import { expect, test } from 'vitest'
 import { instances, runBrowserTests } from './utils'
 
 test('locators work correctly', async () => {
-  const log = vi.fn()
   const { stderr, stdout } = await runBrowserTests({
     root: './fixtures/locators',
     reporters: [
       ['verbose', { isTTY: false }],
-      {
-        onInit(vitest) {
-          vitest.logger.deprecate = log
-        },
-      },
     ],
   })
 
   expect(stderr).toReportNoErrors()
-  expect(log).toHaveBeenCalledWith(
-    expect.stringContaining(
-      `tries to load a deprecated "@vitest/browser/context" module. `
-      + `This import will stop working in the next major version. Please, use "vitest/browser" instead.`,
-    ),
-  )
 
   instances.forEach(({ browser }) => {
     expect(stdout).toReportPassedTest('blog.test.tsx', browser)