fix: replace dynamic ESM imports with static imports and fix error type assertions in coverageGaps.spec.ts

Author: scholarsmate

packages/client/tests/specs/coverageGaps.spec.ts

@@ -19,8 +19,10 @@
 
 import {
   createSession,
+  createViewport,
   del,
   destroySession,
+  destroyViewport,
   editSimple,
   EditStats,
   getChangeCount,
@@ -29,6 +31,7 @@ import {
   getSegment,
   getSessionCount,
   getUndoCount,
+  getViewportData,
   insert,
   IOFlags,
   overwrite,
@@ -316,9 +319,6 @@ describe('Coverage Gaps', () => {
 
   describe('Viewport data integrity', () => {
     it('Should have viewport data matching session segment', async () => {
-      const { createViewport, destroyViewport, getViewportData } =
-        await import('@omega-edit/client')
-
       const session_id = await createTestSession(testPort)
       await insert(session_id, 0, Buffer.from('ABCDEFGHIJKLMNOPQRSTUVWXYZ'))
 
@@ -350,9 +350,6 @@ describe('Coverage Gaps', () => {
     })
 
     it('Should have viewport data matching after edits', async () => {
-      const { createViewport, destroyViewport, getViewportData } =
-        await import('@omega-edit/client')
-
       const session_id = await createTestSession(testPort)
       await insert(session_id, 0, Buffer.from('ABCDEFGHIJKLMNOPQRSTUVWXYZ'))
 
@@ -391,7 +388,7 @@ describe('Coverage Gaps', () => {
         await getComputedFileSize(badId)
         expect.fail('Should have thrown an error')
       } catch (err: any) {
-        expect(err).to.be.a('string')
+        expect(err).to.be.an.instanceOf(Error)
       }
     })
 
@@ -402,7 +399,7 @@ describe('Coverage Gaps', () => {
         await insert(badId, 0, Buffer.from('data'))
         expect.fail('Should have thrown an error')
       } catch (err: any) {
-        expect(err).to.be.a('string')
+        expect(err).to.be.an.instanceOf(Error)
       }
     })
 
@@ -413,7 +410,7 @@ describe('Coverage Gaps', () => {
         await del(badId, 0, 5)
         expect.fail('Should have thrown an error')
       } catch (err: any) {
-        expect(err).to.be.a('string')
+        expect(err).to.be.an.instanceOf(Error)
       }
     })
 
@@ -450,7 +447,7 @@ describe('Coverage Gaps', () => {
         await destroySession('does-not-exist-99999')
         expect.fail('Should have thrown an error')
       } catch (err: any) {
-        expect(err).to.be.a('string')
+        expect(err).to.be.an.instanceOf(Error)
       }
     })
   })