From 8dc9a9e2125b9ef922eb0a129a5da84dec471635 Mon Sep 17 00:00:00 2001
From: kevin940726 <kevin830726@gmail.com>
Date: Sun, 5 Feb 2017 15:10:50 +0800
Subject: [PATCH] write doc, add two more tests

---
 docs/rules/no-unassigned-import.md      | 22 ++++++++++++++++++++++
 tests/src/rules/no-unassigned-import.js | 20 ++++++++++++++++++++
 2 files changed, 42 insertions(+)

diff --git a/docs/rules/no-unassigned-import.md b/docs/rules/no-unassigned-import.md
index 1b25b0169c..3249627670 100644
--- a/docs/rules/no-unassigned-import.md
+++ b/docs/rules/no-unassigned-import.md
@@ -6,11 +6,24 @@ With both CommonJS' `require` and the ES6 modules' `import` syntax, it is possib
 
 This rule aims to remove modules with side-effects by reporting when a module is imported but not assigned.
 
+### Options
+
+This rule supports the following option:
+
+`allow`: An Array of globs. The files that match any of these patterns would be ignored/allowed by the linter. This can be usefull for some build environment (e.g. css-loader in webpack).
+
+Note that the globs start from the where the linter is executed (usually project root), but not from each file that includes the source. Learn more in both the pass and fail examples below.
+
+
 ## Fail
 
 ```js
 import 'should'
 require('should')
+
+// In <PROJECT_ROOT>/src/app.js
+import '../styles/app.css'
+// {"allow": ["styles/*.css"]}
 ```
 
 
@@ -34,4 +47,13 @@ bar(require('foo'))
 require('foo').bar
 require('foo').bar()
 require('foo')()
+
+// With allow option set
+import './style.css' // {"allow": ["**/*.css"]}
+import 'babel-register' // {"allow": ["babel-register"]}
+
+// In <PROJECT_ROOT>/src/app.js
+import './styles/app.css'
+import '../scripts/register.js'
+// {"allow": ["src/styles/**", "**/scripts/*.js"]}
 ```
diff --git a/tests/src/rules/no-unassigned-import.js b/tests/src/rules/no-unassigned-import.js
index b21668fa19..92b2769998 100644
--- a/tests/src/rules/no-unassigned-import.js
+++ b/tests/src/rules/no-unassigned-import.js
@@ -61,6 +61,20 @@ ruleTester.run('no-unassigned-import', rule, {
       code: 'require("./app.css")',
       options: [{ 'allow': ['**/*.css'] }],
     }),
+    test({
+      code: 'import "babel-register"',
+      options: [{ 'allow': ['babel-register'] }],
+    }),
+    test({
+      code: 'import "./styles/app.css"',
+      options: [{ 'allow': ['src/styles/**'] }],
+      filename: path.join(process.cwd(), 'src/app.js'),
+    }),
+    test({
+      code: 'import "../scripts/register.js"',
+      options: [{ 'allow': ['src/styles/**', '**/scripts/*.js'] }],
+      filename: path.join(process.cwd(), 'src/app.js'),
+    }),
   ],
   invalid: [
     test({
@@ -86,5 +100,11 @@ ruleTester.run('no-unassigned-import', rule, {
       options: [{ 'allow': ['**/*.js'] }],
       errors: [error],
     }),
+    test({
+      code: 'import "./styles/app.css"',
+      options: [{ 'allow': ['styles/*.css'] }],
+      filename: path.join(process.cwd(), 'src/app.js'),
+      errors: [error],
+    }),
   ],
 })