js-build-tools 6.5.0 → 6.7.0

package/functions/compileReadme.mjs

@@ -1,9 +1,9 @@
-import { addToReadme, readmeTemplate } from './partials.mjs'
-import { series } from 'gulp'
-
-/**
- * Generate the README.md file based off of the template, then append the generated documentation.
- * @memberOf module:js-build-tools
- * @returns {stream.Stream}
- */
-export const compileReadme = series(readmeTemplate, addToReadme)
+import { addToReadme, readmeTemplate } from './partials.mjs'
+import { series } from 'gulp'
+
+/**
+ * Generate the README.md file based off of the template, then append the generated documentation.
+ * @memberOf module:js-build-tools
+ * @returns {stream.Stream}
+ */
+export const compileReadme = series(readmeTemplate, addToReadme)

package/functions/compileReadme.test.mjs

@@ -1,18 +1,18 @@
-import { compileReadme } from './compileReadme.mjs'
-import { addToReadme, readmeTemplate } from './partials.mjs'
-
-const genericFunction = () => Promise.resolve(true)
-jest.mock('./partials/addToReadme.mjs', () => ({ addToReadme: jest.fn(genericFunction) }))
-jest.mock('./partials/readmeTemplate.mjs', () => ({ readmeTemplate: jest.fn(genericFunction) }))
-
-describe('compileReadme', () => {
-  test('calls readmeTemplate and addToReadme', done => {
-    expect.assertions(2)
-    compileReadme()
-    setTimeout(() => {
-      expect(readmeTemplate).toHaveBeenCalled()
-      expect(addToReadme).toHaveBeenCalled()
-      done()
-    }, 500)
-  })
-})
+import { compileReadme } from './compileReadme.mjs'
+import { addToReadme, readmeTemplate } from './partials.mjs'
+
+const genericFunction = () => Promise.resolve(true)
+jest.mock('./partials/addToReadme.mjs', () => ({ addToReadme: jest.fn(genericFunction) }))
+jest.mock('./partials/readmeTemplate.mjs', () => ({ readmeTemplate: jest.fn(genericFunction) }))
+
+describe('compileReadme', () => {
+  test('calls readmeTemplate and addToReadme', done => {
+    expect.assertions(2)
+    compileReadme()
+    setTimeout(() => {
+      expect(readmeTemplate).toHaveBeenCalled()
+      expect(addToReadme).toHaveBeenCalled()
+      done()
+    }, 500)
+  })
+})

package/functions/copyFiles.mjs

