Skip to content

Don't use reserved name in jasmine tests

username-removed-636429 requested to merge fix-jasmine-describe into master

What does this MR do?

Jasmine's CI reporter was complaining about a describe block in build_spec.js because it used a reserved name.

Screen_Shot_2017-03-16_at_3.13.28_PM

This changes the offending block from "constructor" to "class constructor".

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

The rake karma job should no longer show the output in the above screen shot.

Why was this MR needed?

It was bugging me...

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Merge request reports