Merge pull request #71 from jasonmit/patch-1

Only access instance.getURL if the instance has booted

Author: danmcclain

lib/ember-app.js

@@ -190,7 +190,7 @@ class EmberApp {
    * @param {string} path the URL path to render, like `/photos/1`
    * @param {Object} options
    * @param {string} [options.html] the HTML document to insert the rendered app into
-   * @param {ClientRequest} 
+   * @param {ClientRequest}
    * @returns {Promise<Result>} result
    */
   visit(path, options) {
@@ -218,6 +218,7 @@ class EmberApp {
 
         return instance.boot(bootOptions);
       })
+      .then(() => result.instanceBooted = true)
       .then(() => instance.visit(path, bootOptions))
       .then(() => waitForApp(instance))
       .then(() => createShoebox(doc, fastbootInfo))

lib/result.js

@@ -9,7 +9,8 @@ const HTMLSerializer = new SimpleDOM.HTMLSerializer(SimpleDOM.voidMap);
  * method.
  */
 class Result {
-  constructor(options) {
+  constructor(options) {
+    this.instanceBooted = false;
     this._doc = options.doc;
     this._html = options.html;
     this._fastbootInfo = options.fastbootInfo;
@@ -70,8 +71,10 @@ class Result {
     return this;
   }
 
-  _finalizeMetadata(instance) {
-    this.url = instance.getURL();
+  _finalizeMetadata(instance) {
+    if (this.instanceBooted) {
+      this.url = instance.getURL();
+    }
 
     let response = this._fastbootInfo.response;