test($compile): add test for undefined non-optional reference binding

petebacondarwin · petebacondarwin · commit b2a937d42528 · 2015-11-25T10:48:59.000Z
Demonstrates that angular#13373 fixes angular#13367
diff --git a/test/ng/compileSpec.js b/test/ng/compileSpec.js
@@ -3964,7 +3964,7 @@ describe('$compile', function() {
         expect(componentScope.ref).toBe('hello world');
 
         componentScope.ref = 'ignore me';
-        expect($rootScope.$apply).
+        expect(function() { $rootScope.$apply(); }).
             toThrowMinErr("$compile", "nonassign", "Expression ''hello ' + name' used with directive 'myComponent' is non-assignable!");
         expect(componentScope.ref).toBe('hello world');
         // reset since the exception was rethrown which prevented phase clearing
@@ -3975,6 +3975,21 @@ describe('$compile', function() {
         expect(componentScope.ref).toBe('hello misko');
       }));
 
+      it('should complain if assigning to undefined', inject(function() {
+        compile('<div><span my-component>');
+        $rootScope.$apply();
+        expect(componentScope.ref).toBeUndefined();
+
+        componentScope.ref = 'ignore me';
+        expect(function() { $rootScope.$apply(); }).
+            toThrowMinErr("$compile", "nonassign", "Expression 'undefined' used with directive 'myComponent' is non-assignable!");
+        expect(componentScope.ref).toBeUndefined();
+
+        $rootScope.$$phase = null; // reset since the exception was rethrown which prevented phase clearing
+        $rootScope.$apply();
+        expect(componentScope.ref).toBeUndefined();
+      }));
+
       // regression
       it('should stabilize model', inject(function() {
         compile('<div><span my-component reference="name">');
