From 5047b0ea8fe62a5ad7a6215bb267772ad7e31dc4 Mon Sep 17 00:00:00 2001
From: Tony Butler <tonyjbutler@gmail.com>
Date: Fri, 20 Feb 2015 15:21:54 +0000
Subject: [PATCH] MDL-49075 mod_assign: Add assertions to marking workflow unit
 test

Make sure grades aren't pushed to gradebook for marking workflow
states other than 'released'.
---
 mod/assign/tests/locallib_test.php | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/mod/assign/tests/locallib_test.php b/mod/assign/tests/locallib_test.php
index 19f3d7b1927..f2132cb975b 100644
--- a/mod/assign/tests/locallib_test.php
+++ b/mod/assign/tests/locallib_test.php
@@ -1787,6 +1787,10 @@ class mod_assign_locallib_testcase extends mod_assign_base_testcase {
         $output = $assign->view_student_summary($this->students[0], true);
         $this->assertEquals(false, strpos($output, '50.0'));
 
+        // Make sure the grade isn't pushed to the gradebook.
+        $grades = $assign->get_user_grades_for_gradebook($this->students[0]->id);
+        $this->assertEmpty($grades);
+
         // Mark the submission and set to inmarking.
         $this->setUser($this->teachers[0]);
         $data = new stdClass();
@@ -1799,6 +1803,10 @@ class mod_assign_locallib_testcase extends mod_assign_base_testcase {
         $output = $assign->view_student_summary($this->students[0], true);
         $this->assertEquals(false, strpos($output, '50.0'));
 
+        // Make sure the grade isn't pushed to the gradebook.
+        $grades = $assign->get_user_grades_for_gradebook($this->students[0]->id);
+        $this->assertEmpty($grades);
+
         // Mark the submission and set to readyforreview.
         $this->setUser($this->teachers[0]);
         $data = new stdClass();
@@ -1811,6 +1819,10 @@ class mod_assign_locallib_testcase extends mod_assign_base_testcase {
         $output = $assign->view_student_summary($this->students[0], true);
         $this->assertEquals(false, strpos($output, '50.0'));
 
+        // Make sure the grade isn't pushed to the gradebook.
+        $grades = $assign->get_user_grades_for_gradebook($this->students[0]->id);
+        $this->assertEmpty($grades);
+
         // Mark the submission and set to inreview.
         $this->setUser($this->teachers[0]);
         $data = new stdClass();
@@ -1823,6 +1835,10 @@ class mod_assign_locallib_testcase extends mod_assign_base_testcase {
         $output = $assign->view_student_summary($this->students[0], true);
         $this->assertEquals(false, strpos($output, '50.0'));
 
+        // Make sure the grade isn't pushed to the gradebook.
+        $grades = $assign->get_user_grades_for_gradebook($this->students[0]->id);
+        $this->assertEmpty($grades);
+
         // Mark the submission and set to readyforrelease.
         $this->setUser($this->teachers[0]);
         $data = new stdClass();
@@ -1835,6 +1851,10 @@ class mod_assign_locallib_testcase extends mod_assign_base_testcase {
         $output = $assign->view_student_summary($this->students[0], true);
         $this->assertEquals(false, strpos($output, '50.0'));
 
+        // Make sure the grade isn't pushed to the gradebook.
+        $grades = $assign->get_user_grades_for_gradebook($this->students[0]->id);
+        $this->assertEmpty($grades);
+
         // Mark the submission and set to released.
         $this->setUser($this->teachers[0]);
         $data = new stdClass();
@@ -1846,6 +1866,10 @@ class mod_assign_locallib_testcase extends mod_assign_base_testcase {
         $this->setUser($this->students[0]);
         $output = $assign->view_student_summary($this->students[0], true);
         $this->assertNotEquals(false, strpos($output, '50.0'));
+
+        // Make sure the grade is pushed to the gradebook.
+        $grades = $assign->get_user_grades_for_gradebook($this->students[0]->id);
+        $this->assertEquals(50, (int)$grades[$this->students[0]->id]->rawgrade);
     }
 
     public function test_markerallocation() {