diff options
| -rw-r--r-- | src/ngMock/angular-mocks.js | 2 | ||||
| -rw-r--r-- | src/ngRoute/route.js | 42 | ||||
| -rw-r--r-- | src/ngRoute/routeUtils.js | 17 | ||||
| -rw-r--r-- | test/helpers/matchers.js | 12 | ||||
| -rw-r--r-- | test/helpers/privateMocksSpec.js | 2 | ||||
| -rw-r--r-- | test/ngMock/angular-mocksSpec.js | 2 | ||||
| -rw-r--r-- | test/ngRoute/directive/ngViewSpec.js | 22 | ||||
| -rw-r--r-- | test/ngRoute/routeSpec.js | 24 | 
8 files changed, 55 insertions, 68 deletions
| diff --git a/src/ngMock/angular-mocks.js b/src/ngMock/angular-mocks.js index 3592aacd..05551ea7 100644 --- a/src/ngMock/angular-mocks.js +++ b/src/ngMock/angular-mocks.js @@ -769,7 +769,7 @@ angular.mock.animate = angular.module('mock.animate', ['ng'])          }        }; -      forEach(['enter','leave','move','addClass','removeClass'], function(method) { +      angular.forEach(['enter','leave','move','addClass','removeClass'], function(method) {          animate[method] = function() {            var params = arguments;            animate.queue.push({ diff --git a/src/ngRoute/route.js b/src/ngRoute/route.js index 8a6dacd4..50e2405a 100644 --- a/src/ngRoute/route.js +++ b/src/ngRoute/route.js @@ -28,6 +28,10 @@ var ngRouteModule = angular.module('ngRoute', ['ng']).   * Requires the {@link ngRoute `ngRoute`} module to be installed.   */  function $RouteProvider(){ +  function inherit(parent, extra) { +    return angular.extend(new (angular.extend(function() {}, {prototype:parent}))(), extra); +  } +    var routes = {};    /** @@ -126,7 +130,7 @@ function $RouteProvider(){     * Adds a new route definition to the `$route` service.     */    this.when = function(path, route) { -    routes[path] = extend( +    routes[path] = angular.extend(        {reloadOnSearch: true},        route,        path && pathRegExp(path, route) @@ -138,7 +142,7 @@ function $RouteProvider(){              ? path.substr(0, path.length-1)              : path +'/'; -      routes[redirectPath] = extend( +      routes[redirectPath] = angular.extend(          {redirectTo: path},          pathRegExp(redirectPath, route)        ); @@ -460,9 +464,9 @@ function $RouteProvider(){            last = $route.current;        if (next && last && next.$$route === last.$$route -          && equals(next.pathParams, last.pathParams) && !next.reloadOnSearch && !forceReload) { +          && angular.equals(next.pathParams, last.pathParams) && !next.reloadOnSearch && !forceReload) {          last.params = next.params; -        copy(last.params, $routeParams); +        angular.copy(last.params, $routeParams);          $rootScope.$broadcast('$routeUpdate', last);        } else if (next || last) {          forceReload = false; @@ -470,7 +474,7 @@ function $RouteProvider(){          $route.current = next;          if (next) {            if (next.redirectTo) { -            if (isString(next.redirectTo)) { +            if (angular.isString(next.redirectTo)) {                $location.path(interpolate(next.redirectTo, next.params)).search(next.params)                         .replace();              } else { @@ -483,29 +487,29 @@ function $RouteProvider(){          $q.when(next).            then(function() {              if (next) { -              var locals = extend({}, next.resolve), +              var locals = angular.extend({}, next.resolve),                    template, templateUrl; -              forEach(locals, function(value, key) { -                locals[key] = isString(value) ? $injector.get(value) : $injector.invoke(value); +              angular.forEach(locals, function(value, key) { +                locals[key] = angular.isString(value) ? $injector.get(value) : $injector.invoke(value);                }); -              if (isDefined(template = next.template)) { -                if (isFunction(template)) { +              if (angular.isDefined(template = next.template)) { +                if (angular.isFunction(template)) {                    template = template(next.params);                  } -              } else if (isDefined(templateUrl = next.templateUrl)) { -                if (isFunction(templateUrl)) { +              } else if (angular.isDefined(templateUrl = next.templateUrl)) { +                if (angular.isFunction(templateUrl)) {                    templateUrl = templateUrl(next.params);                  }                  templateUrl = $sce.getTrustedResourceUrl(templateUrl); -                if (isDefined(templateUrl)) { +                if (angular.isDefined(templateUrl)) {                    next.loadedTemplateUrl = templateUrl;                    template = $http.get(templateUrl, {cache: $templateCache}).                        then(function(response) { return response.data; });                  }                } -              if (isDefined(template)) { +              if (angular.isDefined(template)) {                  locals['$template'] = template;                }                return $q.all(locals); @@ -516,7 +520,7 @@ function $RouteProvider(){              if (next == $route.current) {                if (next) {                  next.locals = locals; -                copy(next.params, $routeParams); +                angular.copy(next.params, $routeParams);                }                $rootScope.$broadcast('$routeChangeSuccess', next, last);              } @@ -535,10 +539,10 @@ function $RouteProvider(){      function parseRoute() {        // Match a route        var params, match; -      forEach(routes, function(route, path) { +      angular.forEach(routes, function(route, path) {          if (!match && (params = switchRouteMatcher($location.path(), route))) {            match = inherit(route, { -            params: extend({}, $location.search(), params), +            params: angular.extend({}, $location.search(), params),              pathParams: params});            match.$$route = route;          } @@ -552,7 +556,7 @@ function $RouteProvider(){       */      function interpolate(string, params) {        var result = []; -      forEach((string||'').split(':'), function(segment, i) { +      angular.forEach((string||'').split(':'), function(segment, i) {          if (i === 0) {            result.push(segment);          } else { @@ -566,4 +570,4 @@ function $RouteProvider(){        return result.join('');      }    }]; -} +}
\ No newline at end of file diff --git a/src/ngRoute/routeUtils.js b/src/ngRoute/routeUtils.js deleted file mode 100644 index 0cff7213..00000000 --- a/src/ngRoute/routeUtils.js +++ /dev/null @@ -1,17 +0,0 @@ -'use strict'; - -var copy = angular.copy, -    equals = angular.equals, -    extend = angular.extend, -    forEach = angular.forEach, -    isDefined = angular.isDefined, -    isFunction = angular.isFunction, -    isString = angular.isString, -    jqLite = angular.element, -    noop = angular.noop, -    toJson = angular.toJson; - - -function inherit(parent, extra) { -  return extend(new (extend(function() {}, {prototype:parent}))(), extra); -} diff --git a/test/helpers/matchers.js b/test/helpers/matchers.js index 14430b37..c5d7d6cf 100644 --- a/test/helpers/matchers.js +++ b/test/helpers/matchers.js @@ -74,11 +74,11 @@ beforeEach(function() {        this.message = function() {          var expected;          if (this.actual.message && this.actual.name == 'Error') { -          expected = toJson(this.actual.message); +          expected = angular.toJson(this.actual.message);          } else { -          expected = toJson(this.actual); +          expected = angular.toJson(this.actual);          } -        return "Expected " + expected + " to be an Error with message " + toJson(message); +        return "Expected " + expected + " to be an Error with message " + angular.toJson(message);        };        return this.actual.name == 'Error' && this.actual.message == message;      }, @@ -187,9 +187,9 @@ beforeEach(function() {          codeRegex = new RegExp('^\\[' + escapeRegexp(namespace) + ':' + escapeRegexp(code) + '\\]'),          not = this.isNot ? "not " : "",          regex = jasmine.isA_("RegExp", content) ? content : -                  isDefined(content) ? new RegExp(escapeRegexp(content)) : undefined; +                  angular.isDefined(content) ? new RegExp(escapeRegexp(content)) : undefined; -      if(!isFunction(this.actual)) { +      if(!angular.isFunction(this.actual)) {          throw new Error('Actual is not a function');        } @@ -215,7 +215,7 @@ beforeEach(function() {          return result;        } -      if (isDefined(regex)) { +      if (angular.isDefined(regex)) {          return regex.test(exceptionMessage);        }        return result; diff --git a/test/helpers/privateMocksSpec.js b/test/helpers/privateMocksSpec.js index e58a2b75..d5a1e188 100644 --- a/test/helpers/privateMocksSpec.js +++ b/test/helpers/privateMocksSpec.js @@ -9,7 +9,7 @@ describe('private mocks', function() {        var stylesheet = createMockStyleSheet($document, $window);        expect(doc.styleSheets.length).toBe(count + 1); -      jqLite(doc.body).append($rootElement); +      angular.element(doc.body).append($rootElement);        var elm = $compile('<div class="padded">...</div>')($rootScope);        $rootElement.append(elm); diff --git a/test/ngMock/angular-mocksSpec.js b/test/ngMock/angular-mocksSpec.js index 851f7803..80580fb1 100644 --- a/test/ngMock/angular-mocksSpec.js +++ b/test/ngMock/angular-mocksSpec.js @@ -160,7 +160,7 @@ describe('ngMock', function() {    describe('$log', function() { -    forEach([true, false], function(debugEnabled) { +    angular.forEach([true, false], function(debugEnabled) {        describe('debug ' + debugEnabled, function() {          beforeEach(module(function($logProvider) {            $logProvider.debugEnabled(debugEnabled); diff --git a/test/ngRoute/directive/ngViewSpec.js b/test/ngRoute/directive/ngViewSpec.js index d38b3ba9..2d74760c 100644 --- a/test/ngRoute/directive/ngViewSpec.js +++ b/test/ngRoute/directive/ngViewSpec.js @@ -279,7 +279,7 @@ describe('ngView', function() {    it('should be async even if served from cache', function() {      module(function($routeProvider) { -      $routeProvider.when('/foo', {controller: noop, templateUrl: 'myUrl1'}); +      $routeProvider.when('/foo', {controller: angular.noop, templateUrl: 'myUrl1'});      });      inject(function($route, $rootScope, $location, $templateCache) { @@ -498,16 +498,16 @@ describe('ngView', function() {        $location.url('/foo');        $rootScope.$digest(); -      forEach(element.contents(), function(node) { +      angular.forEach(element.contents(), function(node) {          if(node.nodeType == 3 /* text node */) { -          expect(jqLite(node).scope()).not.toBe($route.current.scope); -          expect(jqLite(node).controller()).not.toBeDefined(); +          expect(angular.element(node).scope()).not.toBe($route.current.scope); +          expect(angular.element(node).controller()).not.toBeDefined();          } else if(node.nodeType == 8 /* comment node */) { -          expect(jqLite(node).scope()).toBe(element.scope()); -          expect(jqLite(node).controller()).toBe(element.controller()); +          expect(angular.element(node).scope()).toBe(element.scope()); +          expect(angular.element(node).controller()).toBe(element.controller());          } else { -          expect(jqLite(node).scope()).toBe($route.current.scope); -          expect(jqLite(node).controller()).toBeDefined(); +          expect(angular.element(node).scope()).toBe($route.current.scope); +          expect(angular.element(node).controller()).toBeDefined();          }        });      }); @@ -530,7 +530,7 @@ describe('ngView animations', function() {      // we need to run animation on attached elements;      return function(_$rootElement_) {        $rootElement = _$rootElement_; -      body = jqLite(document.body); +      body = angular.element(document.body);      };    })); @@ -541,8 +541,8 @@ describe('ngView animations', function() {    beforeEach(module(function($provide, $routeProvider) { -    $routeProvider.when('/foo', {controller: noop, templateUrl: '/foo.html'}); -    $routeProvider.when('/bar', {controller: noop, templateUrl: '/bar.html'}); +    $routeProvider.when('/foo', {controller: angular.noop, templateUrl: '/foo.html'}); +    $routeProvider.when('/bar', {controller: angular.noop, templateUrl: '/bar.html'});      return function($templateCache) {        $templateCache.put('/foo.html', [200, '<div>data</div>', {}]);        $templateCache.put('/bar.html', [200, '<div>data2</div>', {}]); diff --git a/test/ngRoute/routeSpec.js b/test/ngRoute/routeSpec.js index bd7966e1..de7ccb8d 100644 --- a/test/ngRoute/routeSpec.js +++ b/test/ngRoute/routeSpec.js @@ -25,7 +25,7 @@ describe('$route', function() {      module(function($routeProvider) {        $routeProvider.when('/Book/:book/Chapter/:chapter', -          {controller: noop, templateUrl: 'Chapter.html'}); +          {controller: angular.noop, templateUrl: 'Chapter.html'});        $routeProvider.when('/Blank', {});      });      inject(function($route, $location, $rootScope) { @@ -69,9 +69,9 @@ describe('$route', function() {      module(function($routeProvider) {        $routeProvider.when('/Book1/:book/Chapter/:chapter/:highlight*/edit', -          {controller: noop, templateUrl: 'Chapter.html'}); +          {controller: angular.noop, templateUrl: 'Chapter.html'});        $routeProvider.when('/Book2/:book/:highlight*/Chapter/:chapter', -          {controller: noop, templateUrl: 'Chapter.html'}); +          {controller: angular.noop, templateUrl: 'Chapter.html'});        $routeProvider.when('/Blank', {});      });      inject(function($route, $location, $rootScope) { @@ -128,9 +128,9 @@ describe('$route', function() {      module(function($routeProvider) {        $routeProvider.when('/Book1/:book/Chapter/:chapter/:highlight*/edit', -          {controller: noop, templateUrl: 'Chapter.html', caseInsensitiveMatch: true}); +          {controller: angular.noop, templateUrl: 'Chapter.html', caseInsensitiveMatch: true});        $routeProvider.when('/Book2/:book/:highlight*/Chapter/:chapter', -          {controller: noop, templateUrl: 'Chapter.html'}); +          {controller: angular.noop, templateUrl: 'Chapter.html'});        $routeProvider.when('/Blank', {});      });      inject(function($route, $location, $rootScope) { @@ -618,8 +618,8 @@ describe('$route', function() {        inject(function($route, $httpBackend, $location, $rootScope, $routeParams) {          var log = ''; -        $rootScope.$on('$routeChangeStart', function(e, next) { log += '$before' + toJson($routeParams) + ';'}); -        $rootScope.$on('$routeChangeSuccess', function(e, next) { log += '$after' + toJson($routeParams) + ';'}); +        $rootScope.$on('$routeChangeStart', function(e, next) { log += '$before' + angular.toJson($routeParams) + ';'}); +        $rootScope.$on('$routeChangeSuccess', function(e, next) { log += '$after' + angular.toJson($routeParams) + ';'});          $httpBackend.whenGET('r1.html').respond('R1');          $httpBackend.whenGET('r2.html').respond('R2'); @@ -876,7 +876,7 @@ describe('$route', function() {        var reloaded = jasmine.createSpy('route reload');        module(function($routeProvider) { -        $routeProvider.when('/foo', {controller: noop}); +        $routeProvider.when('/foo', {controller: angular.noop});        });        inject(function($route, $location, $rootScope, $routeParams) { @@ -901,7 +901,7 @@ describe('$route', function() {            routeUpdate = jasmine.createSpy('route update');        module(function($routeProvider) { -        $routeProvider.when('/foo', {controller: noop, reloadOnSearch: false}); +        $routeProvider.when('/foo', {controller: angular.noop, reloadOnSearch: false});        });        inject(function($route, $location, $rootScope) { @@ -931,7 +931,7 @@ describe('$route', function() {        var routeChange = jasmine.createSpy('route change');        module(function($routeProvider) { -        $routeProvider.when('/foo/:fooId', {controller: noop, reloadOnSearch: false}); +        $routeProvider.when('/foo/:fooId', {controller: angular.noop, reloadOnSearch: false});        });        inject(function($route, $location, $rootScope) { @@ -963,8 +963,8 @@ describe('$route', function() {        var routeParamsWatcher = jasmine.createSpy('routeParamsWatcher');        module(function($routeProvider) { -        $routeProvider.when('/foo', {controller: noop}); -        $routeProvider.when('/bar/:barId', {controller: noop, reloadOnSearch: false}); +        $routeProvider.when('/foo', {controller: angular.noop}); +        $routeProvider.when('/bar/:barId', {controller: angular.noop, reloadOnSearch: false});        });        inject(function($route, $location, $rootScope, $routeParams) { | 
