style(lib): Formats files after changes to repo

Author: digitalsadhu

lib/util.js

@@ -8,7 +8,7 @@ const readPkgUp = require('read-pkg-up');
 const postcss = require('postcss');
 const atImport = require('postcss-import');
 
-module.exports.identifyCssModule = async function identifyCssModule (filePath) {
+module.exports.identifyCssModule = async function identifyCssModule(filePath) {
     const { pkg: { name, version }, path: packagePath } = await readPkgUp({
         normalize: false,
         cwd: path.dirname(filePath),
@@ -18,7 +18,7 @@ module.exports.identifyCssModule = async function identifyCssModule (filePath) {
     return { name, version, file };
 };
 
-module.exports.bundleCssModule = async function bundleCssModule (filePath) {
+module.exports.bundleCssModule = async function bundleCssModule(filePath) {
     const fileContents = await readFile(filePath, 'utf8');
     const { css } = await postcss()
         .use(atImport())

lib/writer.js

@@ -8,7 +8,7 @@ const assert = require('assert');
 const { hasher } = require('asset-pipe-common');
 
 module.exports = class Writer extends Readable {
-    constructor (files = []) {
+    constructor(files = []) {
         super({ objectMode: true });
 
         assert(
@@ -17,22 +17,36 @@ module.exports = class Writer extends Readable {
         );
         this.files = Array.isArray(files) ? files : [files];
         for (const file of this.files) {
-            assert(typeof file === 'string', `Expected 'file' (${file}) to be of type 'string', instead got '${typeof file}'`);
-            assert(isAbsolute(file), `Expected 'file' (${file}) to be an absolute path to a file but it was not`);
-            assert(existsSync(file), `Expected 'file' (${file}) to exist on file system but it did not`);
+            assert(
+                typeof file === 'string',
+                `Expected 'file' (${file}) to be of type 'string', instead got '${typeof file}'`
+            );
+            assert(
+                isAbsolute(file),
+                `Expected 'file' (${file}) to be an absolute path to a file but it was not`
+            );
+            assert(
+                existsSync(file),
+                `Expected 'file' (${file}) to exist on file system but it did not`
+            );
         }
     }
 
-    async _read () {
+    async _read() {
         const file = this.files.shift();
         if (!file) {
             this.push(null);
             return;
         }
 
         try {
-            const [css, meta] = await Promise.all([bundleCssModule(file), identifyCssModule(file)]);
-            meta.id = hasher(`${meta.name}|${meta.version}|${meta.file}|${css}`);
+            const [css, meta] = await Promise.all([
+                bundleCssModule(file),
+                identifyCssModule(file),
+            ]);
+            meta.id = hasher(
+                `${meta.name}|${meta.version}|${meta.file}|${css}`
+            );
             meta.content = css;
             this.push(meta);
         } catch (err) {

test/writer.test.js

@@ -29,7 +29,10 @@ test('identifyCssModule(filePath)', async () => {
 
 test('identifyCssModule(filePath) css in nested directory', async () => {
     expect.assertions(1);
-    const filePath = path.join(__dirname, 'test-assets/my-module-2/css/main.css');
+    const filePath = path.join(
+        __dirname,
+        'test-assets/my-module-2/css/main.css'
+    );
     const fileRef = 'my-module-2/css/main.css';
 
     const result = await identifyCssModule(filePath);
@@ -44,7 +47,10 @@ test('identifyCssModule(filePath) css in nested directory', async () => {
 
 test('bundleCssModule(filePath)', async () => {
     expect.assertions(3);
-    const filePath = path.join(__dirname, 'test-assets/my-module-3/css/main.css');
+    const filePath = path.join(
+        __dirname,
+        'test-assets/my-module-3/css/main.css'
+    );
 
     const result = await bundleCssModule(filePath);
 
@@ -68,7 +74,9 @@ test('new Writer(filePath)', done => {
     writer.on('end', () => {
         const result1 = items[0];
         const result2 = items[1];
-        expect(result1.id).toBe(hasher(`my-module-1|1.0.1|${fileRef}|${result1.content}`));
+        expect(result1.id).toBe(
+            hasher(`my-module-1|1.0.1|${fileRef}|${result1.content}`)
+        );
         expect(result2).toBeFalsy();
         done();
     });
@@ -98,15 +106,20 @@ test('new Writer([filePath])', done => {
     writer.on('end', () => {
         const result1 = items[0];
         const result2 = items[1];
-        expect(result1.id).toBe(hasher(`my-module-1|1.0.1|${fileRef}|${result1.content}`));
+        expect(result1.id).toBe(
+            hasher(`my-module-1|1.0.1|${fileRef}|${result1.content}`)
+        );
         expect(result2).toBeFalsy();
         done();
     });
 });
 
 test('Writer processes @import statements', done => {
     expect.assertions(5);
-    const filePath = path.join(__dirname, 'test-assets/my-module-3/css/main.css');
+    const filePath = path.join(
+        __dirname,
+        'test-assets/my-module-3/css/main.css'
+    );
     const fileRef = 'my-module-3/css/main.css';
 
     const writer = new Writer([filePath]);
@@ -120,7 +133,9 @@ test('Writer processes @import statements', done => {
         const result1 = items[0];
         const result2 = items[1];
 
-        expect(result1.id).toBe(hasher(`my-module-3|1.0.1|${fileRef}|${result1.content}`));
+        expect(result1.id).toBe(
+            hasher(`my-module-3|1.0.1|${fileRef}|${result1.content}`)
+        );
         expect(result1.content).toMatch('my-module-3/main.css');
         expect(result1.content).toMatch('my-module-3/dep.css');
         expect(result1.content).toMatch('dep/main.css');
@@ -133,7 +148,10 @@ test('new Writer([filePath1, filePath2]) ensures correct order', done => {
     expect.assertions(3);
     const filePath1 = path.join(__dirname, 'test-assets/my-module-1/main.css');
     const fileRef1 = 'my-module-1/main.css';
-    const filePath2 = path.join(__dirname, 'test-assets/my-module-2/css/main.css');
+    const filePath2 = path.join(
+        __dirname,
+        'test-assets/my-module-2/css/main.css'
+    );
     const fileRef2 = 'my-module-2/css/main.css';
 
     const writer = new Writer([filePath1, filePath2]);
@@ -148,8 +166,12 @@ test('new Writer([filePath1, filePath2]) ensures correct order', done => {
         const result2 = items[1];
         const result3 = items[2];
 
-        expect(result1.id).toBe(hasher(`my-module-1|1.0.1|${fileRef1}|${result1.content}`));
-        expect(result2.id).toBe(hasher(`my-module-2|1.0.1|${fileRef2}|${result2.content}`));
+        expect(result1.id).toBe(
+            hasher(`my-module-1|1.0.1|${fileRef1}|${result1.content}`)
+        );
+        expect(result2.id).toBe(
+            hasher(`my-module-2|1.0.1|${fileRef2}|${result2.content}`)
+        );
         expect(result3).toBeFalsy();
         done();
     });
@@ -212,10 +234,10 @@ test('new Writer([filePath]) ensures valid filePaths provided in array', () => {
 test('writer emits error', done => {
     expect.assertions(1);
     jest.mock('../lib/util.js', () => ({
-        bundleCssModule () {
+        bundleCssModule() {
             throw new Error();
         },
-        identifyCssModule () {
+        identifyCssModule() {
             throw new Error();
         },
     }));