fix: remove dependencies bug

[stekovinbranturry]

Fixes #23

[tommy-mitchell]

Can you add tests for the await removePackageDependencies({ dependencies: [ 'myDependency' ] }); style usage?

[stekovinbranturry]

Can you add tests for the await removePackageDependencies({ dependencies: [ 'myDependency' ] }); style usage?

added

[sindresorhus]

It would be better to add a new test than modify existing tests.

[stekovinbranturry]

It would be better to add a new test than modify existing tests.

test('async - multiple types') is literally for the case await removePackageDependencies(Record<DependencyKey, Dependency[]>), do we really need another case?

[sindresorhus]

Feels simpler to just do this:

Suggested change

	t.deepEqual(packageJson, pick(fixture, ['name']));
	t.deepEqual(packageJson, {name: fixture.name}));

[sindresorhus]

The test already does too much. Large tests often hide bugs since they do too much. Not a big issue in this case, but having smaller tests would make it clearer. I guess that could be done later though.

[stekovinbranturry]

The test already does too much. Large tests often hide bugs since they do too much. Not a big issue in this case, but having smaller tests would make it clearer. I guess that could be done later though.

yes, it will be better, i've already add a new case