Skip to content

ensure karma fails if it cannot load all spec files

username-removed-636429 requested to merge fix-ignored-spec-failures into master

What does this MR do?

Ensures that karma will exit with status 1 if any *_spec.js files fail to compile correctly with webpack rather than ignoring them.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

We had some frontend tests in a MR which should have failed but were ignored due to this bug. (see !8844 (merged) & !9032 (merged))

Screenshots (if relevant)

Before:

Screen_Shot_2017-02-07_at_11.49.27_AM

After:

Screen_Shot_2017-02-07_at_11.48.17_AM

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Merge request reports