fix: passing through constructorMixin in configure()

Author: analog-nico

configure/request-next.js

@@ -35,7 +35,8 @@ module.exports = function (options) {
 
 
     var plumbing = core({
-        PromiseImpl: options.PromiseImpl
+        PromiseImpl: options.PromiseImpl,
+        constructorMixin: options.constructorMixin
     });
 
     return function (requestOptions) {

configure/request2.js

@@ -24,7 +24,8 @@ module.exports = function (options) {
 
 
     var plumbing = core({
-        PromiseImpl: options.PromiseImpl
+        PromiseImpl: options.PromiseImpl,
+        constructorMixin: options.constructorMixin
     });
 
 

test/spec/request-next.js

@@ -92,6 +92,34 @@ describe('Promise-Core for Request@next', function () {
 
         });
 
+        it('should forward the constructorMixin', function () {
+
+            var mixinCalled = false; // eslint-disable-line no-unused-vars
+
+            var request = api({
+                type: 'basic',
+                define: {
+                    request: configure({
+                        client: client,
+                        PromiseImpl: Bluebird,
+                        expose: [
+                            'then',
+                            'catch'
+                        ],
+                        constructorMixin: function () {
+                            mixinCalled = true;
+                        }
+                    })
+                }
+            });
+
+            return request('http://localhost:4000') // not started yet so expecting ECONNREFUSED
+                .catch(function () {
+                    expect(mixinCalled).to.eql(true);
+                });
+
+        });
+
     });
 
     /**

test/spec/request2.js

@@ -92,6 +92,30 @@ describe('Promise-Core for Request@2', function () {
 
         });
 
+        it('should forward the constructorMixin', function () {
+
+            var mixinCalled = false; // eslint-disable-line no-unused-vars
+
+            var request = stealthyRequire(require.cache, function () {
+                return require('request');
+            });
+
+            configure({
+                request: request,
+                expose: ['then', 'catch'],
+                PromiseImpl: Bluebird,
+                constructorMixin: function () {
+                    mixinCalled = true;
+                }
+            });
+
+            return request('http://localhost:4000') // not started yet so expecting ECONNREFUSED
+                .catch(function () {
+                    expect(mixinCalled).to.eql(true);
+                });
+
+        });
+
     });
 
     /**