use map for broader compatibility

Fix emberjs#926
cyril-sf · Mar 30, 2022 · 7e74f59 · 7e74f59
1 parent f35dc4b
commit 7e74f59
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/ember_debug/vendor/startup-wrapper.js b/ember_debug/vendor/startup-wrapper.js
@@ -132,7 +132,7 @@ var EMBER_VERSIONS_SUPPORTED = {{EMBER_VERSIONS_SUPPORTED}};
         return;
       }
 
-      sendAppList(adapterInstance, getApplications().mapBy('name'));
+      sendAppList(adapterInstance, getApplications().map(app => app && app.name));
     });
 
     adapterInstance.onMessageReceived(function(message) {

diff --git a/tests/acceptance/app-picker-test.js b/tests/acceptance/app-picker-test.js
@@ -19,15 +19,15 @@ module('App Picker', function(hooks) {
     });
   });
 
-  test("Both apps show up in picker", async function(assert) {
+  test('Both apps show up in picker', async function(assert) {
     await visit('/component-tree');
 
     assert.dom('.app-picker').exists('App Picker is shown');
     assert.dom(findAll('.app-picker option')[0]).hasText('app-one');
     assert.dom(findAll('.app-picker option')[1]).hasText('app-two');
   });
 
-  test("Clicking each app in the picker switches between them", async function(assert) {
+  test('Clicking each app in the picker switches between them', async function(assert) {
     await visit('/component-tree');
 
     await fillIn('.app-picker select', 'app-two');
-Original file line number
+Diff line change
@@ Expand Up @@
             return;
           }
-          sendAppList(adapterInstance, getApplications().mapBy('name'));
+          sendAppList(adapterInstance, getApplications().map(app => app && app.name));
         });
         adapterInstance.onMessageReceived(function(message) {
@@ Expand Down @@