test: Add dev-vitest integration test

Author: yaacovCR

integrationTests/README.md

@@ -14,10 +14,6 @@ Each subdirectory represents a different environment/bundler:
 - `ts` - tests for supported Typescript versions
 - `webpack` - tests for Webpack
 
-### Dual Package Hazard Tests
-
-- `dual-vitest` - ensure that vitest loads only a single package.
-
 ### Verifying Development Mode Tests
 
 Each subdirectory represents a different environment/bundler demonstrating enabling development mode via conditional exports or by explicitly importing `graphql/dev`:
@@ -32,6 +28,7 @@ Each subdirectory represents a different environment/bundler demonstrating enabl
 - `dev-esbuild`: via `esbuild --conditions=development test.js`
 - `dev-rollup`: via `@rollup/plugin-node-resolve` with `conditions: ['development']`
 - `dev-swc`: via `import 'graphql/dev'`
+- `dev-vitest`: via `resolve.conditions: ['development']`
 
 ### Verifying Production Mode Tests
 

integrationTests/dev-vitest/index.test.js

@@ -0,0 +1,15 @@
+import { isObjectType } from 'graphql';
+// eslint-disable-next-line n/no-missing-import
+import { expect, test } from 'vitest';
+
+class GraphQLObjectTypeMimic {
+  get [Symbol.toStringTag]() {
+    return 'GraphQLObjectType';
+  }
+}
+
+test('isObjectType should throw in development mode for instances from another realm/module', () => {
+  expect(() => isObjectType(new GraphQLObjectTypeMimic())).toThrowError(
+    /from another module or realm/,
+  );
+});

integrationTests/dev-vitest/package.json

@@ -0,0 +1,12 @@
+{
+  "description": "graphql-js development condition should work with Vitest",
+  "private": true,
+  "type": "module",
+  "scripts": {
+    "test": "vitest run"
+  },
+  "dependencies": {
+    "graphql": "file:../graphql.tgz",
+    "vitest": "^1.0.0"
+  }
+}

integrationTests/dev-vitest/vitest.config.js

@@ -0,0 +1,14 @@
+// eslint-disable-next-line n/no-missing-import
+import { defineConfig } from 'vitest/config';
+
+const vitestConfig = defineConfig({
+  resolve: {
+    conditions: ['development'],
+  },
+  test: {
+    include: ['**/*.test.js'],
+  },
+});
+
+// eslint-disable-next-line no-restricted-exports, import/no-default-export
+export default vitestConfig;

resources/integration-test.ts

@@ -16,7 +16,7 @@ describe('Integration Tests', () => {
   const archiveName = npm({ cwd: tmpDirPath(), quiet: true }).pack(distDir);
   fs.renameSync(tmpDirPath(archiveName), tmpDirPath('graphql.tgz'));
 
-  // npm().run('build:deno');
+  npm().run('build:deno');
 
   function testOnNodeProject(projectName: string) {
     const projectPath = tmpDirPath(projectName);
@@ -29,9 +29,6 @@ describe('Integration Tests', () => {
     }).timeout(120000);
   }
 
-  testOnNodeProject('dual-vitest');
-  return;
-
   testOnNodeProject('ts');
   testOnNodeProject('node');
   testOnNodeProject('webpack');
@@ -47,13 +44,14 @@ describe('Integration Tests', () => {
   testOnNodeProject('dev-rollup');
   testOnNodeProject('dev-esbuild');
   testOnNodeProject('dev-swc');
+  testOnNodeProject('dev-vitest');
 
   // Production mode tests
   testOnNodeProject('prod-node');
   testOnNodeProject('prod-deno');
   testOnNodeProject('prod-bun');
   testOnNodeProject('prod-webpack');
-  testOnNodeProject('dev-rspack');
+  testOnNodeProject('prod-rspack');
   testOnNodeProject('prod-rollup');
   testOnNodeProject('prod-esbuild');
   testOnNodeProject('prod-swc');