disallow setting version when prefix is set (useblacksmith#13)

useblacksmith · Nov 26, 2024 · c27f005 · c27f005
1 parent 32651f9
commit c27f005
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 0 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/src/__tests__/main.test.ts b/src/__tests__/main.test.ts
@@ -154,6 +154,17 @@ describe("deleteCache", () => {
     ).rejects.toThrow("Cache key cannot be empty unless prefix is true");
   });
 
+  it("should fail if version is specified with prefix", async () => {
+    await expect(
+      deleteCache({
+        cacheKey: "npm-cache",
+        cacheVersion: "v1.0",
+        prefix: true,
+        ...defaultParams,
+      })
+    ).rejects.toThrow("Cannot specify version when using prefix");
+  });
+
   it("should handle 404 response", async () => {
     mockedFetch.mockResolvedValue({
       ok: false,

diff --git a/src/main.ts b/src/main.ts
@@ -29,6 +29,9 @@ export async function deleteCache({
   if (cacheVersion && !cacheKey) {
     throw new Error("Cannot specify version when using empty key");
   }
+  if (prefix && cacheVersion) {
+    throw new Error("Cannot specify version when using prefix");
+  }
 
   const resource = cacheVersion ? `${cacheKey}/${cacheVersion}` : cacheKey;
   const url = `${baseUrl}/caches/${resource}`;