chore: remove unused tests

Author: mandarini

test/websocket-factory.test.ts

@@ -115,34 +115,31 @@ describe('WebSocketFactory', () => {
       global.process = { versions: { node: '14.0.0' } } as any
     })
 
-    test('detects ws package', () => {
-      const spy = vi.spyOn(WebSocketFactory as any, 'dynamicRequire')
-      spy.mockReturnValue(MockWebSocket)
-
-      const env = (WebSocketFactory as any).detectEnvironment()
-      expect(env.type).toBe('ws')
-      expect(env.constructor).toBe(MockWebSocket)
-    })
-
-    test('handles missing ws package', () => {
-      const spy = vi.spyOn(WebSocketFactory as any, 'dynamicRequire')
-      spy.mockReturnValue(null)
-
+    test('detects missing native WebSocket in Node.js < 22', () => {
       const env = (WebSocketFactory as any).detectEnvironment()
       expect(env.type).toBe('unsupported')
       expect(env.error).toContain(
-        'Node.js 14 detected without WebSocket support'
+        'Node.js 14 detected without native WebSocket support'
+      )
+      expect(env.workaround).toContain(
+        'install "ws" package and provide it via the transport option'
       )
-      expect(env.workaround).toContain('Install the "ws" package')
     })
 
-    test('handles ws package with WebSocket property', () => {
-      const spy = vi.spyOn(WebSocketFactory as any, 'dynamicRequire')
-      spy.mockReturnValue({ WebSocket: MockWebSocket })
-
+    test('provides helpful error message for Node.js users', () => {
       const env = (WebSocketFactory as any).detectEnvironment()
-      expect(env.type).toBe('ws')
-      expect(env.constructor).toBe(MockWebSocket)
+      expect(env.type).toBe('unsupported')
+      expect(env.workaround).toContain('import ws from "ws"')
+      expect(env.workaround).toContain('transport: ws')
+    })
+
+    test.skip('throws error when trying to create WebSocket without transport', () => {
+      // Note: This test is skipped because the test runner (Vitest) provides
+      // WebSocket even when we delete it from globals. The actual functionality
+      // works correctly in real Node.js environments without WebSocket.
+      expect(() => {
+        WebSocketFactory.createWebSocket('wss://example.com')
+      }).toThrow()
     })
   })
 
@@ -162,24 +159,16 @@ describe('WebSocketFactory', () => {
       expect(env.constructor).toBe(MockWebSocket)
     })
 
-    test('falls back to undici', () => {
-      const spy = vi.spyOn(WebSocketFactory as any, 'dynamicRequire')
-      spy.mockReturnValue({ WebSocket: MockWebSocket })
-
-      const env = (WebSocketFactory as any).detectEnvironment()
-      expect(env.type).toBe('native')
-      expect(env.constructor).toBe(MockWebSocket)
-    })
-
-    test('handles missing undici', () => {
-      const spy = vi.spyOn(WebSocketFactory as any, 'dynamicRequire')
-      spy.mockReturnValue(null)
-
+    test('handles missing native WebSocket in Node.js 22+', () => {
+      // Node.js 22+ without native WebSocket (shouldn't happen in practice)
       const env = (WebSocketFactory as any).detectEnvironment()
       expect(env.type).toBe('unsupported')
       expect(env.error).toContain(
         'Node.js 22 detected but native WebSocket not found'
       )
+      expect(env.workaround).toContain(
+        'Provide a WebSocket implementation via the transport option'
+      )
     })
   })
 
@@ -254,14 +243,17 @@ describe('WebSocketFactory', () => {
         type: 'unsupported',
         constructor: null,
         error: 'Unknown JavaScript runtime without WebSocket support.',
-        workaround: "Ensure you're running in a supported environment (browser, Node.js, Deno) or provide a custom WebSocket implementation."
+        workaround:
+          "Ensure you're running in a supported environment (browser, Node.js, Deno) or provide a custom WebSocket implementation.",
       })
 
       // Now test that getWebSocketConstructor throws with both error and workaround
       expect(() => {
         WebSocketFactory.getWebSocketConstructor()
-      }).toThrow(/Unknown JavaScript runtime[\s\S]*Ensure you're running in a supported environment/)
-      
+      }).toThrow(
+        /Unknown JavaScript runtime[\s\S]*Ensure you're running in a supported environment/
+      )
+
       spy.mockRestore()
     })
   })
@@ -276,34 +268,4 @@ describe('WebSocketFactory', () => {
       expect(WebSocketFactory.isWebSocketSupported()).toBe(false)
     })
   })
-
-  describe('dynamicRequire', () => {
-    test('returns null when process is undefined', () => {
-      delete global.process
-      const result = (WebSocketFactory as any).dynamicRequire('test-module')
-      expect(result).toBeNull()
-    })
-
-    test('returns null when require is undefined', () => {
-      global.process = { versions: { node: '14.0.0' } } as any
-      // Simulate environment where require is not available
-      const originalRequire = global.require
-      delete global.require
-
-      const result = (WebSocketFactory as any).dynamicRequire('test-module')
-      expect(result).toBeNull()
-
-      global.require = originalRequire
-    })
-
-    test('handles require throwing error', () => {
-      global.process = { versions: { node: '14.0.0' } } as any
-      global.require = vi.fn().mockImplementation(() => {
-        throw new Error('Module not found')
-      })
-
-      const result = (WebSocketFactory as any).dynamicRequire('test-module')
-      expect(result).toBeNull()
-    })
-  })
 })