discourse/app/assets/javascripts/wizard/test
Jarek Radosz a17d54d0bf
DEV: De-arrowify tests (#11068)
Using arrow functions changes `this` context, which is undesired in tests, e.g. it makes it impossible to setup things like pretender (`this.server`) in `beforeEach` hooks.

Ember guides always use classic functions in examples (e.g. https://guides.emberjs.com/release/testing/test-types/), and that's what it uses in its own test suite, as do various addons and ember apps.

It was also already used in Discourse where `this` was required. Moving forward, it will be needed in more places as we migrate toward ember-cli.

(I might later add a custom rule to eslint-discourse-ember to enforce this)
2020-10-30 17:37:32 +01:00
..
acceptance DEV: De-arrowify tests (#11068) 2020-10-30 17:37:32 +01:00
components REFACTOR: Replace global `find` with `queryAll` 2020-10-29 14:45:51 -04:00
helpers REFACTOR: Import `QUnit` and related helpers rather than globals 2020-10-07 11:50:49 -04:00
models DEV: De-arrowify tests (#11068) 2020-10-30 17:37:32 +01:00
test_helper.js REFACTOR: Test assertions should be imported. 2020-10-28 11:39:06 -04:00
wizard-pretender.js REFACTOR: Support importing of `Pretender` 2020-10-05 14:33:55 -04:00