Added temp for handling temp files.

Updated tests.

Author: neogeek

packages/doxdox-parser-jsdoc/.gitignore

@@ -1,5 +1,7 @@
 node_modules/
 
+cache/
+
 coverage/
 
 dist/

packages/doxdox-parser-jsdoc/package.json

@@ -10,7 +10,8 @@
   },
   "dependencies": {
     "jsdoc": "3.6.10",
-    "spawn-please": "1.0.0"
+    "spawn-please": "1.0.0",
+    "temp": "0.9.4"
   },
   "peerDependencies": {
     "doxdox-core": "4.0.0-preview.8"
@@ -23,6 +24,7 @@
   "devDependencies": {
     "@types/jest": "27.4.0",
     "@types/node": "17.0.19",
+    "@types/temp": "0.9.1",
     "jest": "27.5.1",
     "ts-jest": "27.1.3",
     "typescript": "4.5.5"

packages/doxdox-parser-jsdoc/src/index.test.ts

@@ -4,8 +4,7 @@ describe('jsdoc parser', () => {
     it('parse example jsdoc header', async () => {
         await expect(
             parseString(
-                'cache/',
-                'utils.js',
+                'lib/utils/index.js',
                 `/**
 * Finds file in path.
 *
@@ -33,7 +32,7 @@ const getRootDirPath = (url) => {}`
             )
         ).resolves.toEqual(
             expect.objectContaining({
-                path: 'utils.js',
+                path: 'lib/utils/index.js',
                 methods: expect.arrayContaining([
                     expect.objectContaining({
                         fullName: 'findFileInPath(input, fileName)',
@@ -55,7 +54,7 @@ const getRootDirPath = (url) => {}`
                                 types: ['Promise.<(string|null)>']
                             })
                         ]),
-                        slug: 'utils-js-findfileinpath'
+                        slug: 'lib-utils-index-js-findfileinpath'
                     }),
                     expect.objectContaining({
                         fullName: 'getRootDirPath(url)',
@@ -73,14 +72,14 @@ const getRootDirPath = (url) => {}`
                                 types: ['string']
                             })
                         ]),
-                        slug: 'utils-js-getrootdirpath'
+                        slug: 'lib-utils-index-js-getrootdirpath'
                     })
                 ])
             })
         );
     });
     it('parse empty string', async () => {
-        await expect(parseString('cache/', 'test.js', '')).resolves.toEqual(
+        await expect(parseString('test.js', '')).resolves.toEqual(
             expect.objectContaining({
                 path: 'test.js',
                 methods: []

packages/doxdox-parser-jsdoc/src/index.ts

@@ -6,6 +6,8 @@ import { dirname, join } from 'path';
 
 import spawn from 'spawn-please';
 
+import temp from 'temp';
+
 import { findParentNodeModules, sanitizePath, slugify } from 'doxdox-core';
 
 import { File } from 'doxdox-core';
@@ -97,10 +99,14 @@ const parser = async (cwd: string, path: string): Promise<File> => {
 };
 
 export const parseString = async (
-    tempDir: string,
     path: string,
-    content: string
+    content: string,
+    cacheDir = './cache'
 ): Promise<File> => {
+    temp.track();
+
+    const tempDir = await temp.mkdir({ prefix: 'doxdox-', dir: cacheDir });
+
     const tempPath = `${tempDir}/${path}`;
 
     await fs.mkdir(dirname(tempPath), { recursive: true });
@@ -109,12 +115,7 @@ export const parseString = async (
 
     const file = await parser(tempDir, path);
 
-    try {
-        await fs.unlink(tempPath);
-        await fs.rm(tempDir, { recursive: true });
-    } catch (error) {
-        console.log(error);
-    }
+    await temp.cleanup();
 
     return file;
 };