chore(rating): remove redundant validation and add unit tests

Closes angular-ui#3464
fernando-sendMail · Jul 9, 2015 · ae8d865 · ae8d865
1 parent d026851
commit ae8d865
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/src/rating/rating.js b/src/rating/rating.js
@@ -81,10 +81,7 @@ angular.module('ui.bootstrap.rating', [])
     replace: true,
     link: function(scope, element, attrs, ctrls) {
       var ratingCtrl = ctrls[0], ngModelCtrl = ctrls[1];
-
-      if ( ngModelCtrl ) {
-        ratingCtrl.init( ngModelCtrl );
-      }
+      ratingCtrl.init( ngModelCtrl );
     }
   };
 });
diff --git a/src/rating/test/rating.spec.js b/src/rating/test/rating.spec.js
@@ -165,6 +165,17 @@ describe('rating directive', function () {
       expect($rootScope.rate).toBe(3);
     });
 
+    it('supports only arrow keys', function() {
+      $rootScope.rate = undefined;
+      $rootScope.$digest();
+
+      triggerKeyDown(36);
+      expect($rootScope.rate).toBe(undefined);
+
+      triggerKeyDown(41);
+      expect($rootScope.rate).toBe(undefined);
+    });
+
     it('can get zero value but not negative', function() {
       $rootScope.rate = 1;
       $rootScope.$digest();