@angular-wave/angular.ts 0.4.4 → 0.4.5

package/src/directive/messages/messages.spec.js

@@ -48,6 +48,8 @@ describe("ngMessages", () => {
         '  <div ng-message="val">Message is set</div>' +
         "</div>",
     )($rootScope);
+    $rootScope.$digest();
+
     expect(element.text()).not.toContain("Message is set");
 
     $rootScope.$apply(() => {
@@ -63,6 +65,8 @@ describe("ngMessages", () => {
         '  <div ng-message="one, two, three">Message is set</div>' +
         "</div>",
     )($rootScope);
+    $rootScope.$digest();
+
     expect(element.text()).not.toContain("Message is set");
 
     $rootScope.$apply(() => {
@@ -96,6 +100,8 @@ describe("ngMessages", () => {
         '  <ng-message when="val">Message is set</div>' +
         "</ng-messages>",
     )($rootScope);
+    $rootScope.$digest();
+
     expect(element.text()).not.toContain("Message is set");
 
     $rootScope.$apply(() => {
@@ -111,6 +117,8 @@ describe("ngMessages", () => {
         '  <ng-message when=" one two three ">Message is set</div>' +
         "</ng-messages>",
     )($rootScope);
+    $rootScope.$digest();
+
     expect(element.text()).not.toContain("Message is set");
 
     $rootScope.$apply(() => {
@@ -144,6 +152,8 @@ describe("ngMessages", () => {
         '  <div ng-message-exp="variable">Message is crazy</div>' +
         "</div>",
     )($rootScope);
+    $rootScope.$digest();
+
     expect(element.text()).not.toContain("Message is crazy");
 
     $rootScope.$apply(() => {
@@ -178,6 +188,8 @@ describe("ngMessages", () => {
         '  <ng-message when-exp="variable">Message is crazy</ng-message>' +
         "</ng-messages>",
     )($rootScope);
+    $rootScope.$digest();
+
     expect(element.text()).not.toContain("Message is crazy");
 
     $rootScope.$apply(() => {
@@ -224,7 +236,7 @@ describe("ngMessages", () => {
   //           '  <div ng-message="val">Message is set</div>' +
   //           "</div>",
   //       )($rootScope);
-  //       ;
+  //       $rootScope.$digest();
 
   //       $rootScope.$apply(() => {
   //         $rootScope.col = prop;
@@ -448,11 +460,14 @@ describe("ngMessages", () => {
 
     // Trigger the message to be displayed
     $rootScope.col = { primary: true };
+    $rootScope.$digest();
     expect(messageChildren(element).length).toEqual(1);
     const oldMessageNode = messageChildren(element)[0];
 
     // Remove the message
     $rootScope.col = { primary: undefined };
+    $rootScope.$digest();
+
     // Since we have spied on the `leave` method, the message node is still in the DOM
     //expect($animate.leave).toHaveBeenCalled();
     // const nodeToRemove = $animate.leave.calls.mostRecent().args[0][0];
@@ -460,6 +475,8 @@ describe("ngMessages", () => {
 
     // Add the message back in
     $rootScope.col = { primary: true };
+    $rootScope.$digest();
+
     // Simulate the animation completing on the node
     // JQLite(nodeToRemove).remove();
 
@@ -553,7 +570,9 @@ describe("ngMessages", () => {
       const deregisterSpy = spyOn(ctrl, "deregister").and.callThrough();
 
       const nodeA = element[0].querySelector('[ng-message="a"]');
-      JQLite(nodeA).remove(); // The next digest triggers the error
+      JQLite(nodeA).remove();
+      $rootScope.$digest(); // The next digest triggers the error
+
       // Make sure removing the element triggers the deregistration in ngMessages
       expect(trim(deregisterSpy.calls.mostRecent().args[0].nodeValue)).toBe("");
       expect(messageChildren(element).length).toBe(0);
@@ -590,7 +609,9 @@ describe("ngMessages", () => {
         const deregisterSpy = spyOn(ctrl, "deregister").and.callThrough();
 
         const nodeB = element[0].querySelector('[ng-message="b"]');
-        JQLite(nodeB).remove(); // The next digest triggers the error
+        JQLite(nodeB).remove();
+        $rootScope.$digest(); // The next digest triggers the error
+
         // Make sure removing the element triggers the deregistration in ngMessages
         expect(trim(deregisterSpy.calls.mostRecent().args[0].nodeValue)).toBe(
           "",
@@ -658,6 +679,8 @@ describe("ngMessages", () => {
         $rootScope.col = { unexpected: false };
       });
 
+      $rootScope.$digest();
+
       expect(element.text().trim()).toBe("");
       expect(element[0].classList.contains("ng-active")).toBeFalse();
 
@@ -724,6 +747,8 @@ describe("ngMessages", () => {
       )($rootScope);
       $rootScope.default = true;
       $rootScope.col = { unexpected: true };
+      $rootScope.$digest();
+
       expect(element.text().trim()).toBe("Default message is set");
 
       $rootScope.$apply("default = false");
@@ -828,7 +853,7 @@ describe("ngMessages", () => {
     //       $templateCache.set("abc.html", "<div></div>");
 
     //       element = $compile(html)($rootScope);
-    //       ;
+    //       $rootScope.$digest();
 
     //       const includeElement = element[0].querySelector(
     //         "[ng-messages-include], ng-messages-include",
@@ -893,6 +918,7 @@ describe("ngMessages", () => {
         '<div ng-messages="data"><div ng-messages-include="/mock/hello"></div></div>',
       )($rootScope);
 
+      $rootScope.$digest();
       expect($templateCache.get("/mock/hello")).toBeDefined();
     });
 
@@ -908,9 +934,12 @@ describe("ngMessages", () => {
         failed: true,
       };
 
+      $rootScope.$digest();
+
       expect(messageChildren(element).length).toBe(1);
       expect(trim(element.text())).toEqual("Your value is that of failure");
 
+      $rootScope.$digest();
       setTimeout(() => {
         expect(messageChildren(element).length).toBe(1);
         expect(trim(element.text())).toEqual("You did not enter a value");
@@ -1005,6 +1034,8 @@ describe("ngMessages", () => {
       $templateCache.set("messages1.html", "");
       $templateCache.set("messages2.html", "   ");
       element = $compile(html)($rootScope);
+      $rootScope.$digest();
+
       expect(element.text()).toBe("");
       expect(element[0].childNodes.length).toBe(2);
     });

package/src/directive/model/model.js

@@ -28,7 +28,6 @@ import {
 import { defaultModelOptions } from "../model-options/model-options";
 import { startingTag } from "../../shared/jqlite/jqlite";
 import { ScopePhase } from "../../core/scope/scope";
-import { isProxySymbol } from "../../core/model/model";
 
 export const ngModelMinErr = minErr("ngModel");
 
@@ -77,6 +76,8 @@ export class NgModelController {
     "$element",
     "$parse",
     "$animate",
+    "$timeout",
+    "$q",
     "$interpolate",
   ];
 
@@ -87,6 +88,8 @@ export class NgModelController {
    * @param {import('../../shared/jqlite/jqlite').JQLite} $element
    * @param {import("../../core/parse/parse").ParseService} $parse
    * @param {*} $animate
+   * @param {*} $timeout
+   * @param {import("../../core/q/q").QPromise<any>} $q
    * @param {*} $interpolate
    */
   constructor(
@@ -96,6 +99,8 @@ export class NgModelController {
     $element,
     $parse,
     $animate,
+    $timeout,
+    $q,
     $interpolate,
   ) {
     /** @type {any} The actual value from the control's view  */
@@ -154,14 +159,16 @@ export class NgModelController {
     this.$$currentValidationRunId = 0;
 
     /** @type {import('../../core/scope/scope.js').Scope} */
-    this.$$scope = $element.controller() ? $element.controller() : $scope; // attempt to bind to nearest controller if present
+    this.$$scope = $scope;
 
     /** @type {import('../../core/scope/scope.js').Scope} */
     this.$$rootScope = $scope.$root;
     this.$$attr = $attr;
     this.$$element = $element;
     this.$$animate = $animate;
+    this.$$timeout = $timeout;
     this.$$parse = $parse;
+    this.$q = $q;
     this.$$exceptionHandler = $exceptionHandler;
 
     this.$$hasNativeValidators = false;
@@ -517,7 +524,7 @@ export class NgModelController {
    * </example>
    */
   $rollbackViewValue() {
-    clearTimeout(this.$$pendingDebounce);
+    this.$$timeout.cancel(this.$$pendingDebounce);
     this.$viewValue = this.$$lastCommittedViewValue;
     this.$render();
   }
@@ -648,7 +655,7 @@ export class NgModelController {
       if (!validatorPromises.length) {
         validationDone(true);
       } else {
-        Promise.all(validatorPromises).then(
+        that.$q.all(validatorPromises).then(
           () => {
             validationDone(allValid);
           },
@@ -678,7 +685,7 @@ export class NgModelController {
    * usually handles calling this in response to input events.
    */
   $commitViewValue() {
-    clearTimeout(this.$$pendingDebounce);
+    this.$$timeout.cancel(this.$$pendingDebounce);
 
     // If the view value has not changed then we should just exit, except in the case where there is
     // a native validator on the element. In this case the validation state may have changed even though
@@ -729,8 +736,9 @@ export class NgModelController {
     }
     if (isNumberNaN(this.$modelValue)) {
       // this.$modelValue has not been touched yet...
-      // @ts-ignore
-      this.$modelValue = this.$$ngModelGet(this.$$scope);
+      this.$modelValue = /** @type {(Scope) => any} */ (this.$$ngModelGet)(
+        this.$$scope,
+      );
     }
     const prevModelValue = this.$modelValue;
     const allowInvalid = this.$options.getOption("allowInvalid");
@@ -752,9 +760,6 @@ export class NgModelController {
           // external validators (e.g. calculated on the server),
           // that just call $setValidity and need the model value
           // to calculate their validity.
-          if (that.$modelValue[isProxySymbol]) {
-            delete that.$modelValue;
-          }
           that.$modelValue = allValid ? modelValue : undefined;
           writeToModelIfNeeded();
         }
@@ -762,8 +767,7 @@ export class NgModelController {
     );
 
     function writeToModelIfNeeded() {
-      // intentional loose equality
-      if (that.$modelValue != prevModelValue) {
+      if (that.$modelValue !== prevModelValue) {
         that.$$writeModelToScope();
       }
     }
@@ -850,13 +854,13 @@ export class NgModelController {
       debounceDelay = debounceDelay["*"];
     }
 
-    clearTimeout(this.$$pendingDebounce);
+    this.$$timeout.cancel(this.$$pendingDebounce);
     const that = this;
     if (/** @type {number} */ (debounceDelay) > 0) {
       // this fails if debounceDelay is an object
-      this.$$pendingDebounce = setTimeout(() => {
+      this.$$pendingDebounce = this.$$timeout(() => {
         that.$commitViewValue();
-      }, /** @type {number} */ (debounceDelay));
+      }, debounceDelay);
     } else if (this.$$rootScope.$$phase !== ScopePhase.NONE) {
       this.$commitViewValue();
     } else {
@@ -1063,7 +1067,7 @@ function setupModelWatcher(ctrl) {
   //    -> scope value did not change since the last digest as
   //       ng-change executes in apply phase
   // 4. view should be changed back to 'a'
-  ctrl.$$scope.$watch("scope", (scope) => {
+  ctrl.$$scope.$watch((scope) => {
     const modelValue = ctrl.$$ngModelGet(scope);
 
     // if scope model value and ngModel value are out of sync
@@ -1096,7 +1100,6 @@ export function ngModelDirective($rootScope) {
       /** @param {import("../../shared/jqlite/jqlite.js").JQLite} element  */
       (element) => {
         // Setup initial state of the control
-
         element[0].classList.add(PRISTINE_CLASS, UNTOUCHED_CLASS, VALID_CLASS);
 
         return {
@@ -1135,7 +1138,11 @@ export function ngModelDirective($rootScope) {
             element.on("blur", () => {
               if (modelCtrl.$touched) return;
 
-              scope.$apply(setTouched);
+              if ($rootScope.$$phase !== ScopePhase.NONE) {
+                scope.$evalAsync(setTouched);
+              } else {
+                scope.$apply(setTouched);
+              }
             });
           },
         };

package/src/directive/model/model.spec.js

@@ -40,6 +40,7 @@ describe("ngModel", () => {
     element = JQLite("<form><input></form>");
     let $controller = injector.get("$controller");
     scope = injector.get("$rootScope");
+    $q = injector.get("$q");
     $rootScope = scope;
     ctrl = $controller(NgModelController, {
       $scope: scope,
@@ -370,6 +371,8 @@ describe("ngModel", () => {
         };
 
         ctrl.$setViewValue("123");
+        scope.$digest();
+
         expect(element[0].classList.contains("ng-valid-parse")).toBeTrue();
         expect(element[0].classList.contains("ng-invalid-parse")).toBeFalse();
         expect(
@@ -378,6 +381,8 @@ describe("ngModel", () => {
 
         parserIsFailing = true;
         ctrl.$setViewValue("12345");
+        scope.$digest();
+
         expect(element[0].classList.contains("ng-valid-parse")).toBeFalse();
         expect(element[0].classList.contains("ng-invalid-parse")).toBeTrue();
         expect(
@@ -417,6 +422,8 @@ describe("ngModel", () => {
         expect(scope.value).toBeUndefined();
 
         defer.resolve();
+        scope.$digest();
+
         expect(ctrl.$modelValue).toBe("b");
         expect(scope.value).toBe("b");
 
@@ -427,6 +434,8 @@ describe("ngModel", () => {
         expect(scope.value).toBe("b");
 
         defer.resolve();
+        scope.$digest();
+
         expect(ctrl.$modelValue).toBe("c");
         expect(scope.value).toBe("c");
       });
@@ -510,9 +519,11 @@ describe("ngModel", () => {
         )($rootScope);
 
         $rootScope.val = 123;
+        $rootScope.$digest();
         expect($rootScope.form.field.$viewValue).toBe("123");
 
         $rootScope.val = null;
+        $rootScope.$digest();
         expect($rootScope.form.field.$viewValue).toBe(null);
 
         dealoc(form);
@@ -523,9 +534,11 @@ describe("ngModel", () => {
           '<form name="form"><input type="text" name="field" ng-model="val" /></form>',
         )($rootScope);
         $rootScope.val = 123;
+        $rootScope.$digest();
         expect($rootScope.form.field.$viewValue).toBe("123");
 
         $rootScope.val = null;
+        $rootScope.$digest();
         expect($rootScope.form.field.$viewValue).toBe(null);
 
         dealoc(form);
@@ -536,9 +549,11 @@ describe("ngModel", () => {
           '<form name="form"><input type="email" name="field" ng-model="val" /></form>',
         )($rootScope);
         $rootScope.val = 123;
+        $rootScope.$digest();
         expect($rootScope.form.field.$viewValue).toBe("123");
 
         $rootScope.val = null;
+        $rootScope.$digest();
         expect($rootScope.form.field.$viewValue).toBe(null);
 
         dealoc(form);
@@ -549,9 +564,11 @@ describe("ngModel", () => {
           '<form name="form"><input type="url" name="field" ng-model="val" /></form>',
         )($rootScope);
         $rootScope.val = 123;
+        $rootScope.$digest();
         expect($rootScope.form.field.$viewValue).toBe("123");
 
         $rootScope.val = null;
+        $rootScope.$digest();
         expect($rootScope.form.field.$viewValue).toBe(null);
 
         dealoc(form);
@@ -616,7 +633,7 @@ describe("ngModel", () => {
         //   const input = $compile(
         //     '<input name="myControl" maxlength="1" ng-model="value" >',
         //   )($rootScope);
-        //   ;
+        //   $rootScope.$digest();
 
         //   spyOn($animate, "addClass");
         //   spyOn($animate, "removeClass");
@@ -629,7 +646,7 @@ describe("ngModel", () => {
         //   setModelValue(ctrl, 3);
         //   ctrl.$processModelValue();
 
-        //   // $animate adds / removes classes in the $postUpdate, which
+        //   // $animate adds / removes classes in the $$postDigest, which
         //   // we cannot trigger with $digest, because that would set the model from the scope,
         //   // so we simply check if the functions have been called
         //   expect($animate.removeClass.calls.mostRecent().args[0][0]).toBe(
@@ -956,6 +973,8 @@ describe("ngModel", () => {
         expect(ctrl.$pending.promiseValidator).toBe(true);
 
         defer.resolve();
+        scope.$digest();
+
         expect(ctrl.$valid).toBe(true);
         expect(ctrl.$invalid).toBe(false);
         expect(ctrl.$pending).toBeUndefined();
@@ -963,6 +982,8 @@ describe("ngModel", () => {
         scope.$apply('value = "123"');
 
         defer.reject();
+        scope.$digest();
+
         expect(ctrl.$valid).toBe(false);
         expect(ctrl.$invalid).toBe(true);
         expect(ctrl.$pending).toBeUndefined();
@@ -1046,7 +1067,10 @@ describe("ngModel", () => {
         newDefer = defer;
 
         newDefer.reject();
+        scope.$digest();
         oldDefer.resolve();
+        scope.$digest();
+
         expect(ctrl.$valid).toBe(false);
         expect(ctrl.$invalid).toBe(true);
         expect(ctrl.$pending).toBeUndefined();
@@ -1079,6 +1103,7 @@ describe("ngModel", () => {
         expect(isObject(ctrl.$pending)).toBe(true);
 
         defers[1].resolve();
+        scope.$digest();
         expect(ctrl.$valid).toBe(true);
         expect(ctrl.$invalid).toBe(false);
         expect(isObject(ctrl.$pending)).toBe(false);
@@ -1106,6 +1131,8 @@ describe("ngModel", () => {
         expect(isObject(ctrl.$pending)).toBe(false);
 
         defer.resolve();
+        scope.$digest();
+
         expect(ctrl.$valid).toBe(false);
         expect(ctrl.$invalid).toBe(true);
         expect(isObject(ctrl.$pending)).toBe(false);
@@ -1159,10 +1186,13 @@ describe("ngModel", () => {
         const formCtrl = $rootScope.myForm;
         const usernameCtrl = formCtrl.username;
 
+        $rootScope.$digest();
         expect(usernameCtrl.$invalid).toBe(true);
         expect(formCtrl.$invalid).toBe(true);
 
         usernameCtrl.$setViewValue("valid-username");
+        $rootScope.$digest();
+
         expect(usernameCtrl.$invalid).toBe(false);
         expect(formCtrl.$invalid).toBe(false);
         delete Object.prototype.someThing;
@@ -1189,30 +1219,42 @@ describe("ngModel", () => {
           return usernameDefer.promise;
         };
 
+        $rootScope.$digest();
         expect(usernameCtrl.$invalid).toBe(true);
         expect(formCtrl.$invalid).toBe(true);
 
         usernameCtrl.$setViewValue("valid-username");
+        $rootScope.$digest();
+
         expect(formCtrl.$pending.usernameAvailability).toBeTruthy();
         expect(usernameCtrl.$invalid).toBeUndefined();
         expect(formCtrl.$invalid).toBeUndefined();
 
         usernameDefer.resolve();
+        $rootScope.$digest();
         expect(usernameCtrl.$invalid).toBe(false);
         expect(formCtrl.$invalid).toBe(true);
 
         ageCtrl.$setViewValue(22);
+        $rootScope.$digest();
+
         expect(usernameCtrl.$invalid).toBe(false);
         expect(ageCtrl.$invalid).toBe(false);
         expect(formCtrl.$invalid).toBe(false);
 
         usernameCtrl.$setViewValue("valid");
+        $rootScope.$digest();
+
         expect(usernameCtrl.$invalid).toBe(true);
         expect(ageCtrl.$invalid).toBe(false);
         expect(formCtrl.$invalid).toBe(true);
 
         usernameCtrl.$setViewValue("another-valid-username");
+        $rootScope.$digest();
+
         usernameDefer.resolve();
+        $rootScope.$digest();
+
         expect(usernameCtrl.$invalid).toBe(false);
         expect(formCtrl.$invalid).toBe(false);
         expect(formCtrl.$pending).toBeFalsy();
@@ -1429,9 +1471,11 @@ describe("ngModel", () => {
     it("should set ng-empty or ng-not-empty when the view value changes", () => {
       const element = $compile('<input ng-model="value" />')($rootScope);
 
+      $rootScope.$digest();
       expect(element.val()).toBe("");
 
       $rootScope.value = "XXX";
+      $rootScope.$digest();
       expect(element.val()).toBe("XXX");
 
       element.val("");
@@ -1448,6 +1492,7 @@ describe("ngModel", () => {
         $rootScope,
       );
 
+      $rootScope.$digest();
       expect(element[0].classList.contains("ng-valid")).toBeTrue();
       expect(element[0].classList.contains("ng-pristine")).toBeTrue();
       expect(element[0].classList.contains("ng-touched")).toBeFalse();
@@ -1485,6 +1530,8 @@ describe("ngModel", () => {
       const element = $compile(
         '<input type="email" ng-model="value" required />',
       )($rootScope);
+      $rootScope.$digest();
+
       expect(element[0].classList.contains("ng-invalid")).toBeTrue();
       expect(element[0].classList.contains("ng-invalid-required")).toBeTrue();
 

package/src/directive/model-options/model-options.spec.js

@@ -49,6 +49,8 @@ describe("ngModelOptions", () => {
         ]);
         $compile = injector.get("$compile");
         $rootScope = injector.get("$rootScope");
+        $q = injector.get("$q");
+        $timeout = injector.get("$timeout");
       });
 
       describe("should fall back to `defaultModelOptions`", () => {
@@ -271,6 +273,8 @@ describe("ngModelOptions", () => {
               '<input type="text" ng-model="name" name="alias" />' +
               "</form>",
           )($rootScope);
+          $rootScope.$digest();
+
           inputElm = doc.find("input");
           changeGivenInputTo(inputElm, "a");
           expect($rootScope.name).toEqual(undefined);
@@ -931,6 +935,7 @@ describe("ngModelOptions", () => {
           expect($rootScope.value).toBe("12345");
           expect($rootScope.form.input.$pending.promiseValidator).toBe(true);
           defer.reject();
+          $rootScope.$digest();
           expect($rootScope.value).toBe("12345");
           expect(inputElm[0].classList.contains("ng-invalid")).toBeTrue();
         });
@@ -953,6 +958,7 @@ describe("ngModelOptions", () => {
           expect(inputElm.val()).toBe("12345");
           expect($rootScope.form.input.$pending.promiseValidator).toBe(true);
           defer.reject();
+          $rootScope.$digest();
           expect(inputElm.val()).toBe("12345");
           expect(inputElm[0].classList.contains("ng-invalid")).toBeTrue();
         });

package/src/directive/non-bindable/non-bindable.spec.js

@@ -43,6 +43,7 @@ describe("ngNonBindable", () => {
     )($rootScope);
     $rootScope.a = "one";
     $rootScope.b = "two";
+    $rootScope.$digest();
     // Bindings not contained by ng-non-bindable should resolve.
     const spans = element.find("span");
     expect(spans.eq(0).text()).toEqual("one");

package/src/directive/observe/observe.js

@@ -20,6 +20,7 @@ export function ngObserveDirective() {
         ).getAttribute(source);
         if (scope[prop] !== newValue) {
           scope[prop] = newValue;
+          scope.$digest();
         }
       });
 

package/src/directive/observe/observe.spec.js

@@ -25,6 +25,7 @@ describe("observe", () => {
   function createDirective(attributeValue, updateProp) {
     const template = `<div ng-observe="${attributeValue}" data-update="${updateProp}"></div>`;
     element = $compile(template)($scope);
+    $scope.$digest();
   }
 
   it("should set the scope property to the attribute value before any changes", () => {