add unit tests

- add unit test for parse and seach
- update CONTRIBUTING.md: add test commands
- remove undefined value webpack integration test

Author: Ffloriel

CONTRIBUTING.md

@@ -16,7 +16,7 @@ Purgecss use [SemVer](http://semver.org/) for versioning.
    and create your branch from `master`.
 2. If you've added code that should be tested, add tests!
 3. If you've changed APIs, update the documentation.
-   <!-- 4. Ensure the test suite passes (`npm test`). -->
+4. Ensure the test suite passes (`npm test`).
 4. Make sure your code lints (`npm run lint`).
 
 ### Development Workflow
@@ -26,10 +26,10 @@ several commands:
 
 * `npm run dev` will build cjs and es module of Purgecss in the `lib` folder and
   watch for changes.
-  <!-- - `npm run lint` checks the code style.
+* `npm run lint` checks the code style.
 * `npm test` runs the complete test suite.
 * `npm test -- --watch` runs an interactive test watcher.
-* `npm test <pattern>` runs tests with matching filenames. -->
+* `npm test <pattern>` runs tests with matching filenames.
 * `npm run build` creates the cjs and es module of Purgecss in the `lib` folder.
 
 Make sure that your pull request contains unit tests for any new functionality.

__tests__/parse.test.js

@@ -0,0 +1,72 @@
+import { entryPaths, entries, flatten } from './../src/parse'
+
+describe('Parse entry paths', () => {
+    it('returns an empty array by default', () => {
+        expect(entryPaths()).toEqual([])
+    })
+
+    it('returns an object as itself', () => {
+        const o = { a: ['a', 'b', 'c'] }
+        expect(entryPaths(o)).toEqual(o)
+    })
+
+    it('puts a string inside an array', () => {
+        const str = 'foobar'
+        expect(entryPaths(str)).toEqual([str])
+    })
+})
+
+describe('Flatten entry paths', () => {
+    it('returns an array as itself', () => {
+        const a = ['a', 'b', 'c']
+        expect(flatten(a)).toEqual(a)
+    })
+
+    it('returns an object of arrays as one flat array', () => {
+        const o = { a: ['a', 'b'], b: ['c', 'd'] }
+        expect(flatten(o)).toEqual(['a', 'b', 'c', 'd'])
+    })
+})
+
+describe('Parse entries', () => {
+    it('returns paths if there is no chunk name', () => {
+        const paths = ['a', 'b', 'c']
+        expect(entries(paths)).toEqual(paths)
+    })
+
+    it('returns paths if paths are an array already', () => {
+        const paths = ['a', 'b', 'c']
+        expect(entries(paths, 'foobar')).toEqual(paths)
+    })
+
+    it('returns chunk paths', () => {
+        const entryPaths = ['a', 'b', 'c']
+        const paths = {
+            foobar: entryPaths
+        }
+        expect(entries(paths, 'foobar')).toEqual(entryPaths)
+    })
+
+    it('returns chunk path wrapped in an array', () => {
+        const entryPaths = 'a'
+        const paths = {
+            foobar: entryPaths
+        }
+        expect(entries(paths, 'foobar')).toEqual([entryPaths])
+    })
+
+    it('returns an empty array if failed to find entry', () => {
+        const paths = {
+            foobar: 'a'
+        }
+        expect(entries(paths, 'barbar')).toEqual([])
+    })
+
+    it('returns an empty array if failed to find entry with multiple paths', () => {
+        const paths = {
+            foobar: 'a',
+            barbar: 'b'
+        }
+        expect(entries(paths, 'foofoo')).toEqual([])
+    })
+})

__tests__/search.test.js

@@ -0,0 +1,87 @@
+import { assets, files } from './../src/search'
+
+describe('Search assets', () => {
+    it('returns nothing if nothing is passed', () => {
+        expect(assets()).toEqual([])
+    })
+
+    it('returns matches based on a pattern', () => {
+        const modules = {
+            'foobar.txt': {},
+            'barbar.css': {}
+        }
+        const extensions = ['.txt']
+        const matches = [{ name: 'foobar.txt', asset: {} }]
+
+        expect(assets(modules, extensions)).toEqual(matches)
+    })
+
+    it('returns matches if they have query', () => {
+        const modules = {
+            'foobar.txt?123': {},
+            'barbar.css': {}
+        }
+        const extensions = ['.txt']
+        const matches = [{ name: 'foobar.txt?123', asset: {} }]
+
+        expect(assets(modules, extensions)).toEqual(matches)
+    })
+})
+
+describe('Search files', () => {
+    let chunk
+    beforeEach(() => {
+        chunk = {
+            mapModules: function(cb) {
+                return Array.from(this.modules, cb)
+            }
+        }
+    })
+
+    it('returns matches based on extension', () => {
+        chunk.modules = ['foobar.txt', 'barbar.css']
+        const extensions = ['.txt']
+        const matches = ['foobar.txt']
+
+        expect(files(chunk, extensions)).toEqual(matches)
+    })
+
+    it('does not fail with missing modules', () => {
+        chunk.modules = ['foobar.txt', '', 'barbar.css']
+        const extensions = ['.txt']
+        const matches = ['foobar.txt']
+
+        expect(files(chunk, extensions)).toEqual(matches)
+    })
+
+    it('returns matches based on extension with a customized getter', () => {
+        chunk.modules = [
+            {
+                resource: 'foobar.txt'
+            },
+            {
+                resource: 'barbar.css'
+            }
+        ]
+        const extensions = ['.txt']
+        const matches = ['foobar.txt']
+
+        expect(files(chunk, extensions, file => file.resource)).toEqual(matches)
+    })
+
+    it('does not fail with missing modules when a getter fails', () => {
+        chunk.modules = [
+            {
+                resource: 'foobar.txt'
+            },
+            {},
+            {
+                resource: 'barbar.css'
+            }
+        ]
+        const extensions = ['.txt']
+        const matches = ['foobar.txt']
+
+        expect(files(chunk, extensions, file => file.resource)).toEqual(matches)
+    })
+})

__tests__/webpack-integration.test.js

@@ -36,7 +36,7 @@ describe('Webpack Integration Tests', () => {
                 if (stats.hasErrors()) return done(new Error(stats.toString()))
                 const testFile = path.join(outputDirectory, 'test.js')
                 if (fs.existsSync(testFile)) {
-                    require(testFile)(suite)
+                    require(testFile)()
                 }
                 const expectedDirectory = path.join(testDirectory, 'expected')
                 fs.readdirSync(expectedDirectory).forEach(file => {