@@ -1,32 +1,32 @@
-import { copyFor } from './partials/copyFor.mjs'
-import * as gulpConfig from '../gulp.config.mjs'
-
-/**
- * Copy files from one place to another - any number of { from, to, base? } entries, configured via copyFiles.
- * A project with no copyFiles entries gets a task that does nothing. Meant to be run explicitly (its own script,
- * e.g. `gulp copyFiles`), not folded into the default build - a bundle (or any other file this copies) is often
- * the live, publicly-reachable version of something, so publishing it elsewhere should be a deliberate step, not
- * a side effect of every routine local build.
- *
- * Reuses copyFor (already shared with copyFonts) rather than a fresh src()/dest() pair - that keeps its existing
- * encoding: false fix (Gulp 5's default utf8 handling corrupts binary files such as fonts or a minified bundle)
- * instead of silently losing it in a parallel implementation.
- *
- * Each entry runs as its own independent copyFor(), awaited via Promise.all rather than merged into one combined
- * stream (mergeStream, the pattern bundle.mjs uses) - that pattern assumes every merged stream ends up at the
- * same shared destination; here, each entry can have a genuinely different one, and merging streams that each
- * already terminate at their own separate dest() hit a real "write after end" error in practice.
- * @memberOf module:js-build-tools
- * @returns {Promise<void>}
- */
-export const copyFiles = () => {
-  const copies = gulpConfig.get('copyFiles', [])
-  if (!copies.length) {
-    return Promise.resolve()
-  }
-  return Promise.all(copies.map(({ from, to, base }) => new Promise((resolve, reject) => {
-    copyFor(from, to, { base })
-      .on('finish', resolve)
-      .on('error', reject)
-  }))).then(() => {})
-}
+import { copyFor } from './partials/copyFor.mjs'
+import * as gulpConfig from '../gulp.config.mjs'
+
+/**
+ * Copy files from one place to another - any number of { from, to, base? } entries, configured via copyFiles.
+ * A project with no copyFiles entries gets a task that does nothing. Meant to be run explicitly (its own script,
+ * e.g. `gulp copyFiles`), not folded into the default build - a bundle (or any other file this copies) is often
+ * the live, publicly-reachable version of something, so publishing it elsewhere should be a deliberate step, not
+ * a side effect of every routine local build.
+ *
+ * Reuses copyFor (already shared with copyFonts) rather than a fresh src()/dest() pair - that keeps its existing
+ * encoding: false fix (Gulp 5's default utf8 handling corrupts binary files such as fonts or a minified bundle)
+ * instead of silently losing it in a parallel implementation.
+ *
+ * Each entry runs as its own independent copyFor(), awaited via Promise.all rather than merged into one combined
+ * stream (mergeStream, the pattern bundle.mjs uses) - that pattern assumes every merged stream ends up at the
+ * same shared destination; here, each entry can have a genuinely different one, and merging streams that each
+ * already terminate at their own separate dest() hit a real "write after end" error in practice.
+ * @memberOf module:js-build-tools
+ * @returns {Promise<void>}
+ */
+export const copyFiles = () => {
+  const copies = gulpConfig.get('copyFiles', [])
+  if (!copies.length) {
+    return Promise.resolve()
+  }
+  return Promise.all(copies.map(({ from, to, base }) => new Promise((resolve, reject) => {
+    copyFor(from, to, { base })
+      .on('finish', resolve)
+      .on('error', reject)
+  }))).then(() => {})
+}

package/functions/copyFiles.test.mjs

