feat: replace "mkdirp" with "utils.mkdir"

Author: hasezoey

packages/mongodb-memory-server-core/src/util/MongoBinary.ts

@@ -1,11 +1,10 @@
 import os from 'os';
 import path from 'path';
-import mkdirp from 'mkdirp';
 import MongoBinaryDownload from './MongoBinaryDownload';
 import resolveConfig, { envToBool, ResolveConfigVariables } from './resolveConfig';
 import debug from 'debug';
 import * as semver from 'semver';
-import { assertion, isNullOrUndefined } from './utils';
+import { assertion, isNullOrUndefined, mkdir } from './utils';
 import { spawnSync } from 'child_process';
 import { LockFile } from './lockfile';
 import { DryMongoBinary, BaseDryMongoBinaryOptions } from './DryMongoBinary';
@@ -30,7 +29,7 @@ export class MongoBinary {
     log('download');
     const { downloadDir, version } = options;
     // create downloadDir
-    await mkdirp(downloadDir);
+    await mkdir(downloadDir);
 
     /** Lockfile path */
     const lockfile = path.resolve(downloadDir, `${version}.lock`);

packages/mongodb-memory-server-core/src/util/MongoBinaryDownload.ts

@@ -11,9 +11,8 @@ import MongoBinaryDownloadUrl from './MongoBinaryDownloadUrl';
 import { HttpsProxyAgent } from 'https-proxy-agent';
 import resolveConfig, { envToBool, ResolveConfigVariables } from './resolveConfig';
 import debug from 'debug';
-import { assertion, pathExists } from './utils';
+import { assertion, mkdir, pathExists } from './utils';
 import { DryMongoBinary } from './DryMongoBinary';
-import mkdirp from 'mkdirp';
 import { MongoBinaryOpts } from './MongoBinary';
 import { clearLine } from 'readline';
 import { Md5CheckFailedError } from './errors';
@@ -152,7 +151,7 @@ export class MongoBinaryDownload {
     log('startDownload');
     const mbdUrl = new MongoBinaryDownloadUrl(this.binaryOpts);
 
-    await mkdirp(this.downloadDir);
+    await mkdir(this.downloadDir);
 
     try {
       await fspromises.access(this.downloadDir, constants.X_OK | constants.W_OK); // check that this process has permissions to create files & modify file contents & read file contents
@@ -277,7 +276,7 @@ export class MongoBinaryDownload {
     const mongodbFullPath = await this.getPath();
     log(`extract: archive: "${mongoDBArchive}" final: "${mongodbFullPath}"`);
 
-    await mkdirp(path.dirname(mongodbFullPath));
+    await mkdir(path.dirname(mongodbFullPath));
 
     const filter = (file: string) => /(?:bin\/(?:mongod(?:\.exe)?)|(?:.*\.dll))$/i.test(file);
 

packages/mongodb-memory-server-core/src/util/__tests__/MongoBinaryDownload.test.ts

@@ -1,6 +1,5 @@
 import { promises as fspromises } from 'fs';
 import md5file from 'md5-file';
-import * as mkdirp from 'mkdirp';
 import { assertIsError } from '../../__tests__/testUtils/test_utils';
 import { DryMongoBinary } from '../DryMongoBinary';
 import { Md5CheckFailedError } from '../errors';
@@ -11,7 +10,6 @@ import { envName, ResolveConfigVariables } from '../resolveConfig';
 import * as utils from '../utils';
 
 jest.mock('md5-file');
-jest.mock('mkdirp');
 
 describe('MongoBinaryDownload', () => {
   afterEach(() => {
@@ -248,7 +246,7 @@ describe('MongoBinaryDownload', () => {
   it('should return the mongodb archive path (startDownload)', async () => {
     const downloadUrl = 'https://fastdl.mongodb.org/linux/mongod-something-something.tgz';
     const archivePath = '/path/to/archive.tgz';
-    jest.spyOn(mkdirp, 'default').mockResolvedValue(void 0);
+    jest.spyOn(utils, 'mkdir').mockResolvedValue(void 0);
     jest.spyOn(fspromises, 'access').mockResolvedValue(void 0);
     jest.spyOn(MongoBinaryDownloadUrl.prototype, 'getDownloadUrl').mockResolvedValue(downloadUrl);
 
@@ -264,7 +262,7 @@ describe('MongoBinaryDownload', () => {
 
   it('should return the mongodb archive path (startDownload)', async () => {
     const customError = new Error('custom fs error');
-    jest.spyOn(mkdirp, 'default').mockResolvedValue(void 0);
+    jest.spyOn(utils, 'mkdir').mockResolvedValue(void 0);
     jest.spyOn(fspromises, 'access').mockRejectedValue(customError);
     jest.spyOn(console, 'error').mockImplementation(() => void 0);
 

packages/mongodb-memory-server-core/src/util/__tests__/dryBinary.test.ts

@@ -4,7 +4,6 @@ import * as tmp from 'tmp';
 import { constants, promises as fspromises } from 'fs';
 import { envName, ResolveConfigVariables } from '../resolveConfig';
 import * as utils from '../utils';
-import mkdirp from 'mkdirp';
 import { LinuxOS, OtherOS } from '../getos';
 import { NoRegexMatchError, NoSystemBinaryFoundError, ParseArchiveRegexError } from '../errors';
 import { assertIsError } from '../../__tests__/testUtils/test_utils';
@@ -70,9 +69,9 @@ describe('DryBinary', () => {
 
       // Create all directories
       {
-        await mkdirp(path.resolve(tmpDir.name, 'node_modules/mongodb-memory-server')); // mock being in an postinstall directory path
-        await mkdirp(path.resolve(tmpDir.name, 'node_modules/.cache')); // mock having an local modules cache
-        await mkdirp(path.resolve(tmpDir.name, 'homedir/.cache/mongodb-binaries')); // mock having an "legacy" global directory
+        await utils.mkdir(path.resolve(tmpDir.name, 'node_modules/mongodb-memory-server')); // mock being in an postinstall directory path
+        await utils.mkdir(path.resolve(tmpDir.name, 'node_modules/.cache')); // mock having an local modules cache
+        await utils.mkdir(path.resolve(tmpDir.name, 'homedir/.cache/mongodb-binaries')); // mock having an "legacy" global directory
         await fspromises.writeFile(path.resolve(tmpDir.name, 'package.json'), '');
       }
     });

packages/mongodb-memory-server-core/src/util/lockfile.ts

@@ -2,7 +2,6 @@ import { EventEmitter } from 'events';
 import * as utils from './utils';
 import debug from 'debug';
 import * as path from 'path';
-import mkdirp from 'mkdirp';
 import { promises as fspromises } from 'fs';
 import { Mutex } from 'async-mutex';
 import { v4 as uuidv4 } from 'uuid';
@@ -210,7 +209,7 @@ export class LockFile {
           throw new RepeatError(true);
         }
 
-        await mkdirp(path.dirname(file));
+        await utils.mkdir(path.dirname(file));
 
         await fspromises.writeFile(file, `${process.pid.toString()} ${uuid}`);