Skip to content

Commit 2240194

Browse files
author
Robert Jackson
committed
Drop element importable helper for now.
Might bring it back later in a different incarnation, but removing for now.
1 parent da3e65d commit 2240194

File tree

3 files changed

+2
-17
lines changed

3 files changed

+2
-17
lines changed

addon-test-support/index.js

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,7 @@ export { default as TestModuleForModel } from './legacy-0-6-x/test-module-for-mo
77
export { setResolver } from './resolver';
88
export { default as setupContext, getContext, setContext, unsetContext } from './setup-context';
99
export { default as teardownContext } from './teardown-context';
10-
export {
11-
default as setupRenderingContext,
12-
render,
13-
clearRender,
14-
element,
15-
} from './setup-rendering-context';
10+
export { default as setupRenderingContext, render, clearRender } from './setup-rendering-context';
1611
export { default as teardownRenderingContext } from './teardown-rendering-context';
1712

1813
import Ember from 'ember';

addon-test-support/setup-rendering-context.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,6 @@ export function clearRender() {
2929
return context.clearRender();
3030
}
3131

32-
export let element = undefined;
33-
3432
/*
3533
* Responsible for:
3634
*
@@ -78,7 +76,7 @@ export default function(context) {
7876
outlets: {},
7977
};
8078

81-
let hasRendered;
79+
let element, hasRendered;
8280
let templateId = 0;
8381

8482
if (hasOutletTemplate) {

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

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import {
1010
teardownRenderingContext,
1111
render,
1212
clearRender,
13-
element,
1413
} from 'ember-test-helpers';
1514
import hasEmberVersion from 'ember-test-helpers/has-ember-version';
1615
import hasjQuery from '../helpers/has-jquery';
@@ -313,13 +312,6 @@ module('setupRenderingContext', function(hooks) {
313312
assert.equal(this.element.textContent, 'yippie!!');
314313
});
315314

316-
test('imported `element` can be used instead of this.element', async function(assert) {
317-
await this.render(hbs`yippie!!`);
318-
319-
assert.equal(element.textContent, 'yippie!!');
320-
assert.equal(element, this.element);
321-
});
322-
323315
test('imported clearRender can be used instead of this.clearRender', async function(assert) {
324316
let testingRootElement = document.getElementById('ember-testing');
325317

0 commit comments

Comments
 (0)