@@ -1,78 +1,78 @@
-import fs from 'fs'
-import * as setUp from './test-helpers/setUp.mjs'
-import { copyFiles } from './copyFiles.mjs'
-
-setUp.setDefaults('test-copy-files')
-const gulpConfig = setUp.gulpConfig
-
-beforeEach(() => setUp.beforeEach())
-afterEach(() => {
-  gulpConfig.set('copyFiles', [])
-  return setUp.afterEach()
-})
-
-describe('copyFiles', () => {
-  test('with no entries configured, does nothing and resolves', async () => {
-    await expect(copyFiles()).resolves.toBeUndefined()
-  })
-
-  test('copies a file from one place to another', async () => {
-    const sourceDir = `${gulpConfig.get('rootPath')}copy-source`
-    const destDir = `${gulpConfig.get('rootPath')}copy-dest`
-    fs.mkdirSync(sourceDir, { recursive: true })
-    fs.writeFileSync(`${sourceDir}/one.txt`, 'one file\n')
-    gulpConfig.set('copyFiles', [{ from: `${sourceDir}/*.txt`, to: destDir }])
-
-    await copyFiles()
-
-    expect(fs.readFileSync(`${destDir}/one.txt`).toString()).toBe('one file\n')
-  })
-
-  test('copies several configured entries, each to its own destination', async () => {
-    const sourceDirA = `${gulpConfig.get('rootPath')}copy-source-a`
-    const sourceDirB = `${gulpConfig.get('rootPath')}copy-source-b`
-    const destDirA = `${gulpConfig.get('rootPath')}copy-dest-a`
-    const destDirB = `${gulpConfig.get('rootPath')}copy-dest-b`
-    fs.mkdirSync(sourceDirA, { recursive: true })
-    fs.mkdirSync(sourceDirB, { recursive: true })
-    fs.writeFileSync(`${sourceDirA}/a.txt`, 'from a\n')
-    fs.writeFileSync(`${sourceDirB}/b.txt`, 'from b\n')
-    gulpConfig.set('copyFiles', [
-      { from: `${sourceDirA}/*.txt`, to: destDirA },
-      { from: `${sourceDirB}/*.txt`, to: destDirB }
-    ])
-
-    await copyFiles()
-
-    expect(fs.readFileSync(`${destDirA}/a.txt`).toString()).toBe('from a\n')
-    expect(fs.readFileSync(`${destDirB}/b.txt`).toString()).toBe('from b\n')
-  })
-
-  test('base preserves the glob\'s own subdirectory structure in the destination', async () => {
-    const sourceDir = `${gulpConfig.get('rootPath')}copy-source-nested`
-    const destDir = `${gulpConfig.get('rootPath')}copy-dest-nested`
-    fs.mkdirSync(`${sourceDir}/inner`, { recursive: true })
-    fs.writeFileSync(`${sourceDir}/inner/deep.txt`, 'nested file\n')
-    gulpConfig.set('copyFiles', [{ from: `${sourceDir}/**/*.txt`, to: destDir, base: sourceDir }])
-
-    await copyFiles()
-
-    expect(fs.readFileSync(`${destDir}/inner/deep.txt`).toString()).toBe('nested file\n')
-  })
-
-  // The real bug this guards against: a fresh src()/dest() pair (what this used to do) loses copyFor.mjs's own
-  // encoding: false fix - Gulp 5's default utf8 handling corrupts binary files such as fonts or a minified
-  // bundle. Reusing copyFor instead of duplicating its logic keeps that fix automatically.
-  test('copies binary files byte for byte, including bytes above 0x7f', async () => {
-    const sourceDir = `${gulpConfig.get('rootPath')}copy-source-binary`
-    const destDir = `${gulpConfig.get('rootPath')}copy-dest-binary`
-    fs.mkdirSync(sourceDir, { recursive: true })
-    const original = Buffer.from(Array.from({ length: 512 }, (_, i) => i % 256))
-    fs.writeFileSync(`${sourceDir}/font.woff`, original)
-    gulpConfig.set('copyFiles', [{ from: `${sourceDir}/*.woff`, to: destDir }])
-
-    await copyFiles()
-
-    expect(fs.readFileSync(`${destDir}/font.woff`).equals(original)).toBe(true)
-  })
-})
+import fs from 'fs'
+import * as setUp from './test-helpers/setUp.mjs'
+import { copyFiles } from './copyFiles.mjs'
+
+setUp.setDefaults('test-copy-files')
+const gulpConfig = setUp.gulpConfig
+
+beforeEach(() => setUp.beforeEach())
+afterEach(() => {
+  gulpConfig.set('copyFiles', [])
+  return setUp.afterEach()
+})
+
+describe('copyFiles', () => {
+  test('with no entries configured, does nothing and resolves', async () => {
+    await expect(copyFiles()).resolves.toBeUndefined()
+  })
+
+  test('copies a file from one place to another', async () => {
+    const sourceDir = `${gulpConfig.get('rootPath')}copy-source`
+    const destDir = `${gulpConfig.get('rootPath')}copy-dest`
+    fs.mkdirSync(sourceDir, { recursive: true })
+    fs.writeFileSync(`${sourceDir}/one.txt`, 'one file\n')
+    gulpConfig.set('copyFiles', [{ from: `${sourceDir}/*.txt`, to: destDir }])
+
+    await copyFiles()
+
+    expect(fs.readFileSync(`${destDir}/one.txt`).toString()).toBe('one file\n')
+  })
+
+  test('copies several configured entries, each to its own destination', async () => {
+    const sourceDirA = `${gulpConfig.get('rootPath')}copy-source-a`
+    const sourceDirB = `${gulpConfig.get('rootPath')}copy-source-b`
+    const destDirA = `${gulpConfig.get('rootPath')}copy-dest-a`
+    const destDirB = `${gulpConfig.get('rootPath')}copy-dest-b`
+    fs.mkdirSync(sourceDirA, { recursive: true })
+    fs.mkdirSync(sourceDirB, { recursive: true })
+    fs.writeFileSync(`${sourceDirA}/a.txt`, 'from a\n')
+    fs.writeFileSync(`${sourceDirB}/b.txt`, 'from b\n')
+    gulpConfig.set('copyFiles', [
+      { from: `${sourceDirA}/*.txt`, to: destDirA },
+      { from: `${sourceDirB}/*.txt`, to: destDirB }
+    ])
+
+    await copyFiles()
+
+    expect(fs.readFileSync(`${destDirA}/a.txt`).toString()).toBe('from a\n')
+    expect(fs.readFileSync(`${destDirB}/b.txt`).toString()).toBe('from b\n')
+  })
+
+  test('base preserves the glob\'s own subdirectory structure in the destination', async () => {
+    const sourceDir = `${gulpConfig.get('rootPath')}copy-source-nested`
+    const destDir = `${gulpConfig.get('rootPath')}copy-dest-nested`
+    fs.mkdirSync(`${sourceDir}/inner`, { recursive: true })
+    fs.writeFileSync(`${sourceDir}/inner/deep.txt`, 'nested file\n')
+    gulpConfig.set('copyFiles', [{ from: `${sourceDir}/**/*.txt`, to: destDir, base: sourceDir }])
+
+    await copyFiles()
+
+    expect(fs.readFileSync(`${destDir}/inner/deep.txt`).toString()).toBe('nested file\n')
+  })
+
+  // The real bug this guards against: a fresh src()/dest() pair (what this used to do) loses copyFor.mjs's own
+  // encoding: false fix - Gulp 5's default utf8 handling corrupts binary files such as fonts or a minified
+  // bundle. Reusing copyFor instead of duplicating its logic keeps that fix automatically.
+  test('copies binary files byte for byte, including bytes above 0x7f', async () => {
+    const sourceDir = `${gulpConfig.get('rootPath')}copy-source-binary`
+    const destDir = `${gulpConfig.get('rootPath')}copy-dest-binary`
+    fs.mkdirSync(sourceDir, { recursive: true })
+    const original = Buffer.from(Array.from({ length: 512 }, (_, i) => i % 256))
+    fs.writeFileSync(`${sourceDir}/font.woff`, original)
+    gulpConfig.set('copyFiles', [{ from: `${sourceDir}/*.woff`, to: destDir }])
+
+    await copyFiles()
+
+    expect(fs.readFileSync(`${destDir}/font.woff`).equals(original)).toBe(true)
+  })
+})

