test: fix tests

negezor · Mar 2, 2024 · 73e7cde · 73e7cde
1 parent 0a315af
commit 73e7cde
Show file tree

Hide file tree

Showing 3 changed files with 58 additions and 54 deletions.
diff --git a/test/composer.test.ts b/test/composer.test.ts
@@ -100,6 +100,7 @@ describe('Composer', (): void => {
             await next();
         });
 
+        // eslint-disable-next-line @typescript-eslint/require-await
         composer.use(async (): Promise<never> => {
             throw new Error();
         });
@@ -119,7 +120,7 @@ describe('Composer', (): void => {
 
     it('should only accept middleware as functions', (): void => {
         try {
-            // @ts-ignore
+            // @ts-expect-error cause test
             (new Composer()).use(null);
 
             throw new Error('Middleware must be composed of functions');
@@ -129,10 +130,11 @@ describe('Composer', (): void => {
     });
 
     it('composer should be cloned', async (): Promise<void> => {
+        // eslint-disable-next-line @typescript-eslint/consistent-type-definitions
         type CloneContext = {
             baseValue?: boolean;
             value: 'first' | 'second' | 'default';
-        };
+        }
 
         const baseComposer = new Composer<CloneContext>();
 
@@ -166,17 +168,17 @@ describe('Composer', (): void => {
 
         expect(baseContext).toMatchObject({
             baseValue: true,
-            value: 'default'
+            value: 'default',
         });
 
         expect(firstContext).toMatchObject({
             baseValue: true,
-            value: 'first'
+            value: 'first',
         });
 
         expect(secondContext).toMatchObject({
             baseValue: true,
-            value: 'second'
+            value: 'second',
         });
     });
 
@@ -229,9 +231,10 @@ describe('Composer', (): void => {
 
         try {
             await middleware({}, noopNext);
+            // @ts-expect-error cause test
         } catch ({ message }) {
             expect(message).toEqual(
-                expect.stringMatching('multiple times')
+                expect.stringMatching('multiple times'),
             );
 
             return;

diff --git a/test/middleware.test.ts b/test/middleware.test.ts
@@ -44,7 +44,7 @@ describe('compose', (): void => {
                 await delay(1);
 
                 out.push(4);
-            }
+            },
         ]);
 
         await middleware(out, noopNext);
@@ -70,7 +70,7 @@ describe('compose', (): void => {
                 await next();
 
                 expect(ctx).toBe(context);
-            }
+            },
         ]);
 
         await middleware(context, noopNext);
@@ -92,7 +92,7 @@ describe('compose', (): void => {
             },
             async (): Promise<never> => {
                 throw new Error();
-            }
+            },
         ]);
 
         try {
@@ -108,7 +108,7 @@ describe('compose', (): void => {
 
     it('should only accept middleware as functions', (): void => {
         try {
-            // @ts-ignore
+            // @ts-expect-error cause test
             compose([null]);
 
             throw new Error('Middleware must be composed of functions');
@@ -128,14 +128,15 @@ describe('compose', (): void => {
             },
             async (ctx, next): Promise<void> => {
                 await next();
-            }
+            },
         ]);
 
         try {
             await middleware({}, noopNext);
+            // @ts-expect-error cause error
         } catch ({ message }) {
             expect(message).toEqual(
-                expect.stringMatching('multiple times')
+                expect.stringMatching('multiple times'),
             );
 
             return;