Do not remove test directory configured with NODE_FB_TEST_TMP_DIR

asfernandes · asfernandes · commit a327ab227b26 · 2026-02-21T12:50:07.000-03:00
diff --git a/packages/node-firebird-driver/src/test/tests.ts b/packages/node-firebird-driver/src/test/tests.ts
@@ -106,7 +106,7 @@ export function runCommonTests(client: Client) {
 
       expect(client.isValid).toBeFalsy();
 
-      if (isLocal()) {
+      if (isLocal() && !testConfig.tmpDir) {
         fs.rmdirSync(testConfig.tmpDir!);
       }
     });
diff --git a/packages/node-firebird-native-api/src/test/test.ts b/packages/node-firebird-native-api/src/test/test.ts
@@ -73,7 +73,7 @@ describe('node-firebird-native-api', () => {
 
     dispatcher.releaseSync();
 
-    if (isLocal()) {
+    if (isLocal() && !testConfig.tmpDir) {
       fs.rmdirSync(testConfig.tmpDir!);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ export function runCommonTests(client: Client) {`
`106`	`106`
`107`	`107`	`expect(client.isValid).toBeFalsy();`
`108`	`108`
`109`		`- if (isLocal()) {`
	`109`	`+ if (isLocal() && !testConfig.tmpDir) {`
`110`	`110`	`fs.rmdirSync(testConfig.tmpDir!);`
`111`	`111`	`}`
`112`	`112`	`});`
Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ describe('node-firebird-native-api', () => {`
`73`	`73`
`74`	`74`	`dispatcher.releaseSync();`
`75`	`75`
`76`		`- if (isLocal()) {`
	`76`	`+ if (isLocal() && !testConfig.tmpDir) {`
`77`	`77`	`fs.rmdirSync(testConfig.tmpDir!);`
`78`	`78`	`}`
`79`	`79`