chore: linter warnings

Author: atilafassina

packages/appkit/src/connectors/files/tests/client.test.ts

@@ -671,7 +671,7 @@ describe("FilesConnector", () => {
       expect(result.isText).toBe(true);
       expect(result.isImage).toBe(false);
       expect(result.textPreview).not.toBeNull();
-      expect(result.textPreview!.length).toBeLessThanOrEqual(1024);
+      expect(result.textPreview?.length).toBeLessThanOrEqual(1024);
     });
 
     test("text/html files are treated as text", async () => {

packages/appkit/src/plugins/files/tests/plugin.test.ts

@@ -2,48 +2,46 @@ import { mockServiceContext, setupDatabricksEnv } from "@tools/test-helpers";
 import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
 import { ServiceContext } from "../../../context/service-context";
 import { FilesPlugin, files } from "../plugin";
-import { streamFromString } from "./utils";
-
-const { mockFilesApi, mockClient, MockApiError, mockCacheInstance } =
-  vi.hoisted(() => {
-    const mockFilesApi = {
-      listDirectoryContents: vi.fn(),
-      download: vi.fn(),
-      getMetadata: vi.fn(),
-      upload: vi.fn(),
-      createDirectory: vi.fn(),
-      delete: vi.fn(),
-    };
-
-    const mockClient = {
-      files: mockFilesApi,
-      config: {
-        host: "https://test.databricks.com",
-        authenticate: vi.fn(),
-      },
-    };
-
-    class MockApiError extends Error {
-      statusCode: number;
-      constructor(message: string, statusCode: number) {
-        super(message);
-        this.name = "ApiError";
-        this.statusCode = statusCode;
-      }
-    }
-
-    const mockCacheInstance = {
-      get: vi.fn(),
-      set: vi.fn(),
-      delete: vi.fn(),
-      getOrExecute: vi.fn(async (_key: unknown[], fn: () => Promise<unknown>) =>
-        fn(),
-      ),
-      generateKey: vi.fn(),
-    };
 
-    return { mockFilesApi, mockClient, MockApiError, mockCacheInstance };
-  });
+const { mockClient, MockApiError, mockCacheInstance } = vi.hoisted(() => {
+  const mockFilesApi = {
+    listDirectoryContents: vi.fn(),
+    download: vi.fn(),
+    getMetadata: vi.fn(),
+    upload: vi.fn(),
+    createDirectory: vi.fn(),
+    delete: vi.fn(),
+  };
+
+  const mockClient = {
+    files: mockFilesApi,
+    config: {
+      host: "https://test.databricks.com",
+      authenticate: vi.fn(),
+    },
+  };
+
+  class MockApiError extends Error {
+    statusCode: number;
+    constructor(message: string, statusCode: number) {
+      super(message);
+      this.name = "ApiError";
+      this.statusCode = statusCode;
+    }
+  }
+
+  const mockCacheInstance = {
+    get: vi.fn(),
+    set: vi.fn(),
+    delete: vi.fn(),
+    getOrExecute: vi.fn(async (_key: unknown[], fn: () => Promise<unknown>) =>
+      fn(),
+    ),
+    generateKey: vi.fn(),
+  };
+
+  return { mockFilesApi, mockClient, MockApiError, mockCacheInstance };
+});
 
 vi.mock("@databricks/sdk-experimental", () => ({
   WorkspaceClient: vi.fn(() => mockClient),