Skip to content

Commit

Permalink
Better failure messages
Browse files Browse the repository at this point in the history
  • Loading branch information
raphinesse committed Sep 4, 2018
1 parent f8c5b51 commit 665a213
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions integration-tests/plugman_uninstall.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ describe('plugman/uninstall', () => {
it('Test 004 : should throw if platform is unrecognized', function () {
return uninstall.uninstallPlatform('atari', project, 'SomePlugin')
.then(function (result) {
fail();
fail('Expected promise to be rejected');
}, function err (errMsg) {
expect(errMsg.toString()).toContain('Platform "atari" not supported.');
});
Expand All @@ -176,7 +176,7 @@ describe('plugman/uninstall', () => {
it('Test 005 : should throw if plugin is missing', function () {
return uninstall.uninstallPlatform('android', project, 'SomePluginThatDoesntExist')
.then(function (result) {
fail();
fail('Expected promise to be rejected');
}, function err (errMsg) {
expect(errMsg.toString()).toContain('Plugin "SomePluginThatDoesntExist" not found. Already uninstalled?');
});
Expand Down Expand Up @@ -207,7 +207,7 @@ describe('plugman/uninstall', () => {

return uninstall.uninstallPlugin('C', plugins_install_dir)
.then(function (result) {
fail();
fail('Expected promise to be rejected');
}, function err (errMsg) {
expect(errMsg.toString()).toEqual('Plugin "C" is required by (A) and cannot be removed (hint: use -f or --force)');
});
Expand Down Expand Up @@ -261,7 +261,7 @@ describe('plugman/uninstall', () => {
it('Test 011 : should throw if platform is unrecognized', function () {
return uninstall('atari', project, 'SomePlugin')
.then(function (result) {
fail();
fail('Expected promise to be rejected');
}, function err (errMsg) {
expect(errMsg.toString()).toContain('Platform "atari" not supported.');
});
Expand All @@ -270,7 +270,7 @@ describe('plugman/uninstall', () => {
it('Test 012 : should throw if plugin is missing', function () {
return uninstall('android', project, 'SomePluginThatDoesntExist')
.then(function (result) {
fail();
fail('Expected promise to be rejected');
}, function err (errMsg) {
expect(errMsg.toString()).toContain('Plugin "SomePluginThatDoesntExist" not found. Already uninstalled?');
});
Expand Down

0 comments on commit 665a213

Please sign in to comment.