From 176bcaa928dde1c8bb00fa078ab4446dddf4de1a Mon Sep 17 00:00:00 2001
From: =?utf8?q?Pau=20Ferrer=20Oca=C3=B1a?= <crazyserver@gmail.com>
Date: Fri, 11 Mar 2016 12:58:17 +0100
Subject: [PATCH] MDL-53409 question: Add tests for core_question_update_flag

---
 question/tests/externallib_test.php | 41 +++++++++++++++++++++--------
 1 file changed, 30 insertions(+), 11 deletions(-)

diff --git a/question/tests/externallib_test.php b/question/tests/externallib_test.php
index ff19a0f9156..5fcf3989f5c 100644
--- a/question/tests/externallib_test.php
+++ b/question/tests/externallib_test.php
@@ -29,10 +29,10 @@ defined('MOODLE_INTERNAL') || die();
 global $CFG;
 
 require_once($CFG->dirroot . '/webservice/tests/helpers.php');
-require_once($CFG->dirroot . '/mod/wiki/lib.php');
+require_once($CFG->dirroot . '/question/engine/tests/helpers.php');
 
 /**
- * Wiki module external functions tests
+ * Question external functions tests
  *
  * @package    core_question
  * @category   external
@@ -48,10 +48,10 @@ class core_question_external_testcase extends externallib_advanced_testcase {
     public function setUp() {
         global $DB;
         $this->resetAfterTest();
+        $this->setAdminUser();
 
         // Setup test data.
         $this->course = $this->getDataGenerator()->create_course();
-        $this->wiki = $this->getDataGenerator()->create_module('wiki', array('course' => $this->course->id));
 
         // Create users.
         $this->student = self::getDataGenerator()->create_user();
@@ -59,9 +59,6 @@ class core_question_external_testcase extends externallib_advanced_testcase {
         // Users enrolments.
         $this->studentrole = $DB->get_record('role', array('shortname' => 'student'));
         $this->getDataGenerator()->enrol_user($this->student->id, $this->course->id, $this->studentrole->id, 'manual');
-
-        // Create first page.
-        $this->firstpage = $this->getDataGenerator()->get_plugin_generator('mod_wiki')->create_first_page($this->wiki);
     }
 
     /**
@@ -69,18 +66,40 @@ class core_question_external_testcase extends externallib_advanced_testcase {
      */
     public function test_core_question_update_flag() {
 
-        $question = test_question_maker::make_question('shortanswer');
-        $quba = question_engine::make_questions_usage_by_activity('test', context_system::instance());
+        $questiongenerator = $this->getDataGenerator()->get_plugin_generator('core_question');
+
+        // Create a question category.
+        $cat = $questiongenerator->create_question_category();
+
+        $quba = question_engine::make_questions_usage_by_activity('core_question_update_flag', context_system::instance());
+        $quba->set_preferred_behaviour('deferredfeedback');
+        $questiondata = $questiongenerator->create_question('numerical', null, array('category' => $cat->id));
+        $question = question_bank::load_question($questiondata->id);
         $slot = $quba->add_question($question);
-        $quba->start_all_questions();
         $qa = $quba->get_question_attempt($slot);
 
+        self::setUser($this->student);
+
+        $quba->start_all_questions();
+        question_engine::save_questions_usage_by_activity($quba);
+
         $qubaid = $quba->get_id();
         $questionid = $question->id;
-        $qaid = $qa->id;
+        $qaid = $qa->get_database_id();
         $checksum = md5($qubaid . "_" . $this->student->secret . "_" . $questionid . "_" . $qaid . "_" . $slot);
 
         $flag = core_question_external::update_flag($qubaid, $questionid, $qaid, $slot, $checksum, true);
         $this->assertTrue($flag['status']);
+
+        // Test invalid checksum.
+        try {
+            // Using random_string to force failing.
+            $checksum = md5($qubaid . "_" . random_string(11) . "_" . $questionid . "_" . $qaid . "_" . $slot);
+
+            core_question_external::update_flag($qubaid, $questionid, $qaid, $slot, $checksum, true);
+            $this->fail('Exception expected due to invalid checksum.');
+        } catch (moodle_exception $e) {
+            $this->assertEquals('errorsavingflags', $e->errorcode);
+        }
     }
-}
+}
\ No newline at end of file
-- 
2.17.1