package/functions/copyFonts.mjs

@@ -1,9 +1,9 @@
-import { copyFor } from './partials/copyFor.mjs'
-import * as gulpConfig from '../gulp.config.mjs'
-
-/**
- * Move the font files into the browser directory.
- * @memberOf module:js-build-tools
- * @return {stream.Stream}
- */
-export const copyFonts = () => copyFor(gulpConfig.get('fonts.from'), gulpConfig.get('fonts.to'))
+import { copyFor } from './partials/copyFor.mjs'
+import * as gulpConfig from '../gulp.config.mjs'
+
+/**
+ * Move the font files into the browser directory.
+ * @memberOf module:js-build-tools
+ * @return {stream.Stream}
+ */
+export const copyFonts = () => copyFor(gulpConfig.get('fonts.from'), gulpConfig.get('fonts.to'))

package/functions/copyFonts.test.mjs

@@ -1,32 +1,32 @@
-import fs from 'fs'
-import * as setUp from './test-helpers/setUp.mjs'
-import { copyFonts } from './copyFonts.mjs'
-
-setUp.setDefaults('test-copy-fonts')
-const gulpConfig = setUp.gulpConfig
-
-const fontContent = 'comic sans'
-
-beforeEach(setUp.beforeEach)
-
-afterEach(setUp.afterEach)
-
-describe('copyFonts', () => {
-  test('uses the configured paths', done => {
-    const srcPath = gulpConfig.get('srcPath') + '/fonts'
-    fs.mkdirSync(srcPath, { recursive: true })
-    const fontFile = `${srcPath}/comic-sans.woff`
-    fs.writeFileSync(fontFile, fontContent)
-    const fontPath = gulpConfig.get('fonts.to')
-    expect.assertions(1)
-    copyFonts()
-      .on('finish', () => {
-        expect(fs.existsSync(fontPath)).toBeTruthy()
-        done()
-      })
-      .on('error', error => {
-        console.error('Encountered error', error)
-        done()
-      })
-  }, 10000)
-})
+import fs from 'fs'
+import * as setUp from './test-helpers/setUp.mjs'
+import { copyFonts } from './copyFonts.mjs'
+
+setUp.setDefaults('test-copy-fonts')
+const gulpConfig = setUp.gulpConfig
+
+const fontContent = 'comic sans'
+
+beforeEach(setUp.beforeEach)
+
+afterEach(setUp.afterEach)
+
+describe('copyFonts', () => {
+  test('uses the configured paths', done => {
+    const srcPath = gulpConfig.get('srcPath') + '/fonts'
+    fs.mkdirSync(srcPath, { recursive: true })
+    const fontFile = `${srcPath}/comic-sans.woff`
+    fs.writeFileSync(fontFile, fontContent)
+    const fontPath = gulpConfig.get('fonts.to')
+    expect.assertions(1)
+    copyFonts()
+      .on('finish', () => {
+        expect(fs.existsSync(fontPath)).toBeTruthy()
+        done()
+      })
+      .on('error', error => {
+        console.error('Encountered error', error)
+        done()
+      })
+  }, 10000)
+})

