From 91de383826d16add153292f38c0426ef7c44da17 Mon Sep 17 00:00:00 2001 From: Daniel Compton Date: Fri, 26 Feb 2016 17:00:58 +1300 Subject: [PATCH] fix(runner): Merge config.client.args with client.args provided by run Before this change, calling `karma run` would overwrite any value in config.client.args with the value provided in the `karma run` request, even if that value was an empty array. This commit does a _.merge to merge the two values together. Fixes #1746 --- lib/middleware/runner.js | 13 +++- test/unit/middleware/runner.spec.js | 112 +++++++++++++++++++++++----- 2 files changed, 104 insertions(+), 21 deletions(-) diff --git a/lib/middleware/runner.js b/lib/middleware/runner.js index 7c0d13dbc..22e62dd41 100644 --- a/lib/middleware/runner.js +++ b/lib/middleware/runner.js @@ -4,6 +4,7 @@ * It basically triggers a test run and streams stdout back. */ +var _ = require('lodash') var path = require('path') var helper = require('../helper') var log = require('../logger').create() @@ -47,8 +48,16 @@ var createRunnerMiddleware = function (emitter, fileList, capturedBrowsers, repo }) }) - log.debug('Setting client.args to ', data.args) - config.client.args = data.args + if (_.isEmpty(data.args)) { + log.debug('Ignoring empty client.args from run command') + } else if ((_.isArray(data.args) && _.isArray(config.client.args)) || + (_.isPlainObject(data.args) && _.isPlainObject(config.client.args))) { + log.debug('Merging client.args with ', data.args) + config.client.args = _.merge(config.client.args, data.args) + } else { + log.warn('Replacing client.args with ', data.args, ' as their types do not match.') + config.client.args = data.args + } var fullRefresh = true diff --git a/test/unit/middleware/runner.spec.js b/test/unit/middleware/runner.spec.js index 8e7228aba..4961d2683 100644 --- a/test/unit/middleware/runner.spec.js +++ b/test/unit/middleware/runner.spec.js @@ -5,6 +5,7 @@ import {Promise} from 'bluebird' import Browser from '../../../lib/browser' import BrowserCollection from '../../../lib/browser_collection' import MultReporter from '../../../lib/reporters/multi' +var _ = require('lodash') var createRunnerMiddleware = require('../../../lib/middleware/runner').create var HttpResponseMock = mocks.http.ServerResponse var HttpRequestMock = mocks.http.ServerRequest @@ -144,26 +145,99 @@ describe('middleware.runner', () => { handler(new HttpRequestMock('/__run__'), response, nextSpy) }) - it('should parse body and set client.args', (done) => { - capturedBrowsers.add(new Browser()) - sinon.stub(capturedBrowsers, 'areAllReady', () => true) - - emitter.once('run_start', () => { - expect(config.client.args).to.deep.equal(['arg1', 'arg2']) - done() - }) - - var RAW_MESSAGE = '{"args": ["arg1", "arg2"]}' - - var request = new HttpRequestMock('/__run__', { - 'content-type': 'application/json', - 'content-length': RAW_MESSAGE.length + var clientArgsRuns = [ + { + desc: 'should parse body and set client.args', + expected: ['arg1', 'arg2'], + rawMessage: '{"args": ["arg1", "arg2"]}' + }, + { + desc: 'should set array client args passed by run when there are no existing client.args', + expected: ['my_args'], + rawMessage: '{"args": ["my_args"]}' + }, + { + desc: 'should set object client args passed by run when there are no existing client.args', + expected: {arg2: 'fig', arg3: 'chocolate'}, + rawMessage: '{"args": {"arg2": "fig", "arg3": "chocolate"}}' + }, + { + desc: 'should overwrite empty array client.args when run passes an array for client.args', + expected: ['user_arg1'], + rawMessage: '{"args": ["user_arg1"]}', + existingConfig: [] + }, + { + desc: 'should overwrite empty array client.args when run passes an object for client.args', + expected: {arg2: 'figs', arg3: 'chocolates'}, + rawMessage: '{"args": {"arg2": "figs", "arg3": "chocolates"}}', + existingConfig: [] + }, + { + desc: 'should overwrite empty object client.args when run passes an array for client.args', + expected: ['user_arg'], + rawMessage: '{"args": ["user_arg"]}', + existingConfig: {} + }, + { + desc: 'should not overwrite existing array client.args when run passes an empty array for client.args', + expected: ['user_arg'], + rawMessage: '{"args": []}', + existingConfig: ['user_arg'] + }, + { + desc: 'should not overwrite existing array client.args when run passes an empty object for client.args', + expected: ['user_arg'], + rawMessage: '{"args": {}}', + existingConfig: ['user_arg'] + }, + { + desc: 'should not overwrite existing array client.args when run passes no client.args', + expected: ['user_arg'], + rawMessage: '{}', + existingConfig: ['user_arg'] + }, + { + desc: 'should merge existing client.args with client.args passed by run', + expected: {arg1: 'cherry', arg2: 'fig', arg3: 'chocolate'}, + rawMessage: '{"args": {"arg2": "fig", "arg3": "chocolate"}}', + existingConfig: {arg1: 'cherry', arg2: 'mango'} + }, + { + desc: 'should merge empty client.args with client.args passed by run', + expected: {arg2: 'fig', arg3: 'chocolate'}, + rawMessage: '{"args": {"arg2": "fig", "arg3": "chocolate"}}', + existingConfig: {} + } + ] + + describe('', function () { + clientArgsRuns.forEach(function (run) { + it(run.desc, (done) => { + capturedBrowsers.add(new Browser()) + sinon.stub(capturedBrowsers, 'areAllReady', () => true) + if (run.existingConfig) { + config = _.merge(config, {client: {args: run.existingConfig}}) + } + + emitter.once('run_start', () => { + expect(config.client.args).to.deep.equal(run.expected) + done() + }) + + var RAW_MESSAGE = run.rawMessage + + var request = new HttpRequestMock('/__run__', { + 'content-type': 'application/json', + 'content-length': RAW_MESSAGE.length + }) + + handler(request, response, nextSpy) + + request.emit('data', RAW_MESSAGE) + request.emit('end') + }) }) - - handler(request, response, nextSpy) - - request.emit('data', RAW_MESSAGE) - request.emit('end') }) it('should refresh explicit files if specified', (done) => {