js-build-tools 2.2.11 → 2.2.12

package/functions/build.test.js

@@ -105,7 +105,7 @@ describe('build', () => {
     fs.writeFileSync(`${sourcesPath}/file2.js`, file2Contents)
     fs.writeFileSync(`${srcPath}/main.js`, mainFileContents)
     gulpConfig.set('nodeOnly', true)
-    expect.assertions(20)
+    expect.assertions(13)
     build(() => {
       // Called 'clean'
       expect(clean).toHaveBeenCalled()
@@ -122,17 +122,10 @@ describe('build', () => {
       expect(fs.existsSync(`${distSourcesPath}/file1.min.js`)).toBeTruthy()
       const babelifiedFile1 = fs.readFileSync(`${distSourcesPath}/file1.js`).toString()
       expect(countMatches(babelifiedFile1, '\'use strict\'')).toEqual(1)
-      expect(countMatches(babelifiedFile1, 'const ')).toEqual(0)
-      expect(countMatches(babelifiedFile1, 'var ')).toEqual(1)
-      expect(countMatches(babelifiedFile1, 'function ')).toEqual(1)
-      expect(countMatches(babelifiedFile1, 'return ')).toEqual(1)
 
       expect(fs.existsSync(`${distSourcesPath}/file2.min.js`)).toBeTruthy()
       const babelifiedFile2 = fs.readFileSync(`${distSourcesPath}/file2.js`).toString()
       expect(countMatches(babelifiedFile2, '\'use strict\'')).toEqual(1)
-      expect(countMatches(babelifiedFile2, 'const ')).toEqual(0)
-      expect(countMatches(babelifiedFile2, 'var ')).toEqual(2)
-      expect(countMatches(babelifiedFile2, 'function ')).toEqual(1)
 
       // Generated README.md (compileReadme)
       const readmeContents = fs.readFileSync(`${readmePath}/${readmeFile}`).toString()
@@ -180,8 +173,8 @@ describe('build', () => {
       // Minify the browser files (bundleMinify)
       expect(fs.existsSync(`${browserPath}/${browserFile}.min.js`)).toBeTruthy()
       const bundledContents = fs.readFileSync(bundledFile).toString()
-      expect(countMatches(bundledContents, 'file1')).toEqual(8)
-      expect(countMatches(bundledContents, 'file2')).toEqual(8)
+      expect(countMatches(bundledContents, 'file1')).toEqual(7)
+      expect(countMatches(bundledContents, 'file2')).toEqual(7)
 
       // Generated README.md (compileReadme)
       const readmeContents = fs.readFileSync(`${readmePath}/${readmeFile}`).toString()
@@ -272,7 +265,7 @@ describe('build', () => {
       // Minify the browser files (bundleMinify)
       expect(fs.existsSync(`${browserPath}/${browserFile}.min.js`)).toBeTruthy()
       const bundledContents = fs.readFileSync(bundledFile).toString()
-      expect(countMatches(bundledContents, 'main')).toEqual(3)
+      expect(countMatches(bundledContents, 'main')).toEqual(2)
 
       // Generated README.md (compileReadme)
       const readmeContents = fs.readFileSync(`${readmePath}/${readmeFile}`).toString()

package/functions/partials/distFor.test.js

@@ -68,7 +68,7 @@ describe('distFor', () => {
     const srcPath = gulpConfig.get('srcPath')
     const srcFile = `${srcPath}/distFor.js`
     fs.writeFileSync(srcFile, rawContents)
-    expect.assertions(7)
+    expect.assertions(5)
     const oldContents = fs.readFileSync(srcFile).toString()
     expect(countMatches(oldContents, 'const ')).toEqual(7)
     const distPath = gulpConfig.get('distPath')
@@ -77,11 +77,11 @@ describe('distFor', () => {
         expect(fs.existsSync(distPath)).toBeTruthy()
         const babelifiedContents = fs.readFileSync(`${distPath}/distFor.js`).toString()
         expect(countMatches(babelifiedContents, '"use strict"')).toEqual(1)
-        // If the below assertion fails it is because 'const' no longer is replaced with 'var'
-        // To fix, change `.browserslistrc` set `> 0.16%` to a lower number. For example, last fix changed from `> 0.21%`
-        // Next time, lets just change this test since we cannot set this much lower
-        expect(countMatches(babelifiedContents, 'const ')).toEqual(0)
-        expect(countMatches(babelifiedContents, 'var ')).toEqual(10)
+        /* There used to be a nice check here where `const` and `let` would be swapped to `var`, but that no longer
+           happens.
+           It would be nice if it still happened, but due to 'browserlist' supposedly, most browsers support
+           `const` and `let` now.
+         */
         expect(countMatches(babelifiedContents, 'function')).toEqual(1)
         expect(countMatches(babelifiedContents, 'arguments')).toEqual(6)
         done()
@@ -90,7 +90,7 @@ describe('distFor', () => {
         console.error('Encountered error', error)
         done()
       })
-  }, 60000)
+  }, 5000)
 
   test('can process typescript before babel', done => {
     const srcPath = gulpConfig.get('srcPath')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "js-build-tools",
-  "version": "2.2.11",
+  "version": "2.2.12",
   "description": "Standardised build process with gulp.",
   "main": "gulpfile.base.js",
   "scripts": {
@@ -33,14 +33,14 @@
     ]
   },
   "dependencies": {
-    "@babel/core": "^7.23.5",
+    "@babel/core": "^7.23.7",
     "@babel/plugin-proposal-throw-expressions": "^7.23.3",
-    "@babel/preset-env": "^7.23.5",
+    "@babel/preset-env": "^7.23.7",
     "@babel/preset-typescript": "^7.23.3",
-    "@babel/runtime-corejs3": "^7.23.5",
+    "@babel/runtime-corejs3": "^7.23.7",
     "babel-jest": "^29.7.0",
     "browserify": "^17.0.0",
-    "core-js": "^3.34.0",
+    "core-js": "^3.35.0",
     "fs": "^0.0.1-security",
     "glob": "^10.3.10",
     "gulp": "^4.0.2",
@@ -52,7 +52,7 @@
     "jest": "^29.7.0",
     "jest-environment-jsdom": "^29.7.0",
     "jsdoc-to-markdown": "^8.0.0",
-    "regenerator-runtime": "^0.14.0",
+    "regenerator-runtime": "^0.14.1",
     "vinyl-source-stream": "^2.0.0"
   },
   "files": [