test: some more tests

Author: anikethsaha

.github/workflows/release.yml

@@ -16,11 +16,17 @@ jobs:
           node_version: 12
       - name: Install
         run: yarn install --frozen-lockfile
-      # - name: Prepare Npm
-      #   run: sh npm.sh
-      #   env:
-      #     REGISTRY_AUTH_TOKEN: ${{ secrets.REGISTRY_AUTH_TOKEN }}
-      #     REGISTRY_URL: registry.npmjs.org
+      - name: Prepare Npm
+        run: sh npm.sh
+        env:
+          REGISTRY_AUTH_TOKEN: ${{ secrets.REGISTRY_AUTH_TOKEN }}
+          REGISTRY_URL: registry.npmjs.org
+      - name: Test
+        run: npm run test
+        env:
+          REGISTRY_AUTH_TOKEN: ${{ secrets.REGISTRY_AUTH_TOKEN }}
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Github already has this token by default in repo
+          REGISTRY_URL: registry.npmjs.org
       # - name: Run
       #   run: npm run build
       #   env:

.gitignore

@@ -1,3 +1,5 @@
 cssnano
 node_modules
-.npmrc
+.npmrc
+test/fixture/output.css
+test/fixture/package-lock.json

package.json

@@ -14,6 +14,7 @@
   },
   "devDependencies": {
     "jest": "^26.1.0",
+    "postcss-cli": "^7.1.1",
     "tmp": "^0.2.1",
     "verdaccio": "^4.6.2",
     "verdaccio-auth-memory": "^9.5.0",

test/fixture/input.css

@@ -0,0 +1,27 @@
+/* normalize selectors */
+h1::before,
+h1:before {
+  /* reduce shorthand even further */
+  margin: 10px 20px 10px 20px;
+  /* reduce color values */
+  color: #ff0000;
+  /* remove duplicated properties */
+  font-weight: 400;
+  font-weight: 400;
+  /* reduce position values */
+  background-position: bottom right;
+  /* normalize wrapping quotes */
+  quotes: "«" "»";
+  /* reduce gradient parameters */
+  background: linear-gradient(
+    to bottom,
+    #ffe500 0%,
+    #ffe500 50%,
+    #121 50%,
+    #121 100%
+  );
+  /* replace initial values */
+  min-width: initial;
+}
+/* correct invalid placement */
+@charset "utf-8";

test/fixture/package.json

@@ -0,0 +1,5 @@
+{
+  "dependencies": {
+    "cssnano": "^4.0.0-nightly.0.7.8"
+  }
+}

test/fixture/postcss.config.js

@@ -0,0 +1,7 @@
+module.exports = {
+  plugins: [
+    require("cssnano")({
+      preset: "default"
+    })
+  ]
+};

test/index.test.js

@@ -1,11 +1,10 @@
 const path = require("path");
+const fs = require("fs");
 const shell = require("shelljs");
 const { spawnRegistry } = require("./helper/registry");
-// const { runNmp } = require("./helper/npm");
-// const { runCli } = require("./helper/cli");
 const rimraf = require("rimraf");
-const { getTempFolder } = require("./helper/fs-utils");
 const tmp = require("tmp");
+const { runCli } = require("./helper/cli");
 const { tag } = require("../versions");
 const publish = require("../publish");
 
@@ -16,10 +15,8 @@ const registryUrl = "http://localhost:4873/";
 
 describe("E2E cssnano ", () => {
   let childFork;
-  const folder = getTempFolder();
-  let tmpDir;
+
   beforeAll(async () => {
-    // tmpDir = tmp.dirSync();
     childFork = await spawnRegistry();
     try {
       await publish(registryUrl);
@@ -32,9 +29,7 @@ describe("E2E cssnano ", () => {
     if (childFork) {
       childFork.kill();
     }
-    if (tmpDir) {
-      tmpDir.removeCallback();
-    }
+
     rimraf.sync("cssnano");
   });
 
@@ -45,4 +40,32 @@ describe("E2E cssnano ", () => {
     ).toBe(0);
     done();
   });
+
+  it(
+    "should minify the input using postcss cli and cssnano" + tag,
+    async done => {
+      await shell.cd(__dirname + "/fixture");
+      await shell.exec(
+        "npm install cssnano@" + tag + " --registry " + registryUrl
+      );
+
+      const cmdOutput = await runCli(
+        path.resolve(__dirname, "../node_modules", ".bin", "postcss"),
+        [
+          path.resolve(__dirname, "./fixture/input.css"),
+          "-o",
+          path.resolve(__dirname, "./fixture/output.css")
+        ]
+      );
+      expect(
+        fs.existsSync(path.resolve(__dirname, "./fixture/output.css"))
+      ).toBe(true);
+      const outputCode = fs.readFileSync(
+        path.resolve(__dirname, "./fixture/output.css"),
+        "utf-8"
+      );
+      expect(outputCode.split("\n").length).toBe(1);
+      done();
+    }
+  );
 });