Merge pull request #1 from chemmedia/fix-383

fix siddharthkp#383 – ensure files are not matched twice

README.md

@@ -12,6 +12,12 @@
 [![NPM Downloads](https://img.shields.io/npm/dm/bundlesize.svg?style=flat)](https://www.npmjs.com/package/bundlesize)
  
 
+#### Why this fork exists?
+It seems that the original repository is no longer actively maintained. This fork is intended to fix the following:
+
+* remove deprecated brotli compression to improve performance
+* [fix fuzzy matching logic to not include paths that match other patterns](https://github.com/siddharthkp/bundlesize/issues/383)
+
 #### Setup
 
 ```sh

package.json

@@ -1,6 +1,6 @@
 {
   "name": "bundlesize",
-  "version": "1.0.0",
+  "version": "2.0.0",
   "description": "Keep your library size in check",
   "repository": {
     "type": "git",

src/files.js

@@ -14,12 +14,14 @@ config.map(file => {
       silent: true
     })
   } else {
-    paths.map(path => {
-      const maxSize = bytes(file.maxSize) || Infinity
-      const compression = file.compression || 'gzip'
-      const size = compressedSize(fs.readFileSync(path, 'utf8'), compression)
-      files.push({ maxSize, path, size, compression })
-    })
+    paths
+      .filter(path => !files.some(file => file.path === path))
+      .map(path => {
+        const maxSize = bytes(file.maxSize) || Infinity
+        const compression = file.compression || 'gzip'
+        const size = compressedSize(fs.readFileSync(path, 'utf8'), compression)
+        files.push({ maxSize, path, size, compression })
+      })
   }
 })
 

tests/fixtures/12/build/file-241B.js

@@ -0,0 +1,19 @@
+/*
+The contents of this file aren't important,
+what's important is the size
+
+241B gzip
+*/
+
+const { inspect } = require('util')
+const files = require('./src/files')
+const reporter = require('./src/reporter')
+const build = require('./src/build')
+
+reporter(files)
+
+process.on('unhandledRejection', function(reason) {
+  console.log('Unhandled Promise')
+  console.log(inspect(reason))
+  build.error()
+})

tests/fixtures/12/bundlesize.config.json

@@ -0,0 +1,16 @@
+{
+  "files": [
+    {
+      "path": "build/file-734B.js",
+      "maxSize": "1KB"
+    },
+    {
+      "path": "build/file-*.js",
+      "maxSize": "250B"
+    },
+    {
+      "path": "build/file-241B.js",
+      "maxSize": "100B"
+    }
+  ]
+}

tests/index.js

@@ -103,3 +103,9 @@ test.skip('10. bug repro: bundlesize should dedup files', t => {
   t.is(exitCode, 0) // this is failing
   t.snapshot(stdout)
 })
+
+test('11. bug repro: files should not be matched twice, order should matter (https://github.com/siddharthkp/bundlesize/issues/383)', t => {
+  const { stdout, exitCode } = run(12)
+  t.is(exitCode, 0) // this is failing
+  t.snapshot(stdout)
+})

tests/snapshots/index.js.md

@@ -10,70 +10,20 @@ Generated by [AVA](https://ava.li).
 
     'PASS  file-1.js: 270B < maxSize 300B (gzip)'
 
-## 10. pass: match by fuzzy name
+## 11. bug repro: files should not be matched twice, order should matters (https://github.com/siddharthkp/bundlesize/issues/383)
 
 > Snapshot 1
 
-    `PASS  build/vendor-ha5h.js: 270B < maxSize 350B (gzip) ␊
+    `PASS  build/file-734B.js: 734B < maxSize 1KB (gzip) ␊
     ␊
-     PASS  build/chunks/chunk-ch0nk.js: 270B < maxSize 300B (gzip)`
+     PASS  build/file-241B.js: 241B < maxSize 250B (gzip)`
 
 ## 2. fail: single file larger than limit
 
 > Snapshot 1
 
     'FAIL  file-2.js: 270B > maxSize 250B (gzip)'
 
-## 3. pass: use brotli
-
-> Snapshot 1
-
-    'PASS  file-3.js: 245B < maxSize 250B (brotli)'
-
-## 4. fail: dont use compression
-
-> Snapshot 1
-
-    'FAIL  file-4.js: 437B > maxSize 300B (no compression)'
-
-## 5. pass: custom config file
-
-> Snapshot 1
-
-    'PASS  file-5.js: 270B < maxSize 300B (gzip)'
-
-## 6. pass: multiple files, both smaller than limit
-
-> Snapshot 1
-
-    `PASS  file-61.js: 270B < maxSize 300B (gzip) ␊
-    ␊
-     PASS  file-62.js: 270B < maxSize 300B (gzip)`
-
-## 7. fail: multiple files, both bigger than limit
-
-> Snapshot 1
-
-    `FAIL  file-61.js: 270B > maxSize 200B (gzip) ␊
-    ␊
-     FAIL  file-62.js: 270B > maxSize 200B (gzip)`
-
-## 8. fail: multiple files, 1 smaller + 1 bigger than limit
-
-> Snapshot 1
-
-    `PASS  file-61.js: 270B < maxSize 300B (gzip) ␊
-    ␊
-     FAIL  file-62.js: 270B > maxSize 200B (gzip)`
-
-## 9. pass: catch all js files
-
-> Snapshot 1
-
-    `PASS  build/chunks/chunk-ch0nk.js: 270B < maxSize 300B (gzip) ␊
-    ␊
-     PASS  build/vendor-ha5h.js: 270B < maxSize 300B (gzip)`
-
 ## 3. fail: dont use compression
 
 > Snapshot 1