Skip to content

Commit

Permalink
Fix no-new-mixins
Browse files Browse the repository at this point in the history
Not sure why these weren't caught by `yarn lint:js`, but they were
showing up in the builds (but only in ember-try?)
  • Loading branch information
chancancode committed Dec 14, 2019
1 parent 8861e83 commit 0230f6e
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions tests/ember_debug/object-inspector-test.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@

/* eslint-disable ember/no-new-mixins */
import { find, settled, visit } from '@ember/test-helpers';
import Mixin from '@ember/object/mixin';
import Component from '@ember/component';
Expand Down Expand Up @@ -222,7 +224,6 @@ module('Ember Debug - Object Inspector', function(hooks) {
});

test('Correct mixin order with es6 class', function (assert) {

class MyMixinClass extends Mixin {
toString() {
return 'MyMixin';
Expand Down Expand Up @@ -329,14 +330,11 @@ module('Ember Debug - Object Inspector', function(hooks) {
});

test('Correct mixin properties', function (assert) {

// eslint-disable-next-line ember/no-new-mixins
class MyMixin extends Mixin {
toString() {
return 'MyMixin1';
}
}
// eslint-disable-next-line ember/no-new-mixins
class MyMixin2 extends Mixin {
toString() {
return 'MyMixin2';
Expand Down Expand Up @@ -562,7 +560,6 @@ module('Ember Debug - Object Inspector', function(hooks) {
});

test('Property grouping can be customized using _debugInfo', function(assert) {
// eslint-disable-next-line ember/no-new-mixins
let mixinToSkip = Mixin.create({});

let Inspected = EmberObject.extend(mixinToSkip, {
Expand Down Expand Up @@ -627,7 +624,6 @@ module('Ember Debug - Object Inspector', function(hooks) {
});

test('Property grouping can be customized using _debugInfo when using Proxy', function(assert) {

class MyMixin extends Mixin {
toString() {
return 'MixinToSkip';
Expand Down

0 comments on commit 0230f6e

Please sign in to comment.