chore: wip

Author: chrisbbreuer

test/composer-parser.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, it } from 'bun:test'
+import { describe, expect, it, beforeEach, afterEach, spyOn } from 'bun:test'
 import {
   generateComposerUpdates,
   isComposerFile,
@@ -227,39 +227,36 @@ describe('Composer Parser', () => {
       },
     }, null, 2)
 
-    // We'll mock fs.readFileSync inline in each test for better control
-
     it('should generate updates for composer.json files', async () => {
-      // Create a mock composer.json file for this test
+      // Mock fs readFileSync using spyOn
       const fs = await import('node:fs')
-      const path = await import('node:path')
-      const testComposerPath = path.join(process.cwd(), 'test-updates-composer.json')
+      const readFileSpy = spyOn(fs, 'readFileSync')
 
-      // Write test file
-      fs.writeFileSync(testComposerPath, mockComposerContent)
+      readFileSpy.mockImplementation(((filePath: any) => {
+        if (String(filePath).endsWith('composer.json')) {
+          return mockComposerContent
+        }
+        throw new Error(`File not found: ${filePath}`)
+      }) as any)
 
       try {
         const updates = [
-          { name: 'laravel/framework', newVersion: '10.16.0', file: 'test-updates-composer.json' },
-          { name: 'phpunit/phpunit', newVersion: '10.3.0', file: 'test-updates-composer.json' },
+          { name: 'laravel/framework', newVersion: '10.16.0', file: 'composer.json' },
+          { name: 'phpunit/phpunit', newVersion: '10.3.0', file: 'composer.json' },
         ]
 
         const result = await generateComposerUpdates(updates)
 
         expect(result).toHaveLength(1)
-        expect(result[0].path).toBe('test-updates-composer.json')
+        expect(result[0].path).toBe('composer.json')
         expect(result[0].type).toBe('update')
 
         const updatedContent = result[0].content
         expect(updatedContent).toContain('^10.16.0') // laravel/framework updated
         expect(updatedContent).toContain('^10.3.0') // phpunit/phpunit updated
         expect(updatedContent).toContain('^6.0.0') // symfony/console unchanged
-      }
-      finally {
-        // Clean up test file
-        if (fs.existsSync(testComposerPath)) {
-          fs.unlinkSync(testComposerPath)
-        }
+      } finally {
+        readFileSpy.mockRestore()
       }
     })
 
@@ -271,18 +268,21 @@ describe('Composer Parser', () => {
         },
       }, null, 2)
 
-      // Create a mock composer.json file for this test
+      // Mock fs readFileSync using spyOn
       const fs = await import('node:fs')
-      const path = await import('node:path')
-      const testComposerPath = path.join(process.cwd(), 'test-constraints-composer.json')
+      const readFileSpy = spyOn(fs, 'readFileSync')
 
-      // Write test file
-      fs.writeFileSync(testComposerPath, constraintComposerContent)
+      readFileSpy.mockImplementation(((filePath: any) => {
+        if (String(filePath).endsWith('composer.json')) {
+          return constraintComposerContent
+        }
+        throw new Error(`File not found: ${filePath}`)
+      }) as any)
 
       try {
         const updates = [
-          { name: 'laravel/framework', newVersion: '10.16.0', file: 'test-constraints-composer.json' },
-          { name: 'symfony/console', newVersion: '6.3.0', file: 'test-constraints-composer.json' },
+          { name: 'laravel/framework', newVersion: '10.16.0', file: 'composer.json' },
+          { name: 'symfony/console', newVersion: '6.3.0', file: 'composer.json' },
         ]
 
         const result = await generateComposerUpdates(updates)
@@ -292,12 +292,8 @@ describe('Composer Parser', () => {
 
         expect(updatedContent).toContain('~10.16.0') // Preserves ~ constraint
         expect(updatedContent).toContain('>=6.3.0,<7.0') // Preserves complex constraint
-      }
-      finally {
-        // Clean up test file
-        if (fs.existsSync(testComposerPath)) {
-          fs.unlinkSync(testComposerPath)
-        }
+      } finally {
+        readFileSpy.mockRestore()
       }
     })
 
@@ -316,17 +312,20 @@ describe('Composer Parser', () => {
     })
 
     it('should handle missing packages gracefully', async () => {
-      // Create a mock composer.json file for this test
+      // Mock fs readFileSync using spyOn
       const fs = await import('node:fs')
-      const path = await import('node:path')
-      const testComposerPath = path.join(process.cwd(), 'test-missing-composer.json')
+      const readFileSpy = spyOn(fs, 'readFileSync')
 
-      // Write test file
-      fs.writeFileSync(testComposerPath, mockComposerContent)
+      readFileSpy.mockImplementation(((filePath: any) => {
+        if (String(filePath).endsWith('composer.json')) {
+          return mockComposerContent
+        }
+        throw new Error(`File not found: ${filePath}`)
+      }) as any)
 
       try {
         const updates = [
-          { name: 'non-existent/package', newVersion: '1.0.0', file: 'test-missing-composer.json' },
+          { name: 'non-existent/package', newVersion: '1.0.0', file: 'composer.json' },
         ]
 
         const result = await generateComposerUpdates(updates)
@@ -336,12 +335,8 @@ describe('Composer Parser', () => {
         const updatedContent = result[0].content
         expect(updatedContent).toContain('"laravel/framework": "^10.0.0"') // Original content preserved
         expect(updatedContent).not.toContain('non-existent/package')
-      }
-      finally {
-        // Clean up test file
-        if (fs.existsSync(testComposerPath)) {
-          fs.unlinkSync(testComposerPath)
-        }
+      } finally {
+        readFileSpy.mockRestore()
       }
     })
   })