Merge pull request #49 from Automattic/warn_about_not_set_listener

Do not crash if `buildMetricsPreparedAction` null
Automattic · Sep 24, 2024 · bdbaad5 · bdbaad5
2 parents e663dd9 + 6db8f2b
commit bdbaad5
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/...-builds/src/main/java/com/automattic/android/measure/reporters/InMemoryMetricsReporter.kt b/...-builds/src/main/java/com/automattic/android/measure/reporters/InMemoryMetricsReporter.kt
@@ -2,11 +2,12 @@ package com.automattic.android.measure.reporters
 
 import com.automattic.android.measure.InMemoryReport
 import org.gradle.api.Action
+import org.gradle.api.logging.Logging
 import org.gradle.api.provider.Property
 
 object InMemoryMetricsReporter {
 
-    lateinit var buildMetricsPreparedAction: Property<Action<MetricsReport>>
+    var buildMetricsPreparedAction: Property<Action<MetricsReport>>? = null
 
     fun report(
         report: InMemoryReport,
@@ -18,6 +19,11 @@ object InMemoryMetricsReporter {
             override val gradleScanId: String?
                 get() = gradleScanId
         }
-        buildMetricsPreparedAction.get().execute(result)
+        if (buildMetricsPreparedAction == null) {
+            Logging.getLogger(InMemoryMetricsReporter::class.java).warn(
+                "buildMetricsPreparedAction is not set. Metrics will not be reported."
+            )
+        }
+        buildMetricsPreparedAction?.get()?.execute(result)
     }
 }