Options and module options are combined

wad-jet · Jul 13, 2020 · 6a3dcb2 · 6a3dcb2
1 parent e714efa
commit 6a3dcb2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/injection.js b/src/injection.js
@@ -163,7 +163,9 @@ class Injection {
         const container = new Container(options);
 
         if (modules && util.isArray(modules)) {
-            let moduleSetupOptions = options.modulesOptions || { };
+            let moduleSetupOptions = Object.assign({}, options, options.modulesOptions || { });
+            delete options.modulesOptions;
+
             for (let _module of modules) {
                 const moduleSetupFunction = _module.setup;
                 if (_module.options) {

diff --git a/test/setup-module.js b/test/setup-module.js
@@ -112,7 +112,7 @@ describe('Setup modules test', function() {
                 expect(container).to.have.own.property('_dependencies');
                 expect(options).to.be.an('object');
                 expect(options).to.include({ option2: 2 });
-                expect(options).to.not.include({ option1: 1 });
+                expect(options).to.include({ option1: 1 });
             };
             injection.setup(() => { }, { option1: 1, modulesOptions: { option2: 2 }, modules: [ ClassConstructor ] });
         });