package/functions/default.mjs

@@ -1,12 +1,12 @@
-import { dist } from './dist.mjs'
-import { bundle } from './bundle.mjs'
-import * as gulpConfig from '../gulp.config.mjs'
-import { series } from 'gulp'
-
-/**
- * Recommended as the default task, runs the simple dist and bundle tasks.
- * @memberOf module:js-build-tools
- * @param {function} [done=null]
- * @returns {stream.Stream}
- */
-export const defaultCmd = (done = null) => gulpConfig.get('browser.enabled') ? series(dist, bundle)(done) : series(dist)(done)
+import { dist } from './dist.mjs'
+import { bundle } from './bundle.mjs'
+import * as gulpConfig from '../gulp.config.mjs'
+import { series } from 'gulp'
+
+/**
+ * Recommended as the default task, runs the simple dist and bundle tasks.
+ * @memberOf module:js-build-tools
+ * @param {function} [done=null]
+ * @returns {stream.Stream}
+ */
+export const defaultCmd = (done = null) => gulpConfig.get('browser.enabled') ? series(dist, bundle)(done) : series(dist)(done)

package/functions/default.test.mjs

@@ -1,33 +1,33 @@
-import { testHelpers } from './testHelpers.mjs'
-import { bundle } from './bundle.mjs'
-import { dist } from './dist.mjs'
-import { defaultCmd } from './default.mjs'
-
-const gulpConfig = testHelpers.gulpConfig
-
-const genericFunction = () => Promise.resolve(true)
-jest.mock('./bundle.mjs', () => ({ bundle: jest.fn(genericFunction) }))
-jest.mock('./dist.mjs', () => ({ dist: jest.fn(genericFunction) }))
-
-describe('distMinify', () => {
-  test('calls dist for nodeOnly true', done => {
-    gulpConfig.set('browser.enabled', false)
-    expect.assertions(1)
-    defaultCmd()
-    setTimeout(() => {
-      expect(dist).toHaveBeenCalled()
-      done()
-    }, 500)
-  })
-
-  test('calls dist and bundle for nodeOnly false', done => {
-    gulpConfig.set('browser.enabled', true)
-    expect.assertions(2)
-    defaultCmd()
-    setTimeout(() => {
-      expect(dist).toHaveBeenCalled()
-      expect(bundle).toHaveBeenCalled()
-      done()
-    }, 500)
-  })
-})
+import { testHelpers } from './testHelpers.mjs'
+import { bundle } from './bundle.mjs'
+import { dist } from './dist.mjs'
+import { defaultCmd } from './default.mjs'
+
+const gulpConfig = testHelpers.gulpConfig
+
+const genericFunction = () => Promise.resolve(true)
+jest.mock('./bundle.mjs', () => ({ bundle: jest.fn(genericFunction) }))
+jest.mock('./dist.mjs', () => ({ dist: jest.fn(genericFunction) }))
+
+describe('distMinify', () => {
+  test('calls dist for nodeOnly true', done => {
+    gulpConfig.set('browser.enabled', false)
+    expect.assertions(1)
+    defaultCmd()
+    setTimeout(() => {
+      expect(dist).toHaveBeenCalled()
+      done()
+    }, 500)
+  })
+
+  test('calls dist and bundle for nodeOnly false', done => {
+    gulpConfig.set('browser.enabled', true)
+    expect.assertions(2)
+    defaultCmd()
+    setTimeout(() => {
+      expect(dist).toHaveBeenCalled()
+      expect(bundle).toHaveBeenCalled()
+      done()
+    }, 500)
+  })
+})

