chore(deps): upgrade dependencies

jacobq · jacobq · commit 8c50fc72647c · 2020-12-18T11:06:15.000-06:00
Note: For ember-qunit 5.x we had to add qunit and @ember/test-helpers as devDependencies See https://github.com/emberjs/ember-qunit/blob/master/docs/migration.md#upgrading-from-v4x-to-v500 fix(deps): move @ember/test-waiters to dependencies (ref #184) test: add DOM fixtures & move waiter to module scope The truth comes out: I (Jacob) did not actually read all of the ember-qunit 5.x migration guide that I mentioned in a previous commit. Oops :) This commit adds the DOM fixtures as explained here: https://github.com/emberjs/ember-qunit/blob/master/docs/migration.md#dom-fixtures It also addresses the warning triggered by accidentally recreating test waiters since `buildWaiter` was being called from within the class instead of at the module level as recommended by: https://github.com/emberjs/ember-test-waiters#use-buildwaiter-in-module-scope
diff --git a/addon/components/plot-ly.js b/addon/components/plot-ly.js
@@ -3,6 +3,7 @@ import EmberObject, { computed } from '@ember/object';
 import { observes } from '@ember-decorators/object';
 import { debounce, scheduleOnce } from '@ember/runloop';
 import { buildWaiter } from '@ember/test-waiters';
+const waiter = buildWaiter('ember-cli-plotly:component-loaded');
 
 import layout from '../templates/components/plot-ly';
 
@@ -85,10 +86,8 @@ export default class PlotlyComponent extends Component {
     super(...args);
 
     /* global Ember */
-    let waiter;
     let token;
     if (Ember.testing) {
-      waiter = buildWaiter('plotly-loaded-waiter');
       token = waiter.beginAsync();
     }
 
diff --git a/package.json b/package.json
@@ -33,20 +33,21 @@
     "test:ember-compatibility": "ember try:each"
   },
   "dependencies": {
+    "@ember/test-waiters": "^2.3.2",
     "debug": "4.3.1",
-    "ember-auto-import": "^1.8.0",
+    "ember-auto-import": "^1.10.0",
     "ember-cli-babel": "7.23.0",
     "ember-cli-htmlbars": "5.3.1",
     "ember-decorators": "6.1.1",
     "ember-lodash": "4.19.5",
-    "plotly.js": "1.57.1"
+    "plotly.js": "1.58.3"
   },
   "devDependencies": {
     "@babel/core": "^7",
     "@ember/optional-features": "2.0.0",
-    "@ember/test-waiters": "^2.3.0",
-    "@glimmer/component": "^1.0.2",
-    "@glimmer/tracking": "^1.0.2",
+    "@ember/test-helpers": "^2.1.3",
+    "@glimmer/component": "^1.0.3",
+    "@glimmer/tracking": "^1.0.3",
     "babel-eslint": "10.1.0",
     "broccoli-asset-rev": "3.0.0",
     "ember-cli": "3.23.0",
@@ -63,18 +64,19 @@
     "ember-load-initializers": "2.1.2",
     "ember-maybe-import-regenerator": "0.1.6",
     "ember-md5": "1.2.0",
-    "ember-qunit": "4.6.0",
+    "ember-qunit": "5.1.1",
     "ember-resolver": "8.0.2",
     "ember-source": "3.23.1",
     "ember-source-channel-url": "3.0.0",
-    "ember-template-lint": "^2.14.0",
+    "ember-template-lint": "^2.15.0",
     "ember-truth-helpers": "3.0.0",
     "ember-try": "1.4.0",
     "eslint": "^7.15.0",
     "eslint-plugin-ember": "10.0.2",
     "eslint-plugin-node": "11.1.0",
     "loader.js": "4.7.0",
     "npm-run-all": "^4.1.5",
+    "qunit": "2.13.0",
     "qunit-dom": "1.6.0",
     "safe-json-stringify": "1.2.0",
     "testdouble": "3.16.1"
diff --git a/tests/index.html b/tests/index.html
@@ -20,6 +20,12 @@
   <body>
     {{content-for "body"}}
     {{content-for "test-body"}}
+    <div id="qunit"></div>
+    <div id="qunit-fixture">
+      <div id="ember-testing-container">
+        <div id="ember-testing"></div>
+      </div>
+    </div>
 
     <script src="/testem.js" integrity=""></script>
     <script src="{{rootURL}}assets/vendor.js"></script>
diff --git a/yarn.lock b/yarn.lock
