Merge pull request #74 from mtsmfm/master

Return fileName to have sass-loader handle dependency correctly
pmowrer · Sep 28, 2018 · f99f849 · f99f849
2 parents 5badc3a + 3d34986
commit f99f849
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## 4.1.0
+
+- Return `fileName` to have sass-loader handle dependency correctly (#74).
+
 ## 4.0.1
 * Update package.json's main field (#73).
 

diff --git a/src/index.js b/src/index.js
@@ -36,6 +36,7 @@ export default function(options = {}) {
 
       return {
         contents: transformJSONtoSass(json),
+        file: fileName,
       };
     } catch(error) {
       return new Error(`node-sass-json-importer: Error transforming JSON/JSON5 to SASS. Check if your JSON/JSON5 parses correctly. ${error}`);

diff --git a/test/index.js b/test/index.js
@@ -20,6 +20,10 @@ describe('node-sass-json-importer', function() {
     });
 
     expect(result.css.toString()).to.eql(EXPECTATION);
+    expect(result.stats.includedFiles).to.eql([
+      resolve('./test/fixtures/strings/style.scss'),
+      resolve('./test/fixtures/strings/variables.json'),
+    ].sort());
   });
 
   // TODO: Added to verify named exports + CommonJS default export hack (see index.js).