package/functions/dist.mjs

@@ -1,8 +1,8 @@
-import { distFor } from './partials/distFor.mjs'
-
-/**
- * Simplified distribution tasks which will use arguments from distFor.
- * @memberOf module:js-build-tools
- * @returns {*}
- */
-export const dist = () => distFor()
+import { distFor } from './partials/distFor.mjs'
+
+/**
+ * Simplified distribution tasks which will use arguments from distFor.
+ * @memberOf module:js-build-tools
+ * @returns {*}
+ */
+export const dist = () => distFor()

package/functions/dist.test.mjs

@@ -1,44 +1,44 @@
-import fs from 'fs'
-import * as setUp from './test-helpers/setUp.mjs'
-import { dist } from './dist.mjs'
-
-setUp.setDefaults('test-dist')
-const gulpConfig = setUp.gulpConfig
-
-const rawContents = 'const distFor = require(\'./distFor.js\')\n' +
-  '\n' +
-  '/**\n' +
-  ' * Simplified distribution tasks which will use arguments from distFor.\n' +
-  ' * @returns {*}\n' +
-  ' */\n' +
-  'const dist = () => distFor()\n' +
-  '\n' +
-  'module.exports = dist\n'
-
-beforeEach(
-  () => setUp.beforeEach()
-    .then(
-      async () => {
-        const srcPath = gulpConfig.get('srcPath')
-        return fs.writeFileSync(`${srcPath}/dist.js`, rawContents)
-      }
-    )
-)
-
-afterEach(setUp.afterEach)
-
-describe('dist', () => {
-  test('uses the configured paths', done => {
-    const distPath = gulpConfig.get('dist.to')
-    expect.assertions(1)
-    dist()
-      .on('finish', () => {
-        expect(fs.existsSync(distPath)).toBeTruthy()
-        done()
-      })
-      .on('error', error => {
-        console.error('Encountered error', error)
-        done()
-      })
-  }, 10000)
-})
+import fs from 'fs'
+import * as setUp from './test-helpers/setUp.mjs'
+import { dist } from './dist.mjs'
+
+setUp.setDefaults('test-dist')
+const gulpConfig = setUp.gulpConfig
+
+const rawContents = 'const distFor = require(\'./distFor.js\')\n' +
+  '\n' +
+  '/**\n' +
+  ' * Simplified distribution tasks which will use arguments from distFor.\n' +
+  ' * @returns {*}\n' +
+  ' */\n' +
+  'const dist = () => distFor()\n' +
+  '\n' +
+  'module.exports = dist\n'
+
+beforeEach(
+  () => setUp.beforeEach()
+    .then(
+      async () => {
+        const srcPath = gulpConfig.get('srcPath')
+        return fs.writeFileSync(`${srcPath}/dist.js`, rawContents)
+      }
+    )
+)
+
+afterEach(setUp.afterEach)
+
+describe('dist', () => {
+  test('uses the configured paths', done => {
+    const distPath = gulpConfig.get('dist.to')
+    expect.assertions(1)
+    dist()
+      .on('finish', () => {
+        expect(fs.existsSync(distPath)).toBeTruthy()
+        done()
+      })
+      .on('error', error => {
+        console.error('Encountered error', error)
+        done()
+      })
+  }, 10000)
+})

