IMSmobile / app / 1105

96%

push

travis-ci

test: remove compileComponents and async from all specs

As stated in the Angular docs, WebPack developers need not call compileComponents because it inlines templates and css as part of the automated build process that precedes running the test.
https://angular.io/guide/testing#compilecomponents

Using async inside the beforeEach seems to be necessary as a workaround for a jasmine issue with browser disconnects in Travis and Windows, see jasmine/jasmine#1327.
For cleaner code this removes the asyncs and pins jasmine to 2.5.2 in package.json.

128 of 153 branches covered (83.66%)

Branch coverage included in aggregate %.

889 of 902 relevant lines covered (98.56%)

43.38 hits per line