updated quiz controller and added test cases for quiz controller

sridharpg · Dec 28, 2012 · fe29a5b · fe29a5b
1 parent cb43173
commit fe29a5b
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 4 deletions.
diff --git a/app/js/controllers/quizCtrl.js b/app/js/controllers/quizCtrl.js
@@ -19,7 +19,7 @@ quizApp.controller('QuizCtrl', function QuizCtrl($rootScope, $scope, $resource,
     });
 
     $scope.hasNext = function () {
-        return ($scope.currentPosition >= $scope.quiz.questionnaire.length - 1);
+        return !($scope.currentPosition >= $scope.quiz.questionnaire.length - 1);
     };
 
     $scope.updatePage = function () {
@@ -49,7 +49,7 @@ quizApp.controller('QuizCtrl', function QuizCtrl($rootScope, $scope, $resource,
 
     $scope.next = function () {
         var valid = $scope.hasNext();
-        if (valid !== true) {
+        if (valid === true) {
             $scope.currentResponse = "";
             $scope.updatePage();
             $rootScope.$broadcast('restart_timer');

diff --git a/tests/unit/ControllerSpec.js b/tests/unit/ControllerSpec.js
@@ -12,12 +12,12 @@ describe('App controllers', function () {
 
         it('should be valid name', function () {
             scope.userName = "testUser";
-            expect(scope.isValidUserName).toBeTruthy();
+            expect(scope.isValidUserName()).toBeTruthy();
         });
 
         it('should ignore all spaces as name', function () {
             scope.userName = "      ";
-            expect(scope.isValidUserName).toBeFalsy();
+            expect(scope.isValidUserName()).toBeFalsy();
         });
 
     });
@@ -62,6 +62,30 @@ describe('App controllers', function () {
             expect(scope.quiz.questionnaire.length).toBe(2);
         });
 
+        it('should be initialized correctly', function () {
+            $httpMock.flush();
+            expect(scope.currentPosition).toBe(0);
+            expect(scope.currentQuestion).toBeDefined();
+            expect(scope.hasNext()).toBeTruthy();
+            expect(scope.isAnswered()).toBeFalsy();
+        });
+
+        it('should add to the score for correct answer', function () {
+            $httpMock.flush();
+            scope.currentResponse = "false";
+            expect(scope.user.score).toBe(0);
+            scope.submitAns(scope.currentQuestion.id);
+            expect(scope.user.score).toBe(2);
+        });
+
+        it('should not add to the score for wrong answer', function () {
+            $httpMock.flush();
+            scope.currentResponse = "true";
+            expect(scope.user.score).toBe(0);
+            scope.submitAns(scope.currentQuestion.id);
+            expect(scope.user.score).toBe(0);
+        });
+
     });
 
 });