Fix caching using update key

__tests__/actionUtils.test.ts

@@ -194,13 +194,3 @@ test("getInputAsArray handles empty lines correctly", () => {
     testUtils.setInput("foo", "\n\nbar\n\nbaz\n\n");
     expect(actionUtils.getInputAsArray("foo")).toEqual(["bar", "baz"]);
 });
-
-test("getInputAsBoolean returns true if the value is set to 'true'", () => {
-    testUtils.setInput("foo", "true");
-    expect(actionUtils.getInputAsBoolean("foo")).toEqual(true);
-});
-
-test("getInputAsBoolean returns false if the value is set to anything else", () => {
-    testUtils.setInput("foo", "false");
-    expect(actionUtils.getInputAsBoolean("foo")).toEqual(false);
-});

__tests__/save.test.ts

@@ -118,7 +118,8 @@ test("save with exact match returns early", async () => {
     expect(failedMock).toHaveBeenCalledTimes(0);
 });
 
-test("save with exact match updates when configured", async () => {
+test("save with exact match and updates enabled updates the cache", async () => {
+    const infoMock = jest.spyOn(core, "info");
     const failedMock = jest.spyOn(core, "setFailed");
 
     const primaryKey = "Linux-node-bb828da54c148048dd17899ba9fda624811cfb43";
@@ -147,9 +148,11 @@ test("save with exact match updates when configured", async () => {
 
     await run();
 
+    expect(infoMock).toHaveBeenCalledWith(
+        `Cache hit occurred on the primary key ${primaryKey}, but updates were enabled, so updating cache.`
+    );
     expect(saveCacheMock).toHaveBeenCalledTimes(1);
     expect(saveCacheMock).toHaveBeenCalledWith([inputPath], primaryKey);
-
     expect(failedMock).toHaveBeenCalledTimes(0);
 });
 

src/save.ts

@@ -16,21 +16,25 @@ async function run(): Promise<void> {
         }
 
         const state = utils.getCacheState();
-        // Inputs are re-evaluated before the post action, so we want the original key used for restore
+
+        // Inputs are re-evaluted before the post action, so we want the original key used for restore
         const primaryKey = core.getState(State.CachePrimaryKey);
         if (!primaryKey) {
             utils.logWarning(`Error retrieving key from state.`);
             return;
         }
 
-        if (
+        if (utils.isExactKeyMatch(primaryKey, state)) {
-            utils.isExactKeyMatch(primaryKey, state) &&
+          if (core.getInput(Inputs.Update) === "true") {
-            !utils.getInputAsBoolean(Inputs.Update)
+              core.info(
-        ) {
+                  `Cache hit occurred on the primary key ${primaryKey}, but updates were enabled, so updating cache.`
-            core.info(
+              );
-                `Cache hit occurred on the primary key ${primaryKey}, not saving cache.`
+          } else {
-            );
+              core.info(
-            return;
+                  `Cache hit occurred on the primary key ${primaryKey}, not saving cache.`
+              );
+              return;
+          }
         }
 
         const cachePaths = utils.getInputAsArray(Inputs.Path, {

src/utils/actionUtils.ts

@@ -56,10 +56,3 @@ export function getInputAsArray(
         .map(s => s.trim())
         .filter(x => x !== "");
 }
-
-export function getInputAsBoolean(
-    name: string,
-    options?: core.InputOptions
-): boolean {
-    return core.getInput(name, options) === "true";
-}

src/utils/testUtils.ts

@@ -13,11 +13,13 @@ interface CacheInput {
     path: string;
     key: string;
     restoreKeys?: string[];
+    update: string;
 }
 
 export function setInputs(input: CacheInput): void {
     setInput(Inputs.Path, input.path);
     setInput(Inputs.Key, input.key);
+    setInput(Inputs.Update, input.update);
     input.restoreKeys &&
         setInput(Inputs.RestoreKeys, input.restoreKeys.join("\n"));
 }
@@ -25,5 +27,6 @@ export function setInputs(input: CacheInput): void {
 export function clearInputs(): void {
     delete process.env[getInputName(Inputs.Path)];
     delete process.env[getInputName(Inputs.Key)];
+    delete process.env[getInputName(Inputs.Update)];
     delete process.env[getInputName(Inputs.RestoreKeys)];
 }