Fix(state): state reset --plugins|--platforms was resetting both

[jakub-g]

fixes ionic-team/ionic-cli#399

I think I broke some unit tests, will update them later this week

[jakub-g]

Updated the PR, added the tests

Now the only tests failing for me are ones reported in #32 and #71 that fail as of today in master

[jakub-g]

Any chance to get this merged?
a number of people are affected, see ionic-team/ionic-cli#399

[tlancina]

Hey @jakub-g thanks for the ping, does your workflow work with the Cordova commands (ie. restoring by running prepare), I believe it shouldn't install plugins if they are already installed.

The reason being that we are considering removing the state command: ionic-team/ionic-cli#904. Would love to hear your input on that issue if this affects a particular use case you have.

[jakub-g]

Hello,

sorry for late response.

I didn't know that ionic commands were created due to lack of cordova commands, so we went with the "ionic way" in our project (I thought it was the "blessed way" for whatever reason).

It seems indeed there's some confusion as per this SO question, many people are not sure which way to go.

I'll try to check the "cordova way" in my spare time, for now we've invested in some tooling around ionic scripts and it works fine for us.