[typescript] Dependency on `enzyme` should be abstracted

Problem description

The typings contain a dependency on enzyme that would require anyone using material-ui with typescript to also install @types/enzyme, and also enzyme for testing.

Steps to reproduce

N/A

Versions

• Material-UI: 1.0.0-beta.6
• React: N/A
• Browser: N/A

Description

I would like to suggest the testing utils specific to enzyme be moved into another package (e.g. material-ui-enzyme). Any testing utilities that come with material-ui should probably only depend on React itself so as to leave developers the freedom of choice with it comes to testing.

Images & references

N/A

[oliviertassinari]

Alright, I’m going to close the issue. To sum up:

• We want to enzyme dependency of the test helpers to be explicit
• Some might find the test helpers useful, so we keep them
• It would be better in terms of package delivery to ship the tests helpers into their own material-ui-test package (so we can have a finer grain control on the dependency and peer dependency). This is something we started discussing here #7839. But, this has an important maintenance cost. For now, I do think that it would be better to wait and see if that worth it.

Thanks for the discussion, we can always reopen the issue if you don’t agree, or want to push into a different direction.

[oliviertassinari]

Also, you don’t need to install enzyme only the typings.

👍