use HAS_NATIVE_SYMBOL consistently

emberjs · Feb 8, 2019 · 0fe6002 · 0fe6002
1 parent 47edf3d
commit 0fe6002
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/packages/@ember/-internals/glimmer/tests/integration/content-test.js b/packages/@ember/-internals/glimmer/tests/integration/content-test.js
@@ -6,6 +6,7 @@ import { set, computed } from '@ember/-internals/metal';
 import { getDebugFunction, setDebugFunction } from '@ember/debug';
 import { readOnly } from '@ember/object/computed';
 import { Object as EmberObject, ObjectProxy } from '@ember/-internals/runtime';
+import { HAS_NATIVE_SYMBOL } from '@ember/-internals/utils';
 import { constructStyleDeprecationMessage } from '@ember/-internals/views';
 import { Component, SafeString, htmlSafe } from '../utils/helpers';
 
@@ -682,7 +683,7 @@ let GlimmerContentTestCases = new ContentTestGenerator([
   [Object.create(null), EMPTY, 'an object with no toString'],
 ]);
 
-if (typeof Symbol !== 'undefined') {
+if (HAS_NATIVE_SYMBOL) {
   GlimmerContentTestCases.cases.push([Symbol('debug'), 'Symbol(debug)', 'a symbol']);
 }
 

diff --git a/packages/@ember/-internals/utils/tests/guid_for_test.js b/packages/@ember/-internals/utils/tests/guid_for_test.js
@@ -1,4 +1,4 @@
-import { guidFor } from '..';
+import { guidFor, HAS_NATIVE_SYMBOL } from '..';
 import { moduleFor, AbstractTestCase as TestCase } from 'internal-test-helpers';
 
 function sameGuid(assert, a, b, message) {
@@ -49,7 +49,7 @@ moduleFor(
     }
 
     ['@test symbols'](assert) {
-      if (typeof Symbol === 'undefined') {
+      if (HAS_NATIVE_SYMBOL) {
         assert.ok(true, 'symbols are not supported on this browser');
         return;
       }