fix(test): remove unused @ts-expect-error directives and fix toEndWith matcher

Author: jdalton

test/constants/platform.test.ts

@@ -203,7 +203,7 @@ describe('constants/platform', () => {
     it('should support platform-conditional binary extensions', () => {
       const binaryName = WIN32 ? 'app.exe' : 'app'
       if (WIN32) {
-        expect(binaryName).toEndWith('.exe')
+        expect(binaryName.endsWith('.exe')).toBe(true)
       } else {
         expect(binaryName).not.toContain('.')
       }

test/json.test.ts

@@ -679,7 +679,6 @@ describe('json', () => {
       it('should handle array-like objects with non-number first element', () => {
         // Tests line 166-171: length > 0 but obj[0] is not a number
         expect(() => {
-          // @ts-expect-error - testing runtime behavior
           jsonParse({
             length: 1,
             0: 'not a number',
@@ -692,7 +691,6 @@ describe('json', () => {
       it('should handle objects without proper constructor', () => {
         // Tests line 174-177: constructor.isBuffer checks
         expect(() => {
-          // @ts-expect-error - testing runtime behavior
           jsonParse({
             length: 0,
             copy: () => {},
@@ -702,7 +700,6 @@ describe('json', () => {
         }).toThrow()
 
         expect(() => {
-          // @ts-expect-error - testing runtime behavior
           jsonParse({
             length: 0,
             copy: () => {},
@@ -721,7 +718,6 @@ describe('json', () => {
         expect(isJsonPrimitive(0)).toBe(true)
         expect(isJsonPrimitive(false)).toBe(true)
         expect(isJsonPrimitive('')).toBe(true)
-        // @ts-expect-error - testing runtime behavior
         expect(isJsonPrimitive(Number.NaN)).toBe(true) // NaN is a number
       })
 

test/strings.test.ts

@@ -618,11 +618,8 @@ describe('strings', () => {
     describe('stringWidth edge cases', () => {
       it('should return 0 for non-string input', () => {
         // Tests line 546-547: typeof check and !text.length
-        // @ts-expect-error - testing runtime behavior
         expect(stringWidth(null)).toBe(0)
-        // @ts-expect-error - testing runtime behavior
         expect(stringWidth(undefined)).toBe(0)
-        // @ts-expect-error - testing runtime behavior
         expect(stringWidth(123)).toBe(0)
       })
 

test/utils/get-ipc.test.ts

@@ -166,7 +166,6 @@ describe('utils/get-ipc', () => {
       const ipc = await getIpc()
 
       expect(() => {
-        // @ts-expect-error - testing immutability
         ipc.SOCKET_CLI_FIX = 'modified'
       }).toThrow()
     })
@@ -175,7 +174,6 @@ describe('utils/get-ipc', () => {
       const ipc = await getIpc()
 
       expect(() => {
-        // @ts-expect-error - testing immutability
         ipc.NEW_PROPERTY = 'value'
       }).toThrow()
     })
@@ -186,7 +184,6 @@ describe('utils/get-ipc', () => {
 
       if (keys.length > 0) {
         expect(() => {
-          // @ts-expect-error - testing immutability
           delete ipc[keys[0]]
         }).toThrow()
       }