refactor(test): change to not use env command (#2062)

* refactor(test): change to not use `env` command

* fix bun test

package.json

@@ -12,14 +12,14 @@
   "scripts": {
     "test": "tsc --noEmit && vitest --run",
     "test:watch": "vitest --watch",
-    "test:deno": "env NAME=Deno deno test --allow-read --allow-env runtime_tests/deno && deno test --no-lock -c runtime_tests/deno-jsx/deno.precompile.json runtime_tests/deno-jsx && deno test --no-lock -c runtime_tests/deno-jsx/deno.react-jsx.json runtime_tests/deno-jsx",
-    "test:bun": "env NAME=Bun bun test --jsx-import-source ../../src/jsx runtime_tests/bun/index.test.tsx",
+    "test:deno": "deno test --allow-read --allow-env runtime_tests/deno && deno test --no-lock -c runtime_tests/deno-jsx/deno.precompile.json runtime_tests/deno-jsx && deno test --no-lock -c runtime_tests/deno-jsx/deno.react-jsx.json runtime_tests/deno-jsx",
+    "test:bun": "bun test --jsx-import-source ../../src/jsx runtime_tests/bun/index.test.tsx",
     "test:fastly": "vitest --run --config ./runtime_tests/fastly/vitest.config.ts",
     "test:lagon": "start-server-and-test \"lagon dev runtime_tests/lagon/index.ts -e runtime_tests/lagon/.env.lagon\" http://127.0.0.1:1234 \"yarn vitest --run runtime_tests/lagon/index.test.ts --config runtime_tests/lagon/vitest.config.ts\"",
-    "test:node": "env NAME=Node vitest --run --config ./runtime_tests/node/vitest.config.ts",
+    "test:node": "vitest --run --config ./runtime_tests/node/vitest.config.ts",
     "test:wrangler": "vitest --run --config ./runtime_tests/wrangler/vitest.config.ts",
-    "test:lambda": "env NAME=Node vitest --run --config ./runtime_tests/lambda/vitest.config.ts",
-    "test:lambda-edge": "env NAME=Node vitest --run --config ./runtime_tests/lambda-edge/vitest.config.ts",
+    "test:lambda": "vitest --run --config ./runtime_tests/lambda/vitest.config.ts",
+    "test:lambda-edge": "vitest --run --config ./runtime_tests/lambda-edge/vitest.config.ts",
     "test:all": "yarn test && yarn test:deno && yarn test:bun && yarn test:fastly && yarn test:lagon && yarn test:node && yarn test:wrangler && yarn test:lambda && yarn test:lambda-edge",
     "lint": "eslint --ext js,ts src runtime_tests",
     "lint:fix": "eslint --ext js,ts src runtime_tests --fix",

runtime_tests/bun/index.test.tsx

@@ -12,6 +12,8 @@ import { HonoRequest } from '../../src/request'
 // Test just only minimal patterns.
 // Because others are tested well in Cloudflare Workers environment already.
 
+Bun.env.NAME = 'Bun'
+
 describe('Basic', () => {
   const app = new Hono()
   app.get('/a/:foo', (c) => {

runtime_tests/deno/hono.test.ts

@@ -7,6 +7,8 @@ import { assertEquals } from './deps.ts'
 // Test just only minimal patterns.
 // Because others are tested well in Cloudflare Workers environment already.
 
+Deno.env.set('NAME', 'Deno')
+
 Deno.test('Hello World', async () => {
   const app = new Hono()
   app.get('/:foo', (c) => {

runtime_tests/lambda-edge/vitest.config.ts

@@ -3,6 +3,9 @@ import { defineConfig } from 'vitest/config'
 
 export default defineConfig({
   test: {
+    env: {
+      NAME: 'Node',
+    },
     globals: true,
     include: ['**/runtime_tests/lambda-edge/**/*.+(ts|tsx|js)'],
     exclude: ['**/runtime_tests/lambda-edge/vitest.config.ts'],

runtime_tests/lambda/vitest.config.ts

@@ -3,6 +3,9 @@ import { defineConfig } from 'vitest/config'
 
 export default defineConfig({
   test: {
+    env: {
+      NAME: 'Node',
+    },
     globals: true,
     include: ['**/runtime_tests/lambda/**/*.+(ts|tsx|js)'],
     exclude: ['**/runtime_tests/lambda/vitest.config.ts', '**/runtime_tests/lambda/mock.ts'],

runtime_tests/node/vitest.config.ts

@@ -3,6 +3,9 @@ import { defineConfig } from 'vitest/config'
 
 export default defineConfig({
   test: {
+    env: {
+      NAME: 'Node',
+    },
     globals: true,
     include: ['**/runtime_tests/node/**/*.+(ts|tsx|js)'],
     exclude: ['**/runtime_tests/node/vitest.config.ts'],