Skip to content

Commit 5108d78

Browse files
committed
fix(dependencies): avoid failure when no dependencies need to be removed
1 parent f29e7bb commit 5108d78

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

src/dependencies/remover.js

+6-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
import {execa} from 'execa';
2+
import {info} from '@travi/cli-messages';
23

34
export default async function ({packageManager, dependencies}) {
4-
await execa(packageManager, ['remove', ...dependencies]);
5+
if (dependencies.length) {
6+
info('Removing dependencies dependencies', {level: 'secondary'});
7+
8+
await execa(packageManager, ['remove', ...dependencies]);
9+
}
510
}

src/dependencies/remover.test.js

+9
Original file line numberDiff line numberDiff line change
@@ -16,4 +16,13 @@ describe('dependency remover', () => {
1616

1717
expect(execa).toHaveBeenCalledWith(packageManager, ['remove', ...dependencies]);
1818
});
19+
20+
it('should not attempt to remove when no dependencies are provided', async () => {
21+
const packageManager = any.word();
22+
const dependencies = [];
23+
24+
await removeDependencies({packageManager, dependencies});
25+
26+
expect(execa).not.toHaveBeenCalled();
27+
});
1928
});

0 commit comments

Comments
 (0)