Skip to content

Commit 96a69bd

Browse files
committed
Remove assert.expect calls
This is no longer recommended and is a linting violation. Removed them.
1 parent 4a84d5f commit 96a69bd

3 files changed

Lines changed: 0 additions & 14 deletions

File tree

tests/integration/modifiers/did-insert-test.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@ module('Integration | Modifier | did-insert', function (hooks) {
1313
setupRenderingTest(hooks);
1414

1515
test('it basically works', async function (assert) {
16-
assert.expect(2);
17-
1816
this.someMethod = (element) => {
1917
assert.strictEqual(element.tagName, 'DIV', 'correct element tagName');
2018
assert.dom(element).hasAttribute('data-foo', 'some-thing');
@@ -25,8 +23,6 @@ module('Integration | Modifier | did-insert', function (hooks) {
2523
});
2624

2725
test('it can accept arguments', async function (assert) {
28-
assert.expect(4);
29-
3026
this.someMethod = (element, positional, named) => {
3127
assert.strictEqual(element.tagName, 'DIV', 'correct element tagName');
3228
assert.dom(element).hasAttribute('data-foo', 'some-thing');
@@ -45,8 +41,6 @@ module('Integration | Modifier | did-insert', function (hooks) {
4541
});
4642

4743
test('it is not invoked again when arguments change', async function (assert) {
48-
assert.expect(4);
49-
5044
this.someMethod = (element, positional, named) => {
5145
assert.strictEqual(element.tagName, 'DIV', 'correct element tagName');
5246
assert.dom(element).hasAttribute('data-foo', 'some-thing');

tests/integration/modifiers/did-update-test.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,6 @@ module('Integration | Modifier | did-update', function (hooks) {
1010
setupRenderingTest(hooks);
1111

1212
test('it basically works', async function (assert) {
13-
assert.expect(4);
14-
1513
this.someMethod = (element, positional, named) => {
1614
assert.strictEqual(element.tagName, 'DIV', 'correct element tagName');
1715
assert.dom(element).hasAttribute('data-foo', 'some-thing');
@@ -31,8 +29,6 @@ module('Integration | Modifier | did-update', function (hooks) {
3129
// only run the next test where @tracked is present
3230
if (macroCondition(dependencySatisfies('ember-source', '>= 3.12.0'))) {
3331
test('it consumes tracked properties without re-invoking', async function (assert) {
34-
assert.expect(1);
35-
3632
class Context {
3733
@tracked boundValue = 'initial';
3834
@tracked secondaryValue = 'initial';

tests/integration/modifiers/will-destroy-test.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@ module('Integration | Modifier | will-destroy', function (hooks) {
77
setupRenderingTest(hooks);
88

99
test('it basically works', async function (assert) {
10-
assert.expect(2);
11-
1210
this.someMethod = (element) => {
1311
assert.strictEqual(element.tagName, 'DIV', 'correct element tagName');
1412
assert.dom(element).hasAttribute('data-foo', 'some-thing');
@@ -24,8 +22,6 @@ module('Integration | Modifier | will-destroy', function (hooks) {
2422
});
2523

2624
test('it can accept arguments', async function (assert) {
27-
assert.expect(4);
28-
2925
this.someMethod = (element, positional, named) => {
3026
assert.strictEqual(element.tagName, 'DIV', 'correct element tagName');
3127
assert.dom(element).hasAttribute('data-foo', 'some-thing');

0 commit comments

Comments
 (0)