Skip to content

Commit f88ba07

Browse files
committed
Update tests to be compatible with Ember v6
1 parent 41e0bb1 commit f88ba07

2 files changed

Lines changed: 89 additions & 59 deletions

File tree

test-app/tests/integration/setup-rendering-context-test.js

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -65,11 +65,15 @@ const ClickMeButtonComponent = Component.extend({
6565
module('setupRenderingContext "real world"', function (hooks) {
6666
hooks.beforeEach(async function () {
6767
setResolverRegistry({
68-
'component:promise-wrapper': PromiseWrapper,
69-
'template:components/promise-wrapper': PromiseWrapperTemplate,
70-
71-
'component:click-me-button': ClickMeButtonComponent,
72-
'template:components/click-me-button': ClickMeButtonTemplate,
68+
'component:promise-wrapper': setComponentTemplate(
69+
PromiseWrapper,
70+
PromiseWrapperTemplate
71+
),
72+
73+
'component:click-me-button': setComponentTemplate(
74+
ClickMeButtonComponent,
75+
ClickMeButtonTemplate
76+
),
7377
});
7478

7579
await setupContext(this);

test-app/tests/unit/setup-rendering-context-test.js

Lines changed: 80 additions & 54 deletions
Original file line numberDiff line numberDiff line change
@@ -80,13 +80,22 @@ module('setupRenderingContext', function (hooks) {
8080
hooks.beforeEach(async function () {
8181
setResolverRegistry({
8282
'service:foo': Service.extend({ isFoo: true }),
83-
'template:components/template-only': hbs`template-only component here`,
83+
'component:template-only': setComponentTemplate(
84+
hbs`template-only component here`,
85+
class extends Component {}
86+
),
8487
'component:js-only': Component.extend({
8588
classNames: ['js-only'],
8689
}),
8790
'helper:jax': helper(([name]) => `${name}-jax`),
88-
'template:components/outer-comp': hbs`outer{{inner-comp}}outer`,
89-
'template:components/inner-comp': hbs`inner`,
91+
'component:outer-comp': setComponentTemplate(
92+
hbs`outer{{inner-comp}}outer`,
93+
class extends Component {}
94+
),
95+
'component:inner-comp': setComponentTemplate(
96+
hbs`inner`,
97+
class extends Component {}
98+
),
9099
});
91100

92101
await setupContext(this);
@@ -113,10 +122,16 @@ module('setupRenderingContext', function (hooks) {
113122
}
114123

115124
let alternateOwner = await buildEngineOwner(this.owner, {
116-
'template:components/foo': hbs`hello!`,
125+
'component:foo': setComponentTemplate(
126+
hbs`hello!`,
127+
class extends Component {}
128+
),
117129
});
118130

119-
this.owner.register('template:components/foo', hbs`noooooooo!`);
131+
this.owner.register(
132+
'component:foo',
133+
setComponentTemplate(hbs`noooooooo!`, class extends Component {})
134+
);
120135

121136
await render(hbs`<Foo />`, { owner: alternateOwner });
122137

@@ -292,7 +307,10 @@ module('setupRenderingContext', function (hooks) {
292307
});
293308

294309
test('can use the component helper in its layout', async function (assert) {
295-
this.owner.register('template:components/x-foo', hbs`x-foo here`);
310+
this.owner.register(
311+
'component:x-foo',
312+
setComponentTemplate(hbs`x-foo here`, class extends Component {})
313+
);
296314

297315
await render(hbs`{{component 'x-foo'}}`);
298316

@@ -319,15 +337,14 @@ module('setupRenderingContext', function (hooks) {
319337

320338
this.owner.register(
321339
'component:x-foo',
322-
Component.extend({
323-
click() {
324-
assert.ok(true, 'click was fired');
325-
},
326-
})
327-
);
328-
this.owner.register(
329-
'template:components/x-foo',
330-
hbs`<button>Click me!</button>`
340+
setComponentTemplate(
341+
hbs`<button>Click me!</button>`,
342+
Component.extend({
343+
click() {
344+
assert.ok(true, 'click was fired');
345+
},
346+
})
347+
)
331348
);
332349

333350
await render(hbs`{{x-foo}}`);
@@ -345,17 +362,17 @@ module('setupRenderingContext', function (hooks) {
345362

346363
this.owner.register(
347364
'component:x-foo',
348-
Component.extend({
349-
actions: {
350-
clicked() {
351-
assert.ok(true, 'click was fired');
365+
setComponentTemplate(
366+
hbs`<button {{action 'clicked'}}>Click me!</button>`,
367+
368+
Component.extend({
369+
actions: {
370+
clicked() {
371+
assert.ok(true, 'click was fired');
372+
},
352373
},
353-
},
354-
})
355-
);
356-
this.owner.register(
357-
'template:components/x-foo',
358-
hbs`<button {{action 'clicked'}}>Click me!</button>`
374+
})
375+
)
359376
);
360377

361378
await render(hbs`{{x-foo}}`);
@@ -371,10 +388,12 @@ module('setupRenderingContext', function (hooks) {
371388
test('can pass function to be used as a "closure action"', async function (assert) {
372389
assert.expect(2);
373390

374-
this.owner.register('component:x-foo', Component.extend());
375391
this.owner.register(
376-
'template:components/x-foo',
377-
hbs`<button onclick={{action @clicked}}>Click me!</button>`
392+
'component:x-foo',
393+
setComponentTemplate(
394+
hbs`<button onclick={{action @clicked}}>Click me!</button>`,
395+
Component.extend()
396+
)
378397
);
379398

380399
this.set('clicked', () => assert.ok(true, 'action was triggered'));
@@ -393,7 +412,10 @@ module('setupRenderingContext', function (hooks) {
393412

394413
let template = hbs`<button onclick={{action @clicked}}>Click me!</button>`;
395414

396-
this.owner.register('template:components/x-foo', template);
415+
this.owner.register(
416+
'component:x-foo',
417+
setComponentTemplate(template, class extends Component {})
418+
);
397419

398420
this.set('clicked', () => assert.ok(true, 'action was triggered'));
399421
await render(hbs`{{x-foo clicked=this.clicked}}`);
@@ -408,8 +430,11 @@ module('setupRenderingContext', function (hooks) {
408430

409431
test('can update a passed in argument with an <input>', async function (assert) {
410432
this.owner.register(
411-
'template:components/my-input',
412-
hbs`{{input value=@value}}`
433+
'component:my-input',
434+
setComponentTemplate(
435+
hbs`{{input value=@value}}`,
436+
class extends Component {}
437+
)
413438
);
414439

415440
await render(hbs`<MyInput @value={{this.value}} />`);
@@ -436,8 +461,11 @@ module('setupRenderingContext', function (hooks) {
436461

437462
test('it supports dom triggered focus events', async function (assert) {
438463
this.owner.register(
439-
'template:components/x-input',
440-
hbs`<input onblur={{this.onBlur}} onfocusout={{this.onFocus}} />`
464+
'component:x-input',
465+
setComponentTemplate(
466+
hbs`<input onblur={{this.onBlur}} onfocusout={{this.onFocus}} />`,
467+
class extends Component {}
468+
)
441469
);
442470
await render(hbs`<XInput />`);
443471

@@ -463,17 +491,16 @@ module('setupRenderingContext', function (hooks) {
463491
test('two way bound arguments are updated', async function (assert) {
464492
this.owner.register(
465493
'component:my-component',
466-
Component.extend({
467-
actions: {
468-
clicked() {
469-
this.set('foo', 'updated!');
494+
setComponentTemplate(
495+
hbs`<button {{action 'clicked'}}>{{this.foo}}</button>`,
496+
Component.extend({
497+
actions: {
498+
clicked() {
499+
this.set('foo', 'updated!');
500+
},
470501
},
471-
},
472-
})
473-
);
474-
this.owner.register(
475-
'template:components/my-component',
476-
hbs`<button {{action 'clicked'}}>{{this.foo}}</button>`
502+
})
503+
)
477504
);
478505

479506
this.set('foo', 'original');
@@ -496,18 +523,17 @@ module('setupRenderingContext', function (hooks) {
496523
test('two way bound arguments are available after clearRender is called', async function (assert) {
497524
this.owner.register(
498525
'component:my-component',
499-
Component.extend({
500-
actions: {
501-
clicked() {
502-
this.set('foo', 'updated!');
503-
this.set('bar', 'updated bar!');
526+
setComponentTemplate(
527+
hbs`<button {{action 'clicked'}}>{{this.foo}}</button>`,
528+
Component.extend({
529+
actions: {
530+
clicked() {
531+
this.set('foo', 'updated!');
532+
this.set('bar', 'updated bar!');
533+
},
504534
},
505-
},
506-
})
507-
);
508-
this.owner.register(
509-
'template:components/my-component',
510-
hbs`<button {{action 'clicked'}}>{{this.foo}}</button>`
535+
})
536+
)
511537
);
512538

513539
// using two arguments here to ensure the two way binding

0 commit comments

Comments
 (0)