[BUGFIX beta] avoid ordered set deprecation

emberjs · Jun 1, 2018 · 8e15f56 · 8e15f56
1 parent ff6bc16
commit 8e15f56
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 9 deletions.
diff --git a/packages/@ember/map/lib/ordered-set.js b/packages/@ember/map/lib/ordered-set.js
@@ -12,16 +12,14 @@ import { copyNull } from './utils';
   @constructor
   @private
 */
-export default class OrderedSet {
-  constructor() {
-    deprecate('Use of @ember/OrderedSet is deprecated. Please use native `Map` instead', false, {
-      id: 'ember-map-deprecation',
-      until: '3.5.0',
-    });
 
-    this.clear();
-  }
+/**
+ * This is exported so it can be used by the OrderedSet library.
+ * This is private do not use it.
+ @private
+ */
 
+export class __OrderedSet__ {
   /**
     @method create
     @static
@@ -166,3 +164,14 @@ export default class OrderedSet {
     return set;
   }
 }
+
+export default class OrderedSet extends __OrderedSet__ {
+  constructor() {
+    super();
+    deprecate('Use of @ember/OrderedSet is deprecated. Please use native `Map` instead', false, {
+      id: 'ember-map-deprecation',
+      until: '3.5.0',
+    });
+    this.clear();
+  }
+}
diff --git a/packages/@ember/map/tests/map_test.js b/packages/@ember/map/tests/map_test.js
@@ -1,6 +1,6 @@
 import Map from '..';
 import MapWithDefault from '../with-default';
-import OrderedSet from '../lib/ordered-set';
+import OrderedSet, { __OrderedSet__ } from '../lib/ordered-set';
 import { moduleFor, AbstractTestCase } from 'internal-test-helpers';
 
 let object, number, string, map, variety;
@@ -566,3 +566,13 @@ moduleFor(
     }
   }
 );
+
+moduleFor(
+  '__OrderedSet__',
+  class extends AbstractTestCase {
+    ['@test private __OrderedSet__ can be created without deprecation']() {
+      expectNoDeprecation();
+      __OrderedSet__.create();
+    }
+  }
+);