fix(tests): use chrome for unit tests instead of phantomjs

[jarifibrahim]

This PR

• Removes phantomjs from package.json
• Installs chrome in the CI docker container

[rohitkrai03]

LGTM

[aptmac]

Perhaps its outside of the scope of this PR in particular, but there are only 3 unit tests actually being run at the moment [0], because the markdown component [1] has fit instead of it on each of it's 3 unit tests .. would you be able to change them and see how the rest of the tests perform? Unless the fit was an intentional addition of #148 ? @sanbornsen

[0] https://ci.centos.org/job/devtools-ngx-widgets/220/console
[1] https://github.com/fabric8-ui/ngx-widgets/blob/master/src/app/markdown/markdown.component.spec.ts

[rohitkrai03]

@jarifibrahim This commit 083bbd3 adds the same functionality i think. Its part of this Angular upgrade PR - #150 which is going to be merged soon. This PR also fixes skipping of unit tests by fit.
cc: @aptmac

[aptmac]

@rohitkrai03 Good find! Yeah, it looks there are existing plans to use headless chrome and re-enable the unit tests.

[jarifibrahim]

@aptmac Since the all the unit tests will be enabled in #150, should we merge this PR?

[joshuawilson]

I think a lot of this will be fixed by the upgrade.

[jarifibrahim]

I think a lot of this will be fixed by the upgrade.

@joshuawilson so should we merge this PR?

[joshuawilson]

working on it now, should be ready today or tomorrow at the latest

[joshuawilson]

oh, sorry, this PR not the upgrade. Let me finish the upgrade and then anything left over you can submit a PR for that. The upgrade and this would be a hard rebase.

[joshuawilson]

This was taken care of in the upgrade. If you still want to remove references to PhantomJS, please submit a new PR.