Fix: support negative patterns in files field

Refs stylelint/stylelint#2978

The `files` field of package.json supports negative patterns (e.g. `!**/__tests__`), but `no-unpublished-*` rules have not supported it.
This commit makes those rules supporting the negative patterns.

lib/util/get-npmignore.js

@@ -21,7 +21,7 @@ const getPackageJson = require("./get-package-json")
 //------------------------------------------------------------------------------
 
 const cache = new Cache()
-const SLASH_AT_BEGIN_END = /(?:^\/+|\/+$)/g
+const SLASH_AT_BEGIN_AND_END = /^!?\/+|^!|\/+$/g
 const PARENT_RELATIVE_PATH = /^\.\./
 const NEVER_IGNORED = /^(?:readme\.[^.]*|(?:licen[cs]e|changes|changelog|history)(?:\.[^.]*)?)$/i
 
@@ -96,10 +96,11 @@ function parseWhiteList(files) {
     ig.add("*")
 
     for (const file of files) {
-        if (typeof file === "string") {
-            const normalized = `/${file.replace(SLASH_AT_BEGIN_END, "")}`
-            ig.add(`!${normalized}`)
-            ig.add(`!${normalized}/**`)
+        if (typeof file === "string" && file) {
+            const prefix = file.startsWith("!") ? "" : "!"
+            const body = file.replace(SLASH_AT_BEGIN_AND_END, "")
+            ig.add(`${prefix}/${body}`)
+            ig.add(`${prefix}/${body}/**`)
         }
     }
 

package.json

@@ -26,7 +26,7 @@
     "eslint": ">=3.1.0"
   },
   "dependencies": {
-    "ignore": "^3.3.3",
+    "ignore": "^3.3.6",
     "minimatch": "^3.0.4",
     "resolve": "^1.3.3",
     "semver": "5.3.0"

tests/fixtures/no-unpublished/negative-in-files/package.json

@@ -0,0 +1,15 @@
+{
+    "private": true,
+    "name": "test",
+    "version": "0.0.0",
+    "files": [
+        "lib",
+        "!**/__test__"
+    ],
+    "dependencies": {
+        "aaa": "0.0.0"
+    },
+    "devDependencies": {
+        "bbb": "0.0.0"
+    }
+}

tests/lib/rules/no-unpublished-import.js

@@ -128,6 +128,12 @@ ruleTester.run("no-unpublished-import", rule, {
             code: "import bbb from 'bbb';",
             env: {node: true},
         },
+
+        // Negative patterns in files field.
+        {
+            filename: fixture("negative-in-files/lib/__test__/index.js"),
+            code: "import bbb from 'bbb';",
+        },
     ],
     invalid: [
         {

tests/lib/rules/no-unpublished-require.js

@@ -242,6 +242,13 @@ ruleTester.run("no-unpublished-require", rule, {
             code: "require('bbb');",
             env: {node: true},
         },
+
+        // Negative patterns in files field.
+        {
+            filename: fixture("negative-in-files/lib/__test__/index.js"),
+            code: "require('bbb');",
+            env: {node: true},
+        },
     ],
     invalid: [
         {