ci: run tests with native resolver (#163)

Author: sapphi-red

.github/workflows/ci.yml

@@ -143,6 +143,78 @@ jobs:
     steps:
       - run: echo "Build & Test Failed"
 
+  test-native-plugins:
+    needs: changed
+    if: needs.changed.outputs.should_skip != 'true'
+    timeout-minutes: 20
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
+        node_version: [22]
+      fail-fast: false
+
+    name: "Build&Test: node-${{ matrix.node_version }}, ${{ matrix.os }} (native plugins)"
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Install pnpm
+        uses: pnpm/action-setup@a7487c7e89a18df4991f7f222e4898a00d66ddda # v4.1.0
+
+      - name: Set node version to ${{ matrix.node_version }}
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node_version }}
+          cache: "pnpm"
+
+      - name: Install deps
+        run: pnpm install
+
+      # Install playwright's binary under custom directory to cache
+      - name: (non-windows) Set Playwright path and Get playwright version
+        if: runner.os != 'Windows'
+        run: |
+          echo "PLAYWRIGHT_BROWSERS_PATH=$HOME/.cache/playwright-bin" >> $GITHUB_ENV
+          PLAYWRIGHT_VERSION="$(pnpm ls --depth 0 --json -w playwright-chromium | jq --raw-output '.[0].devDependencies["playwright-chromium"].version')"
+          echo "PLAYWRIGHT_VERSION=$PLAYWRIGHT_VERSION" >> $GITHUB_ENV
+      - name: (windows) Set Playwright path and Get playwright version
+        if: runner.os == 'Windows'
+        run: |
+          echo "PLAYWRIGHT_BROWSERS_PATH=$HOME\.cache\playwright-bin" >> $env:GITHUB_ENV
+          $env:PLAYWRIGHT_VERSION="$(pnpm ls --depth 0 --json -w playwright-chromium | jq --raw-output '.[0].devDependencies["playwright-chromium"].version')"
+          echo "PLAYWRIGHT_VERSION=$env:PLAYWRIGHT_VERSION" >> $env:GITHUB_ENV
+
+      - name: Cache Playwright's binary
+        uses: actions/cache@v4
+        with:
+          key: ${{ runner.os }}-playwright-bin-v1-${{ env.PLAYWRIGHT_VERSION }}
+          path: ${{ env.PLAYWRIGHT_BROWSERS_PATH }}
+          restore-keys: |
+            ${{ runner.os }}-playwright-bin-v1-
+
+      - name: Install Playwright
+        # does not need to explicitly set chromium after https://github.com/microsoft/playwright/issues/14862 is solved
+        run: pnpm playwright install chromium
+
+      - name: Build
+        run: pnpm run build
+
+      - name: Test unit
+        run: pnpm run test-unit
+        env:
+          _VITE_TEST_NATIVE_PLUGIN: 1
+
+      - name: Test serve
+        run: pnpm run test-serve
+        env:
+          _VITE_TEST_NATIVE_PLUGIN: 1
+
+      - name: Test build
+        run: pnpm run test-build
+        env:
+          _VITE_TEST_NATIVE_PLUGIN: 1
+
   lint:
     timeout-minutes: 10
     runs-on: ubuntu-latest

packages/vite/src/node/__tests__/resolve.spec.ts

@@ -214,18 +214,22 @@ describe('file url', () => {
       expect(resolved).toBeNull()
     })
 
-    test('non declared node builtin', async () => {
-      await expect(
-        run({
-          builtins: [
-            /* empty */
-          ],
-          idToResolve: 'node:fs',
-        }),
-      ).rejects.toThrowError(
-        /warning: Automatically externalized node built-in module "node:fs"/,
-      )
-    })
+    // skip for native plugin because logs are not output (https://github.com/rolldown/rolldown/issues/4290)
+    test.skipIf(!!process.env._VITE_TEST_NATIVE_PLUGIN)(
+      'non declared node builtin',
+      async () => {
+        await expect(
+          run({
+            builtins: [
+              /* empty */
+            ],
+            idToResolve: 'node:fs',
+          }),
+        ).rejects.toThrowError(
+          /warning: Automatically externalized node built-in module "node:fs"/,
+        )
+      },
+    )
 
     test('default to node-like builtins', async () => {
       const resolved = await run({

packages/vite/src/node/config.ts

@@ -714,7 +714,9 @@ export const configDefaults = Object.freeze({
     renderBuiltUrl: undefined,
     hmrPartialAccept: false,
     skipSsrTransform: false,
-    enableNativePlugin: false,
+    enableNativePlugin: process.env._VITE_TEST_NATIVE_PLUGIN
+      ? 'resolver'
+      : false,
   },
   future: {
     removePluginHookHandleHotUpdate: undefined,
@@ -1713,7 +1715,9 @@ export async function resolveConfig(
     experimental: {
       importGlobRestoreExtension: false,
       hmrPartialAccept: false,
-      enableNativePlugin: false,
+      enableNativePlugin: process.env._VITE_TEST_NATIVE_PLUGIN
+        ? 'resolver'
+        : false,
       ...config.experimental,
     },
     future: config.future,