test: use snapshot-diffs (#572)

* test: add snapshot-diff

Also restructure how custom snapshot matchers are loaded

* test: rollup watch test using snapshot diffs

* test: change read-dir.js output

... and all the snapshots that use that output

Author: tivac

jest.config.js

@@ -17,6 +17,16 @@ module.exports = {
                 "/output/",
                 "/specimens/",
             ],
+
+            setupFilesAfterEnv : [
+                "<rootDir>/packages/test-utils/expect/toMatchDiffSnapshot.js",
+                "<rootDir>/packages/test-utils/expect/toMatchRollupSnapshot.js",
+                "<rootDir>/packages/test-utils/expect/toMatchRollupCodeSnapshot.js",
+            ],
+
+            snapshotSerializers : [
+                require.resolve("snapshot-diff/serializer.js"),
+            ],
         },
         {
             displayName : "lint",

package-lock.json

@@ -44,6 +44,7 @@
     "rollup": "^1.7.0",
     "rollup-plugin-svelte": "^5.0.3",
     "shelljs": "^0.8.3",
+    "snapshot-diff": "^0.5.1",
     "sugarss": "^2.0.0",
     "svelte": "^2.16.1",
     "watchify": "^3.11.1",

package.json

@@ -5,11 +5,9 @@ const { rollup } = require("rollup");
 
 const shell = require("shelljs");
 
-const prefix  = require("@modular-css/test-utils/prefix.js")(__dirname);
-const namer   = require("@modular-css/test-utils/namer.js");
-const logs    = require("@modular-css/test-utils/logs.js");
-
-require("@modular-css/test-utils/rollup-build-snapshot.js");
+const prefix = require("@modular-css/test-utils/prefix.js")(__dirname);
+const namer  = require("@modular-css/test-utils/namer.js");
+const logs   = require("@modular-css/test-utils/logs.js");
 
 const css = require("@modular-css/rollup");
 

packages/rollup-rewriter/test/rewriter.test.js

@@ -1,62 +1,47 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`/rollup.js case sensitivity tests should remove repeated references that point at the same files 1`] = `
-Array [
-  Object {
-    "file": "assets/foo.css",
-    "text": "/* packages/rollup/test/specimens/casing/bar.css */
+Object {
+  "assets/foo.css": "/* packages/rollup/test/specimens/casing/bar.css */
 .mc79ab9c62_bar {
     display: none;
 }
 /* packages/rollup/test/specimens/casing/foo.css */
 .mc3dd08d27_foo {
     color: #F00;
 }",
-  },
-  Object {
-    "file": "main.js",
-    "text": "var bar = \\"mc79ab9c62_bar\\";
+  "main.js": "var bar = \\"mc79ab9c62_bar\\";
 
 var foo = \\"mc79ab9c62_bar mc3dd08d27_foo\\";
 
 var bar$1 = \\"mc79ab9c62_bar\\";
 
 console.log({ foo, bar: bar$1, bar2: bar });
 ",
-  },
-]
+}
 `;
 
 exports[`/rollup.js should accept an existing processor instance (no css in bundle) 1`] = `
-Array [
-  Object {
-    "file": "fake.css",
-    "text": "/* packages/rollup/test/specimens/fake.css */
+Object {
+  "fake.css": "/* packages/rollup/test/specimens/fake.css */
 .fake {
     color: yellow;
 }",
-  },
-]
+}
 `;
 
 exports[`/rollup.js should accept an existing processor instance 1`] = `
-Array [
-  Object {
-    "file": "fake.css",
-    "text": "/* packages/rollup/test/specimens/fake.css */
+Object {
+  "fake.css": "/* packages/rollup/test/specimens/fake.css */
 .fake {
     color: yellow;
 }",
-  },
-  Object {
-    "file": "simple.css",
-    "text": "/* packages/rollup/test/specimens/simple.css */
+  "simple.css": "/* packages/rollup/test/specimens/simple.css */
 .fooga {
     color: red;
 }
 ",
-  },
-]
+}
 `;
 
 exports[`/rollup.js should allow disabling of named exports 1`] = `
@@ -81,63 +66,45 @@ console.log(fooga);
 `;
 
 exports[`/rollup.js should correctly handle hashed output 1`] = `
-Array [
-  Object {
-    "file": "assets/simple-8d1bcf7b.css",
-    "text": "/* packages/rollup/test/specimens/simple.css */
+Object {
+  "assets/simple-8d1bcf7b.css": "/* packages/rollup/test/specimens/simple.css */
 .fooga {
     color: red;
 }
 ",
-  },
-  Object {
-    "file": "hashes.js",
-    "text": "var css = {
+  "hashes.js": "var css = {
     \\"str\\": \\"\\\\\\"string\\\\\\"\\",
     \\"fooga\\": \\"fooga\\"
 };
 
 console.log(css);
 ",
-  },
-]
+}
 `;
 
 exports[`/rollup.js should correctly handle hashed output with external source maps & json files 1`] = `
-Array [
-  Object {
-    "file": "assets/exports-24307d7d.json",
-    "text": "{
+Object {
+  "assets/exports-24307d7d.json": "{
     \\"packages/rollup/test/specimens/simple.css\\": {
         \\"str\\": \\"\\\\\\"string\\\\\\"\\",
         \\"fooga\\": \\"fooga\\"
     }
 }",
-  },
-  Object {
-    "file": "assets/simple-8d1bcf7b.css",
-    "text": "/* packages/rollup/test/specimens/simple.css */
+  "assets/simple-8d1bcf7b.css": "/* packages/rollup/test/specimens/simple.css */
 .fooga {
     color: red;
 }
 
 /*# sourceMappingURL=simple-8d1bcf7b.css.map */",
-  },
-  Object {
-    "file": "assets/simple-8d1bcf7b.css.map",
-    "text": "{\\"version\\":3,\\"sources\\":[\\"../../../specimens/simple.css\\"],\\"names\\":[],\\"mappings\\":\\"AAAA,8CAAA;AAEA;IACI,UAAU;AACd\\",\\"file\\":\\"simple-[hash].css\\",\\"sourcesContent\\":[\\"@value str: \\\\\\"string\\\\\\";\\\\n\\\\n.fooga {\\\\n    color: red;\\\\n}\\\\n\\"]}",
-  },
-  Object {
-    "file": "hashes.js",
-    "text": "var css = {
+  "assets/simple-8d1bcf7b.css.map": "{\\"version\\":3,\\"sources\\":[\\"../../../specimens/simple.css\\"],\\"names\\":[],\\"mappings\\":\\"AAAA,8CAAA;AAEA;IACI,UAAU;AACd\\",\\"file\\":\\"simple-[hash].css\\",\\"sourcesContent\\":[\\"@value str: \\\\\\"string\\\\\\";\\\\n\\\\n.fooga {\\\\n    color: red;\\\\n}\\\\n\\"]}",
+  "hashes.js": "var css = {
     \\"str\\": \\"\\\\\\"string\\\\\\"\\",
     \\"fooga\\": \\"fooga\\"
 };
 
 console.log(css);
 ",
-  },
-]
+}
 `;
 
 exports[`/rollup.js should correctly pass to/from params for relative paths 1`] = `
@@ -416,23 +383,17 @@ console.log(style);
 `;
 
 exports[`/rollup.js should output unreferenced CSS 1`] = `
-Array [
-  Object {
-    "file": "fake.css",
-    "text": "/* packages/rollup/test/specimens/fake.css */
+Object {
+  "fake.css": "/* packages/rollup/test/specimens/fake.css */
 .fake {
     color: yellow;
 }",
-  },
-  Object {
-    "file": "simple.css",
-    "text": "/* packages/rollup/test/specimens/simple.css */
+  "simple.css": "/* packages/rollup/test/specimens/simple.css */
 .fooga {
     color: red;
 }
 ",
-  },
-]
+}
 `;
 
 exports[`/rollup.js should provide named exports 1`] = `