Trial to solve moveNextAssessmentSection/TestPart if last item

is timed out...
oat-sa · Sep 21, 2015 · f5f1697 · f5f1697
1 parent 5f692c1
commit f5f1697
Show file tree

Hide file tree

Showing 6 changed files with 130 additions and 19 deletions.
diff --git a/composer.json b/composer.json
@@ -2,7 +2,7 @@
 	"name": "qtism/qtism",
 	"description": "OAT QTI Software Module Library",
 	"type": "library",
-	"version": "0.9.12",
+	"version": "0.9.13",
 	"authors": [
 		{
 			"name": "Open Assessment Technologies S.A.",

diff --git a/qtism/runtime/tests/AssessmentTestSession.php b/qtism/runtime/tests/AssessmentTestSession.php
@@ -1653,14 +1653,18 @@ public function moveNextTestPart() {
 	    $route = $this->getRoute();
 	    $from = $route->current();
 
-        $route->next();
-	    while ($route->valid() === true && $route->current()->getTestPart() === $from->getTestPart()) {
-	        $this->nextRouteItem();
-	    }
-
-	    if ($this->isRunning() === true) {
-	        $this->interactWithItemSession();
-	    }
+        if ($route->isLast() === true) {
+            $this->endTestSession();
+        } else {
+            $route->next();
+            while ($route->valid() === true && $route->current()->getTestPart() === $from->getTestPart()) {
+                $this->nextRouteItem();
+            }
+
+            if ($this->isRunning() === true) {
+                $this->interactWithItemSession();
+            }
+        }
 	}
 
 	/**
@@ -1681,14 +1685,18 @@ public function moveNextAssessmentSection() {
 	    $route = $this->getRoute();
 	    $from = $route->current();
 
-        $route->next();
-	    while ($route->valid() === true && $route->current()->getAssessmentSection() === $from->getAssessmentSection()) {
-	        $this->nextRouteItem();
-	    }
-
-	    if ($this->isRunning() === true) {
-	        $this->interactWithItemSession();
-	    }
+        if ($route->isLast() === true) {
+            $this->endTestSession();
+        } else {
+            $route->next();
+            while ($route->valid() === true && $route->current()->getAssessmentSection() === $from->getAssessmentSection()) {
+                $this->nextRouteItem();
+            }
+
+            if ($this->isRunning() === true) {
+                $this->interactWithItemSession();
+            }
+        }
 	}
 
 	/**
@@ -2526,4 +2534,4 @@ protected function buildCurrentItemSessionIdentifier() {
 	protected function timeLimitsInForce($excludeItem = false) {
 	    return count($this->getCurrentRouteItem()->getTimeLimits($excludeItem)) !== 0;
 	}
-}
+}
diff --git a/test/qtism/runtime/tests/AssessmentTestSessionTimingTest.php b/test/qtism/runtime/tests/AssessmentTestSessionTimingTest.php
@@ -334,4 +334,35 @@ public function testMultipleOccurences() {
         $this->assertEquals(2, $session['Q01.2.duration']->getSeconds(true));
         $this->assertEquals(0, $session['Q01.3.duration']->getSeconds(true));
     }
- }
+
+    public function testLastItemTimeout() {
+        $session = self::instantiate(self::samplesDir() . 'custom/runtime/timings/last_item_timeout.xml');
+        $session->beginTestSession();
+
+        $session->beginAttempt();
+        sleep(2);
+        $session->moveNext();
+        $this->assertEquals(AssessmentTestSessionState::CLOSED, $session->getState());
+    }
+
+    public function testLastItemSectionTimeout() {
+        $session = self::instantiate(self::samplesDir() . 'custom/runtime/timings/last_item_section_timeout.xml');
+        $session->beginTestSession();
+
+        $session->beginAttempt();
+        sleep(2);
+        $session->moveNextAssessmentSection();
+        $this->assertEquals(AssessmentTestSessionState::CLOSED, $session->getState());
+    }
+
+    public function testLastItemTestPartTimeout() {
+        $session = self::instantiate(self::samplesDir() . 'custom/runtime/timings/last_item_testpart_timeout.xml');
+        $session->beginTestSession();
+
+        $session->beginAttempt();
+        sleep(2);
+        $session->moveNextTestPart();
+        $this->assertEquals(AssessmentTestSessionState::CLOSED, $session->getState());
+    }
+ }
+
diff --git a/test/samples/custom/runtime/timings/last_item_section_timeout.xml b/test/samples/custom/runtime/timings/last_item_section_timeout.xml
@@ -0,0 +1,24 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<assessmentTest xmlns="http://www.imsglobal.org/xsd/imsqti_v2p1"
+	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+	xsi:schemaLocation="http://www.imsglobal.org/xsd/imsqti_v2p1 http://www.taotesting.com/xsd/qticompact_v1p0.xsd"
+	identifier="last_item_section_timeout" title="Last Item Section Timeout">
+	<testPart identifier="TP01" navigationMode="nonlinear" submissionMode="individual">
+		<assessmentSection identifier="S01" fixed="false" title="Section 01" visible="true">
+            <timeLimits maxTime="1"/>
+			<assessmentItemRef identifier="Q01" href="Q01.xml" fixed="false" adaptive="false" timeDependent="false">
+				<responseDeclaration identifier="RESPONSE" cardinality="single" baseType="identifier">
+					<correctResponse>
+						<value>ChoiceB</value>
+					</correctResponse>
+				</responseDeclaration>
+				<outcomeDeclaration identifier="SCORE" cardinality="single" baseType="float">
+					<defaultValue>
+						<value>0.0</value>
+					</defaultValue>
+				</outcomeDeclaration>
+				<responseProcessing template="http://www.imsglobal.org/question/qti_v2p1/rptemplates/match_correct"/>
+			</assessmentItemRef>
+		</assessmentSection>
+	</testPart>
+</assessmentTest>
diff --git a/test/samples/custom/runtime/timings/last_item_testpart_timeout.xml b/test/samples/custom/runtime/timings/last_item_testpart_timeout.xml
@@ -0,0 +1,24 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<assessmentTest xmlns="http://www.imsglobal.org/xsd/imsqti_v2p1"
+	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+	xsi:schemaLocation="http://www.imsglobal.org/xsd/imsqti_v2p1 http://www.taotesting.com/xsd/qticompact_v1p0.xsd"
+	identifier="last_item_testpart_timeout" title="Last Item Test Part Timeout">
+	<testPart identifier="TP01" navigationMode="nonlinear" submissionMode="individual">
+        <timeLimits maxTime="1"/>
+		<assessmentSection identifier="S01" fixed="false" title="Section 01" visible="true">
+			<assessmentItemRef identifier="Q01" href="Q01.xml" fixed="false" adaptive="false" timeDependent="false">
+				<responseDeclaration identifier="RESPONSE" cardinality="single" baseType="identifier">
+					<correctResponse>
+						<value>ChoiceB</value>
+					</correctResponse>
+				</responseDeclaration>
+				<outcomeDeclaration identifier="SCORE" cardinality="single" baseType="float">
+					<defaultValue>
+						<value>0.0</value>
+					</defaultValue>
+				</outcomeDeclaration>
+				<responseProcessing template="http://www.imsglobal.org/question/qti_v2p1/rptemplates/match_correct"/>
+			</assessmentItemRef>
+		</assessmentSection>
+	</testPart>
+</assessmentTest>
diff --git a/test/samples/custom/runtime/timings/last_item_timeout.xml b/test/samples/custom/runtime/timings/last_item_timeout.xml
@@ -0,0 +1,24 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<assessmentTest xmlns="http://www.imsglobal.org/xsd/imsqti_v2p1"
+	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+	xsi:schemaLocation="http://www.imsglobal.org/xsd/imsqti_v2p1 http://www.taotesting.com/xsd/qticompact_v1p0.xsd"
+	identifier="last_item_timeout" title="Last Item Timeout">
+	<testPart identifier="TP01" navigationMode="nonlinear" submissionMode="individual">
+		<assessmentSection identifier="S01" fixed="false" title="Section 01" visible="true">
+			<assessmentItemRef identifier="Q01" href="Q01.xml" fixed="false" adaptive="false" timeDependent="false">
+                <timeLimits maxTime="1"/>
+				<responseDeclaration identifier="RESPONSE" cardinality="single" baseType="identifier">
+					<correctResponse>
+						<value>ChoiceB</value>
+					</correctResponse>
+				</responseDeclaration>
+				<outcomeDeclaration identifier="SCORE" cardinality="single" baseType="float">
+					<defaultValue>
+						<value>0.0</value>
+					</defaultValue>
+				</outcomeDeclaration>
+				<responseProcessing template="http://www.imsglobal.org/question/qti_v2p1/rptemplates/match_correct"/>
+			</assessmentItemRef>
+		</assessmentSection>
+	</testPart>
+</assessmentTest>