fix: restore loader object in postcss config context (#355)

kisenka · ai · commit 2ff1735200fb · 2018-04-15T21:15:37.000-04:00
* fix: restore loader object in postcss config context

* test: add testcase which checking webpack object exists in config context
diff --git a/lib/index.js b/lib/index.js
@@ -90,6 +90,8 @@ module.exports = function loader (css, map, meta) {
       }
     }
 
+    rc.ctx.webpack = this;
+
     return postcssrc(rc.ctx, rc.path, { argv: false })
   }).then((config) => {
     if (!config) config = {}
diff --git a/test/fixtures/config/context/plugin.js b/test/fixtures/config/context/plugin.js
@@ -0,0 +1,15 @@
+'use strict'
+
+const postcss = require('postcss')
+
+// This plugin creates asset file in webpack compilation
+module.exports = postcss.plugin('plugin', (ctx) => {
+  ctx.webpack._compilation.assets['asset.txt'] = {
+    source() {
+      return '123';
+    },
+    size() {
+      return 0;
+    }
+  }
+})
diff --git a/test/fixtures/config/context/postcss.config.js b/test/fixtures/config/context/postcss.config.js
@@ -0,0 +1,5 @@
+module.exports = (ctx) => ({
+  plugins: [
+    require('./plugin')(ctx)
+  ]
+})
diff --git a/test/options/config.test.js b/test/options/config.test.js
@@ -75,4 +75,24 @@ describe('Options', () => {
         expect(src).toMatchSnapshot()
       })
   })
+
+  test('Pass loader object to config context', () => {
+    const config = {
+      loader: {
+        options: {
+          config: {
+            path: 'test/fixtures/config/context/postcss.config.js'
+          }
+        }
+      }
+    }
+
+    return webpack('css/index.js', config).then((stats) => {
+      const assets = stats.compilation.assets;
+      const expectedAssetName = 'asset.txt';
+
+      expect(expectedAssetName in assets).toBeTruthy();
+      expect(assets[expectedAssetName].source()).toBe('123');
+    })
+  })
 })

Original file line number	Diff line number	Diff line change
`@@ -90,6 +90,8 @@ module.exports = function loader (css, map, meta) {`
`90`	`90`	`}`
`91`	`91`	`}`
`92`	`92`
	`93`	`+ rc.ctx.webpack = this;`
	`94`	`+`
`93`	`95`	`return postcssrc(rc.ctx, rc.path, { argv: false })`
`94`	`96`	`}).then((config) => {`
`95`	`97`	`if (!config) config = {}`