package/functions/distMinify.mjs

@@ -1,9 +1,9 @@
-import * as gulpConfig from '../gulp.config.mjs'
-import { minifyFor } from './partials/minifyFor.mjs'
-
-/**
- * Creates minified versions of the dist files.
- * @memberOf module:js-build-tools
- * @returns {*}
- */
-export const distMinify = () => minifyFor(gulpConfig.get('browser.from'), gulpConfig.get('dist.to'))
+import * as gulpConfig from '../gulp.config.mjs'
+import { minifyFor } from './partials/minifyFor.mjs'
+
+/**
+ * Creates minified versions of the dist files.
+ * @memberOf module:js-build-tools
+ * @returns {*}
+ */
+export const distMinify = () => minifyFor(gulpConfig.get('browser.from'), gulpConfig.get('dist.to'))

package/functions/distMinify.test.mjs

@@ -1,15 +1,15 @@
-import { testHelpers } from './testHelpers.mjs'
-import { distMinify } from './distMinify.mjs'
-import { minifyFor } from './partials/minifyFor.mjs'
-
-const gulpConfig = testHelpers.gulpConfig
-
-jest.mock('./partials/minifyFor.mjs', () => ({ minifyFor: jest.fn(() => Promise.resolve(true)) }))
-
-describe('distMinify', () => {
-  test('calls minifyFor with configurations', () => {
-    expect.assertions(1)
-    distMinify()
-    expect(minifyFor).toHaveBeenCalledWith(gulpConfig.get('browser.from'), gulpConfig.get('dist.to'))
-  })
-})
+import { testHelpers } from './testHelpers.mjs'
+import { distMinify } from './distMinify.mjs'
+import { minifyFor } from './partials/minifyFor.mjs'
+
+const gulpConfig = testHelpers.gulpConfig
+
+jest.mock('./partials/minifyFor.mjs', () => ({ minifyFor: jest.fn(() => Promise.resolve(true)) }))
+
+describe('distMinify', () => {
+  test('calls minifyFor with configurations', () => {
+    expect.assertions(1)
+    distMinify()
+    expect(minifyFor).toHaveBeenCalledWith(gulpConfig.get('browser.from'), gulpConfig.get('dist.to'))
+  })
+})

package/functions/docs.mjs

@@ -1,13 +1,13 @@
-import { jsdocDocs } from './jsdocDocs.mjs'
-import { resolveDocsMethod } from './partials/resolveDocsMethod.mjs'
-import { typeDocs } from './typeDocs.mjs'
-
-/**
- * Generate the HTML documentation with whichever engine docs.method (or its typescript.enabled-based default)
- * resolves to - see {@link module:partials.resolveDocsMethod}. The standalone counterpart to build's own
- * internal branching, for running `gulp docs` directly instead of as part of a full build.
- * @memberOf module:js-build-tools
- * @param {Function} [done]
- * @returns {*}
- */
-export const docs = (done = null) => (resolveDocsMethod() === 'jsdoc' ? jsdocDocs : typeDocs)(done)
+import { jsdocDocs } from './jsdocDocs.mjs'
+import { resolveDocsMethod } from './partials/resolveDocsMethod.mjs'
+import { typeDocs } from './typeDocs.mjs'
+
+/**
+ * Generate the HTML documentation with whichever engine docs.method (or its typescript.enabled-based default)
+ * resolves to - see {@link module:partials.resolveDocsMethod}. The standalone counterpart to build's own
+ * internal branching, for running `gulp docs` directly instead of as part of a full build.
+ * @memberOf module:js-build-tools
+ * @param {Function} [done]
+ * @returns {*}
+ */
+export const docs = (done = null) => (resolveDocsMethod() === 'jsdoc' ? jsdocDocs : typeDocs)(done)