From 7b4e7cfc42ddb94da46f4af55787af753be62d43 Mon Sep 17 00:00:00 2001 From: Honza Javorek Date: Mon, 13 Aug 2018 11:09:13 +0200 Subject: [PATCH] test: remove redundant tests They test something already tested and they should actually test the interplay between 'prompt' and 'applyAnswers' than the 'applyAnswers' alone. This should be handled by integration tests in the future. --- test/unit/init/applyAnswers-test.js | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/test/unit/init/applyAnswers-test.js b/test/unit/init/applyAnswers-test.js index 05e874814..971bcebe3 100644 --- a/test/unit/init/applyAnswers-test.js +++ b/test/unit/init/applyAnswers-test.js @@ -54,22 +54,10 @@ describe('init._applyAnswers()', () => { const config = applyAnswers(createConfig(), { apiaryApiKey: '1234' }); assert.equal(config.custom.apiaryApiKey, '1234'); }); - it('keeps the Apiary API key if already present in the config', () => { - const config = applyAnswers(Object(createConfig()), { - custom: { apiaryApiKey: '1234' } - }); - assert.equal(config.custom.apiaryApiKey, '1234'); - }); it('sets the Apiary API name if provided', () => { const config = applyAnswers(createConfig(), { apiaryApiName: 'myproject' }); assert.equal(config.custom.apiaryApiName, 'myproject'); }); - it('keeps the Apiary API name if already present in the config', () => { - const config = applyAnswers(Object(createConfig()), { - custom: { apiaryApiName: 'myproject' } - }); - assert.equal(config.custom.apiaryApiName, 'myproject'); - }); it('creates selected CI configuration if asked', () => { applyAnswers(createConfig(), { createCI: 'wercker' }, { ci }); assert.isTrue(ci.wercker.calledOnce);