Make samples tests permissive of lack of data. (#7)

GoogleCloudPlatform · Nov 10, 2022 · f82d8b8 · f82d8b8
1 parent a60d8c0
commit f82d8b8
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/monitoring/snippets/metrics.js b/monitoring/snippets/metrics.js
@@ -429,6 +429,10 @@ function readTimeSeriesReduce(projectId) {
   client
     .listTimeSeries(request)
     .then(results => {
+      if (results[0].length === 0) {
+        console.log('No data');
+        return;
+      }
       const reductions = results[0][0].points;
 
       console.log('Average CPU utilization across all GCE instances:');

diff --git a/monitoring/snippets/system-test/metrics.test.js b/monitoring/snippets/system-test/metrics.test.js
@@ -102,6 +102,7 @@ test(`should read time series data`, async t => {
     },
   });
   const output = await tools.runAsync(`${cmd} read '${filter}'`, cwd);
+  t.true(true); // Do not fail if there is simply no data to return.
   timeSeries.forEach(data => {
     t.true(output.includes(`${data.metric.labels.instance_name}:`));
     data.points.forEach(point => {
@@ -187,7 +188,14 @@ test(`should read time series data reduced`, async t => {
     },
   });
   const output = await tools.runAsync(`${cmd} read-reduce`, cwd);
-  t.true(output.includes(`Average CPU utilization across all GCE instances:`));
-  t.true(output.includes(`  Last 10 min`));
-  t.true(output.includes(`  10-20 min ago`));
+  // Special case: No output.
+  if (output === 'No data') {
+    t.true(output.includes('No data'));
+  } else {
+    t.true(
+      output.includes(`Average CPU utilization across all GCE instances:`)
+    );
+    t.true(output.includes(`  Last 10 min`));
+    t.true(output.includes(`  10-20 min ago`));
+  }
 });