Merge pull request #1 from css-modules/simpler-tests

simpler tests

Author: geelen

.gitignore

@@ -1,4 +1,3 @@
 node_modules
 coverage
 lib
-test

package.json

@@ -7,11 +7,10 @@
     "lint": "eslint src",
     "build": "babel --out-dir lib src",
     "watch": "chokidar src -c 'npm run build'",
-    "build-test": "babel --out-dir test test-src",
-    "pretest": "npm run lint && npm run build && npm run build-test",
+    "pretest": "npm run lint && npm run build",
     "test": "mocha",
-    "autotest": "chokidar src test-src -c 'npm test'",
-    "precover": "npm run lint && npm run build && npm run build-test",
+    "autotest": "chokidar src test -c 'npm test'",
+    "precover": "npm run lint && npm run build",
     "cover": "istanbul cover node_modules/mocha/bin/_mocha",
     "travis": "npm run cover -- --report lcovonly",
     "prepublish": "npm run build"

test-src/basic.js

@@ -0,0 +1,45 @@
+"use strict";
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { "default": obj }; }
+
+var _assert = require("assert");
+
+var _assert2 = _interopRequireDefault(_assert);
+
+var _postcss = require("postcss");
+
+var _postcss2 = _interopRequireDefault(_postcss);
+
+var _ = require("../");
+
+var _2 = _interopRequireDefault(_);
+
+var pipeline = (0, _postcss2["default"])([_2["default"]]),
+    check = function check(desc, input, expected, randomStrs) {
+  it(desc, function () {
+    _2["default"].getRandomStr = randomStrs ? function () {
+      return randomStrs.shift();
+    } : _2["default"].defaultRandomStr;
+    _assert2["default"].equal(pipeline.process(input).css, expected);
+  });
+};
+
+describe("processor", function () {
+  check("it should extract an import within a :local", ":local(.exportName) { extends: importName from \"path/library.css\"; other: rule; }", "\n:import(\"path/library.css\") {\n  importName: __tmp_importName_rand0ml0l0l; }\n:local(.exportName) { extends: __tmp_importName_rand0ml0l0l; other: rule; }", ["rand0ml0l0l"]);
+
+  check("it should not care if single-quotes are used", ":local(.exportName) { extends: importName from 'path/library.css'; other: rule; }", "\n:import(\"path/library.css\") {\n  importName: __tmp_importName_rand0ml0l0l; }\n:local(.exportName) { extends: __tmp_importName_rand0ml0l0l; other: rule; }", ["rand0ml0l0l"]);
+
+  check("should import multiple classes on a single line", ":local(.exportName) { extends: importName secondImport from 'path/library.css'; other: rule; }", "\n:import(\"path/library.css\") {\n  importName: __tmp_importName_rand0ml0l0l;\n  secondImport: __tmp_secondImport_rand0ml1l1l; }\n:local(.exportName) { extends: __tmp_importName_rand0ml0l0l __tmp_secondImport_rand0ml1l1l; other: rule; }", ["rand0ml0l0l", "rand0ml1l1l"]);
+
+  check("should consolidate imports by file for multiple files and multiple classes",
+
+  /* INPUT */
+  "\n:local(.exportName) {\n  extends: importName secondImport from 'path/library.css';\n  other: rule;\n}\n:local(.otherExport) {\n  extends: thirdImport from 'path/library.css';\n  extends: otherLibImport from 'path/other-lib.css';\n}",
+
+  /* OUTPUT */
+  "\n:import(\"path/other-lib.css\") {\n  otherLibImport: __tmp_otherLibImport_rand0ml3l3l3;\n}\n:import(\"path/library.css\") {\n  importName: __tmp_importName_rand0ml0l0l;\n  secondImport: __tmp_secondImport_rand0ml1l1l;\n  thirdImport: __tmp_thirdImport_rand0ml2l2l;\n}\n:local(.exportName) {\n  extends: __tmp_importName_rand0ml0l0l __tmp_secondImport_rand0ml1l1l;\n  other: rule;\n}\n:local(.otherExport) {\n  extends: __tmp_thirdImport_rand0ml2l2l;\n  extends: __tmp_otherLibImport_rand0ml3l3l3;\n}",
+  /* RANDOMS */
+  ["rand0ml0l0l", "rand0ml1l1l", "rand0ml2l2l", "rand0ml3l3l3"]);
+
+  check("should ignore imports not inside our rule whitelist", ":local(.exportName) { imports: importName from \"path/library.css\"; something-else: otherLibImport from \"path/other-lib.css\"; }", ":local(.exportName) { imports: importName from \"path/library.css\"; something-else: otherLibImport from \"path/other-lib.css\"; }");
+});

test/basic.js

@@ -0,0 +1,30 @@
+"use strict";
+
+/*globals describe it */
+
+var assert = require("assert");
+var fs = require("fs");
+var path = require("path");
+var postcss = require("postcss");
+var processor = require("../");
+
+var pipeline = postcss([processor]);
+
+function normalize(str) {
+  return str.replace(/\r\n?/g, "\n");
+}
+
+describe("test-cases", function() {
+  var testDir = path.join(__dirname, "test-cases");
+  fs.readdirSync(testDir).forEach(function(testCase) {
+    if(fs.existsSync(path.join(testDir, testCase, "source.css"))) {
+      it("should " + testCase.replace(/-/g, " "), function() {
+        var i = 0;
+        processor.getRandomStr = function() { return "rand" + (i++); };
+        var input = normalize(fs.readFileSync(path.join(testDir, testCase, "source.css"), "utf-8"));
+        var expected = normalize(fs.readFileSync(path.join(testDir, testCase, "expected.css"), "utf-8"));
+        assert.equal(pipeline.process(input).css, expected);
+      });
+    }
+  });
+});

test/test-cases.js

@@ -0,0 +1,17 @@
+
+:import("path/other-lib.css") {
+  otherLibImport: __tmp_otherLibImport_rand3;
+}
+:import("path/library.css") {
+  importName: __tmp_importName_rand0;
+  secondImport: __tmp_secondImport_rand1;
+  thirdImport: __tmp_thirdImport_rand2;
+}
+:local(.exportName) {
+  extends: __tmp_importName_rand0 __tmp_secondImport_rand1;
+  other: rule;
+}
+:local(.otherExport) {
+  extends: __tmp_thirdImport_rand2;
+  extends: __tmp_otherLibImport_rand3;
+}

test/test-cases/import-consolidate/expected.css

@@ -0,0 +1,9 @@
+
+:local(.exportName) {
+  extends: importName secondImport from 'path/library.css';
+  other: rule;
+}
+:local(.otherExport) {
+  extends: thirdImport from 'path/library.css';
+  extends: otherLibImport from 'path/other-lib.css';
+}

test/test-cases/import-consolidate/source.css

@@ -0,0 +1,5 @@
+
+:import("path/library.css") {
+  importName: __tmp_importName_rand0;
+  secondImport: __tmp_secondImport_rand1; }
+:local(.exportName) { extends: __tmp_importName_rand0 __tmp_secondImport_rand1; other: rule; }

test/test-cases/import-multiple-classes/expected.css

@@ -0,0 +1 @@
+:local(.exportName) { extends: importName secondImport from 'path/library.css'; other: rule; }