fall back to body if parentSelector does not exist

Author: stefanotorresi

src/loading-bar.js

@@ -194,25 +194,31 @@ angular.module('cfp.loadingBar', [])
           $animate = $injector.get('$animate');
         }
 
-        // $document.find is limited to lookups by tag name in jqLite
-        // https://docs.angularjs.org/api/ng/function/angular.element
-        var $parent = angular.element($document[0].querySelector($parentSelector)).eq(0);
         $timeout.cancel(completeTimeout);
 
         // do not continually broadcast the started event:
         if (started) {
           return;
         }
 
+        var parent = $document[0].querySelector($parentSelector);
+
+        if (! parent) {
+          parent = $document[0].querySelector('body');
+        }
+
+        var $parent = angular.element(parent);
+        var $after = parent.lastChild && angular.element(parent.lastChild);
+
         $rootScope.$broadcast('cfpLoadingBar:started');
         started = true;
 
         if (includeBar) {
-          $animate.enter(loadingBarContainer, $parent, $parent[0].lastChild && angular.element($parent[0].lastChild));
+          $animate.enter(loadingBarContainer, $parent, $after);
         }
 
         if (includeSpinner) {
-          $animate.enter(spinner, $parent, $parent[0].lastChild && angular.element($parent[0].lastChild));
+          $animate.enter(spinner, $parent, loadingBarContainer);
         }
 
         _set(startSize);

test/loading-bar-interceptor-config.coffee

@@ -96,3 +96,19 @@ describe 'loadingBarInterceptor Service - config options', ->
       expect(children[1].id).toBe 'loading-bar-spinner'
       cfpLoadingBar.complete()
       $timeout.flush()
+
+  it 'should append the loading bar to the body if parentSelector is empty', ->
+    module 'chieffancypants.loadingBar', (cfpLoadingBarProvider) ->
+      cfpLoadingBarProvider.parentSelector = '#doesnotexist'
+      return
+    inject ($timeout, $document, cfpLoadingBar) ->
+      parent = $document[0].querySelector(cfpLoadingBar.parentSelector)
+      expect(parent).toBeFalsy;
+      body = $document[0].querySelector 'body'
+      cfpLoadingBar.start()
+      bar = angular.element(body.querySelector '#loading-bar');
+      spinner = angular.element(body.querySelector '#loading-bar-spinner');
+      expect(bar.length).toBe 1
+      expect(spinner.length).toBe 1
+      cfpLoadingBar.complete()
+      $timeout.flush()