Fix relative imports through relative load paths in the Node API (#315)

Closes #314

Author: nex3

CHANGELOG.md

@@ -1,3 +1,10 @@
+## 1.3.1
+
+### Node API
+
+* Fix loading imports relative to stylesheets that were themselves imported
+  though relative include paths.
+
 ## 1.3.0
 
 ### Command-Line Interface

lib/src/importer/node/implementation.dart

@@ -119,7 +119,7 @@ class NodeImporter {
 
     // 3: Filesystem imports relative to [_includePaths].
     for (var includePath in _includePaths) {
-      var result = _tryPath(p.join(includePath, path));
+      var result = _tryPath(p.absolute(p.join(includePath, path)));
       if (result != null) return result;
     }
 

pubspec.yaml

@@ -1,5 +1,5 @@
 name: sass
-version: 1.3.0
+version: 1.3.1
 description: A Sass implementation in Dart.
 author: Dart Team <[email protected]>
 homepage: https://github.com/sass/dart-sass

test/node_api_test.dart

@@ -96,6 +96,22 @@ void main() {
           equalsIgnoringWhitespace('a { b: c; }'));
     });
 
+    // Regression test for #314
+    test(
+        "a file imported through a relative load path supports relative "
+        "imports", () async {
+      var subDir = p.join(sandbox, 'sub');
+      await createDirectory(subDir);
+      await writeTextFile(p.join(subDir, '_test.scss'), '@import "other"');
+
+      await writeTextFile(p.join(subDir, '_other.scss'), 'x {y: z}');
+
+      expect(
+          renderSync(new RenderOptions(
+              data: "@import 'sub/test'", includePaths: [p.relative(sandbox)])),
+          equalsIgnoringWhitespace('x { y: z; }'));
+    });
+
     test("can render the indented syntax", () {
       expect(
           renderSync(