allow functions option to be a function of loaderContext; add test

webpack-contrib · Dec 16, 2018 · f4ac3db · f4ac3db
1 parent 2d6045b
commit f4ac3db
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/lib/normalizeOptions.js b/lib/normalizeOptions.js
@@ -23,6 +23,11 @@ function normalizeOptions(loaderContext, content, webpackImporter) {
   const options = cloneDeep(utils.getOptions(loaderContext)) || {};
   const { resourcePath } = loaderContext;
 
+  // allow opt.functions to be configured WRT loaderContext
+  if (typeof options.functions === 'function') {
+    options.functions = options.functions(loaderContext);
+  }
+
   let { data } = options;
 
   if (typeof options.data === 'function') {

diff --git a/test/index.test.js b/test/index.test.js
@@ -190,6 +190,13 @@ implementations.forEach((implementation) => {
             execTest('custom-functions', {
               functions: customFunctions(implementation),
             }));
+          it('should expose custom functions if the option is a function', () =>
+            execTest('custom-functions', {
+              functions: (loaderContext) => {
+                should.exist(loaderContext);
+                return customFunctions(implementation);
+              },
+            }));
         });
         describe('prepending data', () => {
           it('should extend the data option if present and it is string', () =>