test: do not force a directory delete for performance tests (#2163)

* test: do not force a directory delete for performance tests * remove unused import
googleapis · Mar 16, 2023 · 6f58201 · 6f58201
1 parent 0501127
commit 6f58201
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 4 deletions.
diff --git a/internal-tooling/performPerformanceTest.ts b/internal-tooling/performPerformanceTest.ts
@@ -33,7 +33,6 @@ import {
   TestResult,
 } from './performanceUtils';
 import {Bucket} from '../src';
-import {rmSync} from 'fs';
 
 const TEST_NAME_STRING = 'nodejs-perf-metrics';
 const DEFAULT_NUMBER_OF_WRITES = 1;
@@ -119,7 +118,6 @@ async function performRangedReadTest(): Promise<TestResult[]> {
     iterationResult.elapsedTimeUs = Math.round((end - start) * 1000);
   }
 
-  rmSync(TEST_NAME_STRING, {recursive: true, force: true});
   await file.delete();
   results.push(iterationResult);
   return results;
@@ -199,7 +197,6 @@ async function performWriteReadTest(): Promise<TestResult[]> {
     iterationResult.elapsedTimeUs = Math.round((end - start) * 1000);
   }
 
-  rmSync(TEST_NAME_STRING, {recursive: true, force: true});
   await file.delete();
   results.push(iterationResult);
   return results;

diff --git a/internal-tooling/performTransferManagerTest.ts b/internal-tooling/performTransferManagerTest.ts
@@ -234,7 +234,6 @@ async function performDownloadFileInChunksTest(): Promise<TestResult> {
     workers: argv.workers,
   };
 
-  rmSync(TEST_NAME_STRING, {recursive: true, force: true});
   return result;
 }
-Original file line number
+Diff line change
@@ Expand Up @@
         workers: argv.workers,
       };
-      rmSync(TEST_NAME_STRING, {recursive: true, force: true});
       return result;
     }
@@ Expand Down @@