feat(import-destructuring-spacing): add fixer

[rafaelss95]

This adds a fixer for import-destructuring-spacing rule and adds a bunch of new tests.

[wKoza]

Should not we throw an error in this case ?

[rafaelss95]

Well, that’s how the rule works actually. It just enforces that you must leave whitespaces inside of the import statements curly braces, but not the quantity.

[rafaelss95]

So, @wKoza @mgechev, should the current behavior be changed to report failures in case of multiple spaces?

[mgechev]

@rafaelss95 yes, it should be a single space.

[rafaelss95]

Done.

---

Not related to this PR, but don't you think it's a good idea to add coveralls to this repo? It seems to be a really great tool to ensure that we cover more specifics parts of a rule, for example. Also, it can be integrated with CI.

[mgechev]

@rafaelss95 this sounds good to me! @wKoza what are your thoughts?

[rafaelss95]

I had to change the failure message to reflect the new behavior. What do you guys think?

[wKoza]

statements' => statement's ?

[rafaelss95]

Done.

[wKoza]

Just last one comment and It's ok for me.

[intellix]

Just out of curiosity since I didn't see a test for it, what would happen with these (multiple spaces in middle):

import { A,  B, C } from 'somewhere';
import { A, B , C } from 'somewhere';

[rafaelss95]

@intellix Currently, the rule only cares about spaces relative to curly braces.

[wKoza]

LGTM ! thanks @rafaelss95 !