Skip to content

Commit

Permalink
Better removal of options set by previous tests
Browse files Browse the repository at this point in the history
Fixes #5920
  • Loading branch information
ycombinator authored and epixa committed Jan 16, 2016
1 parent 904d91a commit e2c93e2
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/cli/plugin/__tests__/setting_parser.js
Original file line number Diff line number Diff line change
Expand Up @@ -318,7 +318,7 @@ describe('kibana cli', function () {
describe('remove option', function () {

it('should set settings.action property to "remove"', function () {
options.install = null;
delete options.install;
options.remove = 'package';
parser = settingParser(options);

Expand All @@ -328,7 +328,7 @@ describe('kibana cli', function () {
});

it('should allow one part to the remove parameter', function () {
options.install = null;
delete options.install;
options.remove = 'test-plugin';
parser = settingParser(options);

Expand All @@ -338,7 +338,7 @@ describe('kibana cli', function () {
});

it('should not allow more than one part to the remove parameter', function () {
options.install = null;
delete options.install;
options.remove = 'kibana/test-plugin';
parser = settingParser(options);

Expand All @@ -347,7 +347,7 @@ describe('kibana cli', function () {
});

it('should populate the pluginPath', function () {
options.install = null;
delete options.install;
options.remove = 'test-plugin';
parser = settingParser(options);

Expand Down

0 comments on commit e2c93e2

Please sign in to comment.