Add resolve tests

Author: TrySound

README.md

@@ -92,17 +92,17 @@ Checkout [tests](test) for more examples.
 #### `root`
 
 Type: `String`  
-Default: `process.cwd()`
+Default: `process.cwd()` or _dirname of [the postcss `from`](https://github.com/postcss/postcss#node-source)_
 
-Define the root where to resolve path (eg: place where `node_modules` are). Should not be used that much.
+Define the root where to resolve path (eg: place where `node_modules` are). Should not be used that much.  
+_Note: nested `@import` will additionally benefit of the relative dirname of imported files._
 
 #### `path`
 
 Type: `String|Array`  
-Default: `process.cwd()` or _dirname of [the postcss `from`](https://github.com/postcss/postcss#node-source)_
+Default: `[]`
 
-A string or an array of paths in where to look for files.  
-_Note: nested `@import` will additionally benefit of the relative dirname of imported files._
+A string or an array of paths in where to look for files.
 
 #### `transform`
 

index.js

@@ -22,8 +22,12 @@ function AtImport(options) {
     options.path = [ options.path ]
   }
 
+  if (!Array.isArray(options.path)) {
+    options.path = []
+  }
+
   options.path = options.path.map(function(p) {
-    return path.resolve(p)
+    return path.resolve(options.root, p)
   })
 
   return function(styles, result) {

lib/resolve-id.js

@@ -41,7 +41,12 @@ module.exports = function(id, base, options) {
     paths: paths,
     extensions: [ ".css" ],
     packageFilter: function processPackage(pkg) {
-      pkg.main = pkg.style || "index.css"
+      if (pkg.style) {
+        pkg.main = pkg.style
+      }
+      else if (!pkg.main || !/\.css$/.test(pkg.main)) {
+        pkg.main = "index.css"
+      }
       return pkg
     },
   }

test/fixtures/imports/modules/empty/index.css

@@ -0,0 +1 @@
+empty {}

test/fixtures/imports/modules/empty/package.json

@@ -0,0 +1 @@
+{}

test/fixtures/imports/modules/main-js/index.css

@@ -0,0 +1 @@
+main-js {}

test/fixtures/imports/modules/main-js/main.js

@@ -0,0 +1 @@
+module.exports = {}

test/fixtures/imports/modules/main-js/package.json

@@ -0,0 +1,3 @@
+{
+  "main": "main.js"
+}

test/fixtures/imports/modules/main-style/main.css

@@ -0,0 +1 @@
+style-mained {}

test/fixtures/imports/modules/main-style/package.json

@@ -0,0 +1,4 @@
+{
+  "main": "main.css",
+  "style": "style.css"
+}

test/fixtures/imports/modules/main-style/style.css

@@ -0,0 +1 @@
+main-styled {}

test/fixtures/imports/modules/main/main.css

@@ -0,0 +1 @@
+main {}

test/fixtures/imports/modules/main/package.json

@@ -0,0 +1,3 @@
+{
+  "main": "main.css"
+}

test/fixtures/imports/modules/simple/index.css

@@ -0,0 +1 @@
+simple {}

test/fixtures/imports/modules/style/package.json

@@ -0,0 +1,3 @@
+{
+  "style": "style.css"
+}

test/fixtures/imports/modules/style/style.css

@@ -0,0 +1 @@
+style {}

test/fixtures/imports/relative/baz.css

@@ -0,0 +1,2 @@
+@import "imports/foo.css";
+@import "imports/foo-recursive.css";

test/fixtures/imports/relative/import.css

@@ -0,0 +1,3 @@
+foo{}
+bar{}
+foo.recursive{}

test/fixtures/imports/relative/qux.css

@@ -0,0 +1,6 @@
+@import "fixtures/imports/modules/simple";
+@import "fixtures/imports/modules/empty";
+@import "fixtures/imports/modules/style";
+@import "fixtures/imports/modules/main";
+@import "fixtures/imports/modules/main-js";
+@import "fixtures/imports/modules/main-style";

test/fixtures/relative-to-source.css

@@ -0,0 +1,6 @@
+simple {}
+empty {}
+style {}
+main {}
+main-js {}
+main-styled {}

test/fixtures/relative-to-source.expected.css

@@ -0,0 +1,2 @@
+@import "foo.css";
+@import "foo-recursive.css";

test/fixtures/relative.css

@@ -0,0 +1,3 @@
+foo{}
+bar{}
+foo.recursive{}

test/fixtures/relative.expected.css

@@ -0,0 +1,2 @@
+@import "foo.css";
+@import "foo-recursive.css";

test/fixtures/cwd.css renamed to test/fixtures/resolve-cwd.css

@@ -0,0 +1,3 @@
+foo{}
+bar{}
+foo.recursive{}

test/fixtures/cwd.expected.css renamed to test/fixtures/resolve-cwd.expected.css

@@ -0,0 +1,2 @@
+@import "imports/foo.css";
+@import "imports/foo-recursive.css";

test/fixtures/resolve-from.css

@@ -0,0 +1,3 @@
+foo{}
+bar{}
+foo.recursive{}

test/fixtures/resolve-from.expected.css

@@ -27,20 +27,6 @@ test("should ignore & adjust external import", t => {
   return compareFixtures(t, "ignore")
 })
 
-test("should import stylsheets relatively", t => {
-  return compareFixtures(t, "relative")
-})
-
-test("should work without a specified path", t => {
-  return compareFixtures(t, "cwd")
-})
-
-test("should not need `path` option if `source` option has been passed", t => {
-  return compareFixtures(t, "relative-to-source", null, {
-    from: "fixtures/relative-to-source.css",
-  })
-})
-
 test("should be able to consume npm package or local modules", t => {
   return compareFixtures(t, "modules", {
     root: ".",
@@ -100,15 +86,6 @@ test("should contain a correct sourcemap", t => {
     })
 })
 
-test("import relative files using path option only", t => {
-  return postcss()
-    .use(atImport({ path: "fixtures/imports/relative" }))
-    .process(readFileSync("fixtures/imports/relative/import.css"))
-    .then(result => {
-      t.is(result.css, readFileSync("fixtures/imports/bar.css", "utf-8"))
-    })
-})
-
 test("inlined @import should keep PostCSS AST references clean", t => {
   return postcss()
     .use(atImport())

test/fixtures/resolve-local-modules.css

@@ -0,0 +1,42 @@
+import test from "ava"
+import compareFixtures from "./lib/compare-fixtures"
+
+test("should resolve relative to cwd", t => {
+  return compareFixtures(t, "resolve-cwd", {
+    path: null,
+  })
+})
+
+test(`should resolve relative to 'root' option`, t => {
+  return compareFixtures(t, "resolve-root", {
+    root: "fixtures",
+    path: null,
+  })
+})
+
+test(`should resolve relative to postcss 'from' option`, t => {
+  return compareFixtures(t, "resolve-from", {
+    path: null,
+  }, {
+    from: "fixtures/file.css",
+  })
+})
+
+test(`should resolve relative to 'path' which resolved with cwd`, t => {
+  return compareFixtures(t, "resolve-path-cwd", {
+    path: "fixtures/imports",
+  })
+})
+
+test(`should resolve relative to 'path' which resolved with 'root'`, t => {
+  return compareFixtures(t, "resolve-path-root", {
+    root: "fixtures",
+    path: "imports",
+  })
+})
+
+test("should resolve local modules", t => {
+  return compareFixtures(t, "resolve-local-modules", {
+    path: null,
+  })
+})