Fix Ember._Renderer export issue.

* Update tests to ensure that the global path must not be `undefined` * Fix issue with `Ember._Renderer` export * Remove `Ember.rewatch` export remnants (it was removed in #9323) * Remove `Ember.apply` export remnants (it was removed in #13655) * Remove `Ember.tryCatchFinally` export remnants (it was removed in #11547)
emberjs · Sep 3, 2016 · fe4c161 · fe4c161
1 parent 39b9386
commit fe4c161
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 12 deletions.
diff --git a/packages/ember-glimmer/lib/index.js b/packages/ember-glimmer/lib/index.js
@@ -17,7 +17,11 @@ export {
   isHTMLSafe,
   getSafeString as _getSafeString
 } from './utils/string';
-export { default as _Renderer } from './renderer';
+export {
+  Renderer,
+  InertRenderer,
+  InteractiveRenderer
+} from './renderer';
 export {
   getTemplate,
   setTemplate,

diff --git a/packages/ember-metal/lib/index.js b/packages/ember-metal/lib/index.js
@@ -31,14 +31,12 @@ export {
   intern,
   GUID_KEY,
   GUID_KEY_PROPERTY,
-  apply,
   applyStr,
   canInvoke,
   generateGuid,
   guidFor,
   inspect,
   makeArray,
-  deprecatedTryCatchFinally,
   tryInvoke,
   uuid,
   wrap

diff --git a/packages/ember/lib/index.js b/packages/ember/lib/index.js
@@ -44,12 +44,10 @@ Ember.GUID_KEY = metal.GUID_KEY;
 Ember.guidFor = metal.guidFor;
 Ember.inspect = metal.inspect;
 
-Ember.tryCatchFinally = metal.deprecatedTryCatchFinally;
 Ember.makeArray = metal.makeArray;
 Ember.canInvoke = metal.canInvoke;
 Ember.tryInvoke = metal.tryInvoke;
 Ember.wrap = metal.wrap;
-Ember.apply = metal.apply;
 Ember.applyStr = metal.applyStr;
 Ember.uuid = metal.uuid;
 Ember.Error = metal.Error;
@@ -469,7 +467,7 @@ import {
   getTemplates,
   setTemplates,
   _getSafeString,
-  _Renderer
+  Renderer
 } from 'ember-glimmer';
 
 Ember.Component = Component;
@@ -479,7 +477,7 @@ Ember.Checkbox = Checkbox;
 Ember.TextField = TextField;
 Ember.TextArea = TextArea;
 Ember.LinkComponent = LinkComponent;
-Ember._Renderer = _Renderer;
+Ember._Renderer = Renderer;
 
 if (ENV.EXTEND_PROTOTYPES.String) {
   String.prototype.htmlSafe = function() {

diff --git a/packages/ember/tests/reexports_test.js b/packages/ember/tests/reexports_test.js
@@ -36,12 +36,10 @@ QUnit.module('ember reexports');
   ['GUID_KEY', 'ember-metal'],
   ['guidFor', 'ember-metal'],
   ['inspect', 'ember-metal'],
-  ['tryCatchFinally', 'ember-metal', 'deprecatedTryCatchFinally'],
   ['makeArray', 'ember-metal'],
   ['canInvoke', 'ember-metal'],
   ['tryInvoke', 'ember-metal'],
   ['wrap', 'ember-metal'],
-  ['apply', 'ember-metal'],
   ['applyStr', 'ember-metal'],
   ['uuid', 'ember-metal'],
   ['testing', 'ember-metal', { get: 'isTesting', set: 'setTesting' }],
@@ -94,7 +92,6 @@ QUnit.module('ember reexports');
   ['watch', 'ember-metal'],
   ['isWatching', 'ember-metal'],
   ['unwatch', 'ember-metal'],
-  ['rewatch', 'ember-metal'],
   ['destroy', 'ember-metal'],
   ['libraries', 'ember-metal'],
   ['OrderedSet', 'ember-metal'],
@@ -132,7 +129,7 @@ QUnit.module('ember reexports');
   ['EventDispatcher', 'ember-views'],
 
   // ember-glimmer
-  ['_Renderer',     'ember-glimmer', '_Renderer'],
+  ['_Renderer',     'ember-glimmer', 'Renderer'],
   ['Component',     'ember-glimmer', 'Component'],
   ['Helper',        'ember-glimmer', 'Helper'],
   ['Helper.helper', 'ember-glimmer', 'helper'],

diff --git a/packages/internal-test-helpers/lib/index.js b/packages/internal-test-helpers/lib/index.js
@@ -44,9 +44,15 @@ export function confirmExport(Ember, assert, path, moduleId, exportName) {
   let mod = require(moduleId);
   if (typeof exportName === 'string') {
     assert.equal(desc.value, mod[exportName], `Ember.${path} is exported correctly`);
+    assert.notEqual(mod[exportName], undefined, `Ember.${path} is not \`undefined\``);
   } else {
     assert.equal(desc.get, mod[exportName.get], `Ember.${path} getter is exported correctly`);
-    assert.equal(desc.set, mod[exportName.set], `Ember.${path} setter is exported correctly`);
+    assert.notEqual(desc.get, undefined, `Ember.${path} getter is not undefined`);
+
+    if (exportName.set) {
+      assert.equal(desc.set, mod[exportName.set], `Ember.${path} setter is exported correctly`);
+      assert.notEqual(desc.set, undefined, `Ember.${path} setter is not undefined`);
+    }
   }
 }