ember-inspector 4.13.1-alpha.2024.8.9 → 4.13.1-alpha.2025.6.8

package/tests/ember_debug/deprecation-debug-test.js

@@ -35,6 +35,7 @@ module('Ember Debug - Deprecation', function (hooks) {
 
     this.owner.register(
       'route:application',
+      // eslint-disable-next-line ember/no-classic-classes
       Route.extend({
         setupController() {
           EmberDebug.IGNORE_DEPRECATIONS = false;
@@ -59,9 +60,10 @@ module('Ember Debug - Deprecation', function (hooks) {
           });
           EmberDebug.IGNORE_DEPRECATIONS = true;
         },
-      })
+      }),
     );
 
+    // eslint-disable-next-line ember/no-runloop
     run(EmberDebug.port, 'trigger', 'deprecation:watch');
 
     await visit('/');
@@ -74,7 +76,7 @@ module('Ember Debug - Deprecation', function (hooks) {
     assert.strictEqual(
       deprecation.sources.length,
       2,
-      'Correctly separated by source'
+      'Correctly separated by source',
     );
 
     deprecation = deprecations[1];
@@ -87,14 +89,14 @@ module('Ember Debug - Deprecation', function (hooks) {
   });
 
   test('Warns once about deprecations', async function t(assert) {
-    assert.expect(2);
     let count = 0;
+    // eslint-disable-next-line ember/no-runloop
     run(EmberDebug.port, 'trigger', 'deprecation:watch');
     EmberDebug.port.adapter.reopen({
       warn(message) {
         assert.strictEqual(
           message,
-          'Deprecations were detected, see the Ember Inspector deprecations tab for more details.'
+          'Deprecations were detected, see the Ember Inspector deprecations tab for more details.',
         );
         assert.strictEqual(++count, 1, 'Warns once');
       },
@@ -102,6 +104,7 @@ module('Ember Debug - Deprecation', function (hooks) {
 
     this.owner.register(
       'route:application',
+      // eslint-disable-next-line ember/no-classic-classes
       Route.extend({
         setupController() {
           EmberDebug.IGNORE_DEPRECATIONS = false;
@@ -119,7 +122,7 @@ module('Ember Debug - Deprecation', function (hooks) {
           });
           EmberDebug.IGNORE_DEPRECATIONS = true;
         },
-      })
+      }),
     );
 
     await visit('/');

package/tests/ember_debug/ember-debug-test.js

@@ -25,14 +25,12 @@ module('Ember Debug', function (hooks) {
     try {
       EmberDebug.inspect(obj);
       assert.ok(false);
-    } catch (e) {
+    } catch {
       // Intentionally empty
     }
   }
 
   test('EmberDebug#inspect sends inspectable objects', function (assert) {
-    assert.expect(2);
-
     let obj = EmberObject.create();
     EmberDebug.inspect(obj);
     assert.strictEqual(name, 'objectInspector:updateObject');
@@ -46,7 +44,6 @@ module('Ember Debug', function (hooks) {
   });
 
   test('Errors are caught and handled by EmberDebug', async function t(assert) {
-    assert.expect(1);
     const error = new Error('test error');
     EmberDebug.port.on('test:errors', () => {
       throw error;

package/tests/ember_debug/object-inspector-test.js

@@ -1,10 +1,12 @@
-/* eslint-disable ember/avoid-leaking-state-in-ember-objects, ember/no-new-mixins */
+/* eslint-disable ember/avoid-leaking-state-in-ember-objects, ember/no-classic-classes, ember/no-new-mixins, ember/no-runloop */
 import { find, visit } from '@ember/test-helpers';
 import Mixin from '@ember/object/mixin';
+// eslint-disable-next-line ember/no-classic-components
 import Component from '@ember/component';
 import { inspect } from '@ember/debug';
 import { run } from '@ember/runloop';
 import { guidFor } from '@ember/object/internals';
+// eslint-disable-next-line ember/no-computed-properties-in-native-classes
 import EmberObject, { computed } from '@ember/object';
 import MutableArray from '@ember/array/mutable';
 import ArrayProxy from '@ember/array/proxy';
@@ -25,7 +27,7 @@ import Controller from '@ember/controller';
 const GlimmerComponent = (function () {
   try {
     return require('@glimmer/component').default;
-  } catch (e) {
+  } catch {
     // ignore, return undefined
   }
 })();
@@ -89,8 +91,8 @@ module('Ember Debug - Object Inspector', function (hooks) {
       'template:application',
       hbs(
         '<div class="application" style="line-height: normal;">{{outlet}}</div>',
-        { moduleName: 'my-app/templates/application.hbs' }
-      )
+        { moduleName: 'my-app/templates/application.hbs' },
+      ),
     );
     this.owner.register('route:simple', EmberRoute);
     this.owner.register('component:x-simple', Component);
@@ -99,7 +101,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
       hbs`
       {{! template-lint-disable}}
       Simple <Input class="simple-input"/> {{x-simple class="simple-view"}}
-      `
+      `,
     );
 
     objectInspector = EmberDebug.objectInspector;
@@ -138,7 +140,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
     assert.strictEqual(
       firstDetail.properties.length,
       4,
-      'methods are included'
+      'methods are included',
     );
 
     let idProperty = firstDetail.properties[0];
@@ -185,7 +187,6 @@ module('Ember Debug - Object Inspector', function (hooks) {
     }
 
     class Parent {
-      // eslint-disable-next-line constructor-super
       constructor(param) {
         Object.assign(this, param);
       }
@@ -236,7 +237,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
     assert.strictEqual(
       firstDetail.properties.length,
       6,
-      'methods are included'
+      'methods are included',
     );
 
     let objectWithTrackedProperty = firstDetail.properties[0];
@@ -370,7 +371,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
     expectedMixinNames.forEach((expectedMixinName, i) => {
       assert.ok(
         mixinNames[i].includes(expectedMixinName),
-        `${mixinNames[i]} : ${expectedMixinName}`
+        `${mixinNames[i]} : ${expectedMixinName}`,
       );
     });
   });
@@ -463,7 +464,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
     assert.strictEqual(
       details[0].properties.length,
       1,
-      'should not show mixin properties'
+      'should not show mixin properties',
     );
     assert.strictEqual(details[0].properties[0].name, 'ownProp');
 
@@ -471,22 +472,22 @@ module('Ember Debug - Object Inspector', function (hooks) {
     assert.strictEqual(
       details[2].properties.length,
       1,
-      'should only show own mixin properties'
+      'should only show own mixin properties',
     );
     assert.strictEqual(
       details[2].properties[0].value.inspect,
-      inspect('custom1')
+      inspect('custom1'),
     );
 
     assert.strictEqual(details[3].name, 'MyMixin2');
     assert.strictEqual(
       details[3].properties.length,
       1,
-      'should only show own mixin properties'
+      'should only show own mixin properties',
     );
     assert.strictEqual(
       details[3].properties[0].value.inspect,
-      inspect('custom2')
+      inspect('custom2'),
     );
   });
 
@@ -648,7 +649,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
       });
     });
 
-    assert.strictEqual(inspected.get('name'), 'Alex');
+    assert.strictEqual(inspected.name, 'Alex');
 
     assert.strictEqual(message.property, 'name');
     assert.strictEqual(message.value.inspect, inspect('Alex'));
@@ -674,9 +675,9 @@ module('Ember Debug - Object Inspector', function (hooks) {
       });
     });
 
-    assert.strictEqual(inspected.get('date').getFullYear(), 2015);
-    assert.strictEqual(inspected.get('date').getMonth(), 0);
-    assert.strictEqual(inspected.get('date').getDate(), 1);
+    assert.strictEqual(inspected.date.getFullYear(), 2015);
+    assert.strictEqual(inspected.date.getMonth(), 0);
+    assert.strictEqual(inspected.date.getDate(), 1);
 
     assert.strictEqual(message.property, 'date');
     assert.strictEqual(message.value.inspect, inspect(newDate));
@@ -746,42 +747,42 @@ module('Ember Debug - Object Inspector', function (hooks) {
     assert.strictEqual(
       message.details[2].properties.length,
       0,
-      'Correctly skips properties'
+      'Correctly skips properties',
     );
 
     assert.strictEqual(message.details[3].name, 'TestObject');
     assert.strictEqual(
       message.details[3].properties.length,
       3,
-      'Correctly merges properties'
+      'Correctly merges properties',
     );
 
     const toString = message.details[3].properties.find(
-      (p) => p.name === 'toString'
+      (p) => p.name === 'toString',
     );
     const hasChildren = message.details[3].properties.find(
-      (p) => p.name === 'hasChildren'
+      (p) => p.name === 'hasChildren',
     );
     const expensiveProperty = message.details[3].properties.find(
-      (p) => p.name === 'expensiveProperty'
+      (p) => p.name === 'expensiveProperty',
     );
     assert.ok(toString, 'has toString');
     assert.ok(hasChildren, 'has hasChildren');
     assert.strictEqual(
       expensiveProperty.name,
       'expensiveProperty',
-      'property name is correct'
+      'property name is correct',
     );
     assert.strictEqual(
       expensiveProperty.value.isCalculated,
       undefined,
-      'Does not calculate expensive properties'
+      'Does not calculate expensive properties',
     );
 
     assert.notStrictEqual(
       message.details[3].name,
       'MixinToSkip',
-      'Correctly skips mixins'
+      'Correctly skips mixins',
     );
   });
 
@@ -846,7 +847,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
 
     assert.ok(
       message.name.includes('ObjectProxy'),
-      'object name should start with <ObjectProxy:'
+      'object name should start with <ObjectProxy:',
     );
 
     assert.strictEqual(message.details[0].name, 'Basic Info');
@@ -861,38 +862,38 @@ module('Ember Debug - Object Inspector', function (hooks) {
     assert.strictEqual(
       message.details[2].properties.length,
       0,
-      'Correctly skips properties'
+      'Correctly skips properties',
     );
 
     assert.strictEqual(message.details[3].name, 'TestObject');
     assert.strictEqual(
       message.details[3].properties.length,
       3,
-      'Correctly merges properties'
+      'Correctly merges properties',
     );
 
     const hasChildren = message.details[3].properties.find(
-      (p) => p.name === 'hasChildren'
+      (p) => p.name === 'hasChildren',
     );
     const expensiveProperty = message.details[3].properties.find(
-      (p) => p.name === 'expensiveProperty'
+      (p) => p.name === 'expensiveProperty',
     );
     assert.strictEqual(hasChildren.name, 'hasChildren');
     assert.strictEqual(
       expensiveProperty.name,
       'expensiveProperty',
-      'property name is correct'
+      'property name is correct',
     );
     assert.strictEqual(
       expensiveProperty.value.isCalculated,
       undefined,
-      'Does not calculate expensive properties'
+      'Does not calculate expensive properties',
     );
 
     assert.notStrictEqual(
       message.details[3].name,
       'MixinToSkip',
-      'Correctly skips mixins'
+      'Correctly skips mixins',
     );
   });
 
@@ -962,7 +963,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
 
     assert.ok(
       objectInspector.sentObjects[objectId],
-      'Object successfully retained.'
+      'Object successfully retained.',
     );
 
     await visit('/');
@@ -985,7 +986,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
 
     assert.ok(
       objectInspector.sentObjects[objectId],
-      'Object successfully retained.'
+      'Object successfully retained.',
     );
 
     message = await captureMessage('objectInspector:droppedObject', () => {
@@ -997,7 +998,6 @@ module('Ember Debug - Object Inspector', function (hooks) {
     assert.deepEqual(message, { objectId });
   });
 
-  // eslint-disable-next-line qunit/require-expect
   test('Properties ending with `Binding` are skipped', async function (assert) {
     let object = EmberObject.create({
       bar: 'test',
@@ -1011,14 +1011,14 @@ module('Ember Debug - Object Inspector', function (hooks) {
       assert.strictEqual(
         props.length,
         2,
-        'Props should be foo and bar without fooBinding'
+        'Props should be foo and bar without fooBinding',
       );
       assert.strictEqual(props[1].name, 'foo');
     } else {
       assert.strictEqual(
         props.length,
         1,
-        'Props should be only bar without fooBinding, in Ember 3.0+'
+        'Props should be only bar without fooBinding, in Ember 3.0+',
       );
     }
     assert.strictEqual(props[0].name, 'bar');
@@ -1058,7 +1058,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
           }
           return 'bar';
         }),
-      }).create()
+      }).create(),
     );
 
     let message = await inspectObject(object);
@@ -1253,7 +1253,7 @@ module('Ember Debug - Object Inspector', function (hooks) {
         `template:components/foo`,
         hbs('text only', {
           moduleName: 'my-app/templates/components/foo.hbs',
-        })
+        }),
       );
       await visit('/simple');
 
@@ -1280,17 +1280,17 @@ module('Ember Debug - Object Inspector', function (hooks) {
       assert.strictEqual(
         properties.indexOf('bounds'),
         -1,
-        'does not contain bounds'
+        'does not contain bounds',
       );
       assert.strictEqual(
         properties.indexOf('element'),
         -1,
-        'does not contain element'
+        'does not contain element',
       );
       assert.strictEqual(
         properties.indexOf('debugName'),
         -1,
-        'does not contain debugName'
+        'does not contain debugName',
       );
     });
   }