Minor improvement to logging in post-action

- Ensure that "Caching Gradle state" group always has 1 message - Only print cache report when entries were restored or saved
gradle · Nov 5, 2021 · 4137be6 · 4137be6
1 parent 4e89983
commit 4137be6
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 6 deletions.
diff --git a/dist/main/index.js b/dist/main/index.js
diff --git a/dist/main/index.js.map b/dist/main/index.js.map
diff --git a/dist/post/index.js b/dist/post/index.js
diff --git a/dist/post/index.js.map b/dist/post/index.js.map
diff --git a/src/cache-base.ts b/src/cache-base.ts
@@ -185,15 +185,15 @@ export abstract class AbstractCache {
 
     async save(listener: CacheListener): Promise<void> {
         if (!this.cacheOutputExists()) {
-            this.debug(`No ${this.cacheDescription} to cache.`)
+            core.info(`No ${this.cacheDescription} to cache.`)
             return
         }
 
         const cacheKey = core.getState(this.cacheKeyStateKey)
         const cacheResult = core.getState(this.cacheResultStateKey)
 
         if (!cacheKey) {
-            this.debug(`${this.cacheDescription} existed prior to cache restore. Not saving.`)
+            core.info(`${this.cacheDescription} existed prior to cache restore. Not saving.`)
             return
         }
 

diff --git a/src/caches.ts b/src/caches.ts
@@ -54,6 +54,10 @@ export async function save(): Promise<void> {
 }
 
 function logCachingReport(listener: CacheListener): void {
+    if (listener.cacheEntries.length === 0) {
+        return
+    }
+
     core.info(`---------- Caching Summary -------------
 Restored Entries Count: ${getCount(listener.cacheEntries, e => e.restoredSize)}
                   Size: ${getSum(listener.cacheEntries, e => e.restoredSize)}