chore: offer way to get cache names instead of CacheInfos from ListCa…

…chesResponse (#79) * chore: offer way to get cache names instead of CacheInfos from ListCachesResponse * dart format
momentohq · Jan 8, 2024 · 6de1b98 · 6de1b98
1 parent 822f96b
commit 6de1b98
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lib/src/messages/responses/cache/control/list_caches_response.dart b/lib/src/messages/responses/cache/control/list_caches_response.dart
@@ -33,6 +33,8 @@ class ListCachesSuccess implements ListCachesResponse {
   String description() {
     return "[ListCachesSuccess] length of caches list: ${caches.length}";
   }
+
+  List<String> get cacheNames => caches.map((cache) => cache.name).toList();
 }
 
 /// Indicates that an error occurred during the list caches request.

diff --git a/test/src/cache/cache_test.dart b/test/src/cache/cache_test.dart
@@ -62,7 +62,7 @@ void main() {
       final listResp = await cacheClient.listCaches();
       switch (listResp) {
         case ListCachesSuccess():
-          final cacheNames = listResp.caches.map((cacheInfo) => cacheInfo.name);
+          final cacheNames = listResp.cacheNames;
           expect(cacheNames.contains(newTestCacheName), true,
               reason: "new cache should be in list of caches");
           expect(cacheNames.contains(integrationTestCacheName), true,
@@ -87,8 +87,7 @@ void main() {
       final listResp2 = await cacheClient.listCaches();
       switch (listResp2) {
         case ListCachesSuccess():
-          final cacheNames =
-              listResp2.caches.map((cacheInfo) => cacheInfo.name);
+          final cacheNames = listResp2.cacheNames;
           expect(cacheNames.contains(newTestCacheName), false,
               reason: "new cache should no longer be in list of caches");
           expect(cacheNames.contains(integrationTestCacheName), true,