add an acceptance test; get it to pass by wrapping load callback w/ E…

…mber.run()

also handles case where JSAPI is loaded before assertions are run

CHANGELOG.md

@@ -2,6 +2,11 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## Unreleased
+
+### Changed
+- added acceptance test
+
 ## 0.1.2
 
 ### Changed

addon/services/esri-loader.js

@@ -32,15 +32,17 @@ export default Ember.Service.extend({
     // otherwise create a promise that will resolve when the JSAPI is loaded
     this._loadPromise = new Ember.RSVP.Promise((resolve, reject) => {
       esriLoader.bootstrap(err => {
-        if (err) {
-          reject(err);
-        } else {
-          // notify any watchers of isLoaded copmuted property
-          this.notifyPropertyChange('isLoaded');
-          // let the caller know that the API has been successfully loaded
-          // TODO: would there be something more useful to return here?
-          resolve({ success: true });
-        }
+        Ember.run(() => {
+          if (err) {
+            reject(err);
+          } else {
+            // notify any watchers of isLoaded copmuted property
+            this.notifyPropertyChange('isLoaded');
+            // let the caller know that the API has been successfully loaded
+            // TODO: would there be something more useful to return here?
+            resolve({ success: true });
+          }
+        });
       }, options);
     });
     return this._loadPromise;

tests/acceptance/index-test.js

@@ -0,0 +1,13 @@
+import { test } from 'qunit';
+import moduleForAcceptance from '../../tests/helpers/module-for-acceptance';
+
+moduleForAcceptance('Acceptance | index');
+
+test('visiting /', function(assert) {
+  visit('/');
+
+  andThen(function() {
+    assert.equal(currentURL(), '/');
+    assert.equal(find('p strong span').text().substr(0, 4), 'Load', 'status should be either Loading... or Loaded');
+  });
+});