BREAKING CHANGE: Promisify the cleanup function

index.d.ts

@@ -3,7 +3,7 @@ import { FileOptions, DirOptions, TmpNameOptions } from "tmp";
 
 export interface DirectoryResult {
   path: string;
-  cleanup(): void;
+  cleanup(): Promise<void>;
 }
 
 export interface FileResult extends DirectoryResult {

index.js

@@ -6,7 +6,7 @@ module.exports.fileSync = tmp.fileSync;
 const fileWithOptions = promisify(
   (options, cb) => tmp.file(
     options,
-    (err, path, fd, cleanup) => cb(err, {path, fd, cleanup})
+    (err, path, fd, cleanup) => cb(err, {path, fd, cleanup: promisify(cleanup)})
   )
 );
 module.exports.file = async (options) => fileWithOptions(options);
@@ -16,7 +16,7 @@ module.exports.withFile = async function withFile(fn, options) {
   try {
     return await fn({ path, fd });
   } finally {
-    await promisify(cleanup)();
+    await cleanup();
   }
 };
 
@@ -26,7 +26,7 @@ module.exports.dirSync = tmp.dirSync;
 const dirWithOptions = promisify(
   (options, cb) => tmp.dir(
     options,
-    (err, path, cleanup) => cb(err, {path, cleanup})
+    (err, path, cleanup) => cb(err, {path, cleanup: promisify(cleanup)})
   )
 );
 module.exports.dir = async (options) => dirWithOptions(options);
@@ -36,7 +36,7 @@ module.exports.withDir = async function withDir(fn, options) {
   try {
     return await fn({ path });
   } finally {
-    await promisify(cleanup)();
+    await cleanup();
   }
 };
 

index.test-d.ts

@@ -2,21 +2,25 @@ import { file, withFile, dir, withDir, tmpName } from ".";
 
 async function fileExample() {
   const { path, fd, cleanup } = await file({ discardDescriptor: true });
+  await cleanup();
 
   await withFile(
     async ({ path, fd, cleanup }) => {
       console.log(fd);
+      await cleanup();
     },
     { discardDescriptor: true }
   );
 }
 
 async function dirExample() {
   const { path, cleanup } = await dir({ unsafeCleanup: true });
+  await cleanup();
 
   await withDir(
     async ({ path, cleanup }) => {
       console.log(path);
+      await cleanup();
     },
     { unsafeCleanup: true }
   );

test.js

@@ -23,7 +23,7 @@ async function checkFileResult(result) {
   assert.equal(fs.readFileSync(path), message)
 
   // Check that the cleanup works.
-  await promisify(cleanup)()
+  await cleanup()
   assert.throws(() => fs.statSync(path))
 }
 
@@ -59,7 +59,7 @@ async function checkDirResult(result) {
 
   assert.ok(fs.statSync(path).isDirectory())
 
-  await promisify(cleanup)()
+  await cleanup()
   assert.throws(() => fs.statSync(path))
 }