@uploadista/core 0.0.13-beta.5 → 0.0.14

package/src/testing/mock-zip-plugin.ts

@@ -0,0 +1,41 @@
+import { Effect, Layer } from "effect";
+import { ZipPlugin } from "../flow";
+
+/**
+ * Mock ZipPlugin implementation for testing.
+ *
+ * Provides a simple in-memory implementation that creates mock zip data
+ * by serializing file metadata as JSON.
+ *
+ * @example
+ * ```typescript
+ * import { TestZipPlugin } from "@uploadista/core/testing";
+ *
+ * const program = Effect.gen(function* () {
+ *   const zipPlugin = yield* ZipPlugin;
+ *   const zipData = yield* zipPlugin.zip(inputs, options);
+ *   return zipData;
+ * }).pipe(Effect.provide(TestZipPlugin));
+ * ```
+ */
+export const TestZipPlugin = Layer.succeed(
+  ZipPlugin,
+  ZipPlugin.of({
+    zip: (inputs, options) =>
+      Effect.gen(function* () {
+        // Create mock zip data
+        const files = inputs.map((input) => ({
+          id: input.id,
+          size: input.data.byteLength,
+        }));
+
+        const zipContent = JSON.stringify({
+          zipName: options.zipName,
+          includeMetadata: options.includeMetadata,
+          files,
+        });
+
+        return new TextEncoder().encode(zipContent);
+      }),
+  }),
+);

package/src/types/data-store.ts

@@ -158,7 +158,7 @@ export type DataStore<TData = unknown> = {
       onProgress?: (chunkSize: number) => void;
     },
   ) => Effect.Effect<number, UploadistaError>;
-  readonly deleteExpired?: Effect.Effect<number, UploadistaError>;
+  readonly deleteExpired?: () => Effect.Effect<number, UploadistaError>;
   readonly getCapabilities: () => DataStoreCapabilities;
   readonly validateUploadStrategy: (
     strategy: UploadStrategy,

package/{src/errors/__tests__ → tests/errors}/uploadista-error.test.ts

@@ -1,4 +1,4 @@
-import { Effect } from "effect";
+import { Effect, Exit } from "effect";
 import { describe, expect, it } from "vitest";
 import {
   ERROR_CATALOG,
@@ -6,7 +6,7 @@ import {
   isUploadistaError,
   UploadistaError,
   type UploadistaErrorCode,
-} from "../uploadista-error";
+} from "../../src/errors";
 
 describe("ERROR_CATALOG", () => {
   it("should contain all error codes with status and body", () => {
@@ -76,7 +76,6 @@ describe("UploadistaError", () => {
     expect(error.status).toBe(404);
     expect(error.status_code).toBe(404); // legacy alias
     expect(error.body).toBe("File not found");
-    expect(error.message).toBe("File not found");
     expect(error.details).toEqual({ fileId: "123" });
     expect((error as { cause?: unknown }).cause).toBeInstanceOf(Error);
   });
@@ -173,12 +172,14 @@ describe("UploadistaError", () => {
   });
 
   describe("toFailure", () => {
-    it("should convert error to failure result", () => {
+    it("should convert error to failure result", async () => {
       const error = UploadistaError.fromCode("FILE_NOT_FOUND");
-      const result = error.toEffect();
+      const effect = error.toEffect();
 
-      expect(Effect.isFailure(result)).toBe(true);
-      expect(Effect.isSuccess(result)).toBe(false);
+      const result = await Effect.runPromiseExit(effect);
+
+      expect(Exit.isFailure(result)).toBe(true);
+      expect(Exit.isSuccess(result)).toBe(false);
     });
   });
 });
@@ -216,25 +217,27 @@ describe("isUploadistaError", () => {
 });
 
 describe("httpFailure", () => {
-  it("should create failure result from error code", () => {
-    const result = httpFailure("FILE_NOT_FOUND");
+  it("should create failure result from error code", async () => {
+    const effect = httpFailure("FILE_NOT_FOUND");
+    const result = await Effect.runPromiseExit(effect);
 
-    expect(Effect.isFailure(result)).toBe(true);
-    expect(Effect.isSuccess(result)).toBe(false);
+    expect(Exit.isFailure(result)).toBe(true);
+    expect(Exit.isSuccess(result)).toBe(false);
   });
 
-  it("should allow overriding error properties", () => {
-    const result = httpFailure("UNKNOWN_ERROR", {
+  it("should allow overriding error properties", async () => {
+    const effect = httpFailure("UNKNOWN_ERROR", {
       status: 503,
       body: "Service unavailable",
       details: { retryAfter: 60 },
     });
+    const result = await Effect.runPromiseExit(effect);
 
-    expect(Effect.isFailure(result)).toBe(true);
-    expect(Effect.isSuccess(result)).toBe(false);
+    expect(Exit.isFailure(result)).toBe(true);
+    expect(Exit.isSuccess(result)).toBe(false);
   });
 
-  it("should work with all error codes", () => {
+  it("should work with all error codes", async () => {
     const errorCodes: UploadistaErrorCode[] = [
       "MISSING_OFFSET",
       "FLOW_NODE_ERROR",
@@ -242,9 +245,10 @@ describe("httpFailure", () => {
     ];
 
     for (const code of errorCodes) {
-      const result = httpFailure(code);
-      expect(Effect.isFailure(result)).toBe(true);
-      expect(Effect.isSuccess(result)).toBe(false);
+      const effect = httpFailure(code);
+      const result = await Effect.runPromiseExit(effect);
+      expect(Exit.isFailure(result)).toBe(true);
+      expect(Exit.isSuccess(result)).toBe(false);
     }
   });
 });

package/{src → tests}/flow/edge.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from "vitest";
-import { createFlowEdge, type FlowEdge } from "./edge";
+import { createFlowEdge, type FlowEdge } from "../../src/flow/edge";
 
 describe("FlowEdge", () => {
   describe("createFlowEdge", () => {