Fix linting errors

Author: evgenyneu

.eslintrc.cjs

@@ -26,6 +26,13 @@ module.exports = {
         },
       },
     },
+    {
+      // Apply to all test files. Proper type checking in tests with mocks can be tedious and counterproductive.
+      files: ['**/*.test.ts', '**/*.spec.ts'],  
+      rules: {
+        '@typescript-eslint/no-explicit-any': 'off'
+      }
+    }
   ],
   env: {
     browser: true,

src/lib/emails/welcome_email_html.svelte

@@ -193,7 +193,6 @@
                                   style="font-family: Helvetica, sans-serif; font-size: 16px; vertical-align: top; border-radius: 4px; text-align: center; background-color: #0867ec;"
                                   valign="top"
                                   align="center"
-                                  bgcolor="#0867ec"
                                 >
                                   <a
                                     href="https://github.com/CriticalMoments/CMSaasStarter"

src/routes/(admin)/account/api/+page.server.ts

@@ -282,6 +282,7 @@ export const actions = {
         company_name: companyName,
         website: website,
         updated_at: new Date(),
+        unsubscribed: priorProfile?.unsubscribed ?? false,
       })
       .select()
 

src/routes/(admin)/account/api/page.server.test.ts

@@ -33,8 +33,11 @@ describe("toggleEmailSubscription", () => {
 
     await expect(
       actions.toggleEmailSubscription({
-        locals: { supabase: mockSupabase, safeGetSession: mockSafeGetSession },
-      }),
+        locals: {
+          supabase: mockSupabase,
+          safeGetSession: mockSafeGetSession,
+        },
+      } as any),
     ).rejects.toThrow("Redirect")
 
     expect(redirect).toHaveBeenCalledWith(303, "/login")
@@ -56,7 +59,7 @@ describe("toggleEmailSubscription", () => {
 
     const result = await actions.toggleEmailSubscription({
       locals: { supabase: mockSupabase, safeGetSession: mockSafeGetSession },
-    })
+    } as any)
 
     expect(mockSupabase.from).toHaveBeenCalledWith("profiles")
     expect(mockSupabase.select).toHaveBeenCalledWith("unsubscribed")
@@ -83,7 +86,7 @@ describe("toggleEmailSubscription", () => {
 
     const result = await actions.toggleEmailSubscription({
       locals: { supabase: mockSupabase, safeGetSession: mockSafeGetSession },
-    })
+    } as any)
 
     expect(mockSupabase.from).toHaveBeenCalledWith("profiles")
     expect(mockSupabase.select).toHaveBeenCalledWith("unsubscribed")
@@ -110,7 +113,7 @@ describe("toggleEmailSubscription", () => {
 
     await actions.toggleEmailSubscription({
       locals: { supabase: mockSupabase, safeGetSession: mockSafeGetSession },
-    })
+    } as any)
 
     // Check if fail was called with the correct arguments
     expect(fail).toHaveBeenCalledWith(500, {

tsconfig.json

@@ -10,7 +10,7 @@
     "sourceMap": true,
     "strict": true,
     "types": ["vitest/globals"] // allows to skip import of test functions like `describe`, `it`, `expect`, etc.
-  },
+  }
   // Path aliases are handled by https://kit.svelte.dev/docs/configuration#alias and https://kit.svelte.dev/docs/configuration#files
   //
   // If you want to overwrite includes/excludes, make sure to copy over the relevant includes/excludes

vite.config.ts

@@ -5,6 +5,6 @@ export default defineConfig({
   plugins: [sveltekit()],
   test: {
     include: ["src/**/*.{test,spec}.{js,ts}"],
-    globals: true /// allows to skip import of test functions like `describe`, `it`, `expect`, etc.
+    globals: true, /// allows to skip import of test functions like `describe`, `it`, `expect`, etc.
   },
 })