commit f721b1372f0acb55e066dc02ef14ea75c977d9a7
author Rui Qiu <ruiqiu@google.com> Fri Sep 10 12:41:09 2021 -0700
committer Rui Qiu <ruiqiu@google.com> Mon Sep 13 12:04:10 2021 -0700
tree 8f5dfc74a48f0c460f4c6622f0fc81d4ed3e87f4
parent bb2cea3338052b62b2d40e8d237791bf75ac1cee

Implement delete methods in MetricsConfigStore and ResultStore

This CL also uses the deletion methods in CarTelemetryService

Bug: 198797367
Bug: 198784116
Test: m RUN_ERROR_PRONE=true CarService
Test: m RUN_ERROR_PRONE=true CarServiceUnitTest
Test: atest CarServiceUnitTest:CarTelemetryServiceTest
Test: atest CarServiceUnitTest:MetricsConfigStoreTest
Test: atest CarServiceUnitTest:ResultStoreTest

Change-Id: I91c575248aeb1abc88196b654f0683e2fa48570f

service/src/com/android/car/telemetry/MetricsConfigStore.java

diff --git a/service/src/com/android/car/telemetry/MetricsConfigStore.java b/service/src/com/android/car/telemetry/MetricsConfigStore.java
index 9802e0e..7a5512a 100644
--- a/service/src/com/android/car/telemetry/MetricsConfigStore.java
+++ b/service/src/com/android/car/telemetry/MetricsConfigStore.java
@@ -89,6 +89,7 @@
         } else if (currentVersion == metricsConfig.getVersion()) {
             return ERROR_METRICS_CONFIG_ALREADY_EXISTS;
         }
+        mActiveConfigs.put(metricsConfig.getName(), metricsConfig);
         mNameVersionMap.put(metricsConfig.getName(), metricsConfig.getVersion());
         try {
             Files.write(
@@ -108,7 +109,10 @@
         return new File(mConfigDirectory, metricsConfigName).delete();
     }
 
-    void deleteAllMetricsConfig() {
-        // TODO(b/198784116): implement
+    void deleteAllMetricsConfigs() {
+        mActiveConfigs.clear();
+        for (File file : mConfigDirectory.listFiles()) {
+            file.delete();
+        }
     }
 }