Merge pull request #74 from mtsmfm/master

Return fileName to have sass-loader handle dependency correctly

Author: pmowrer

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).
 

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}`);

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).