test: refactors system test deleteDatasets() (#754)

* tests(system): adds check for creationTime

* refactor: replaces parsInt with Number

* test: refactor system test before hook

* test(refactor): checks dataset existence before attempting deletion

* chore: fix lint

* refactor: deleteDatasets

Author: steffnay

handwritten/bigquery/system-test/bigquery.ts

@@ -1698,16 +1698,18 @@ describe('BigQuery', () => {
       filter: `labels.${GCLOUD_TESTS_PREFIX}`,
     });
 
-    const deleteDatasetPromises = datasets
-      .filter(async dataset => {
-        const [metadata] = await dataset.getMetadata();
-        const creationTime = Number(metadata.creationTime);
-        return creationTime && isResourceStale(creationTime);
-      })
-      .map(dataset => {
-        return dataset.delete({force: true});
-      });
-
-    await Promise.all(deleteDatasetPromises);
+    for (const dataset of datasets) {
+      const [metadata] = await dataset.getMetadata();
+      const creationTime = Number(metadata.creationTime);
+
+      if (isResourceStale(creationTime)) {
+        try {
+          await dataset.delete({force: true});
+        } catch (e) {
+          console.log(`dataset(${dataset.id}).delete() failed`);
+          console.log(e);
+        }
+      }
+    }
   }
 });