WebSVN – Moodle – Diff – /question/tests/privacy/provider_test.php

+ *
  * @package    core_question
  * @copyright  2018 Andrew Nicols <andrew@nicols.co.uk>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class provider_test extends \core_privacy\tests\provider_testcase {
+final class provider_test extends \core_privacy\tests\provider_testcase {
     // Include the privacy helper which has assertions on it.
     use \core_question_privacy_helper;
         $otheruser = $this->getDataGenerator()->create_user();
         // Create one question as each user in diferent contexts.
         $this->setUser($user);
         $userdata = $questiongenerator->setup_course_and_questions();
-        $expectedcontext = \context_course::instance($userdata[1]->id);
+        $expectedcontext = \context_module::instance($userdata[4]->cmid);
         $this->setUser($otheruser);
         $otheruserdata = $questiongenerator->setup_course_and_questions();
-        $unexpectedcontext = \context_course::instance($otheruserdata[1]->id);
+        $unexpectedcontext = \context_module::instance($otheruserdata[4]->cmid);
         // And create another one where we'll update a question as the test user.
         $moreotheruserdata = $questiongenerator->setup_course_and_questions();
-        $otherexpectedcontext = \context_course::instance($moreotheruserdata[1]->id);
+        $otherexpectedcontext = \context_module::instance($moreotheruserdata[4]->cmid);
         $expectedcontexts = [
                 $expectedcontext->id,
                 $otherexpectedcontext->id,
+            ];
+        $this->assertEqualsCanonicalizing(
-            ];
+            $expectedcontexts,
+            array_values($contextlist->get_contextids()),
+            'Contexts not equal',
-        $this->assertEqualsCanonicalizing($expectedcontexts, $contextlist->get_contextids(), 'Contexts not equal');
+        );
         // Run the export_user_Data as the test user.
         $user = \core_user::get_user_by_username('admin');
         $otheruser = $this->getDataGenerator()->create_user();
+        $course = $this->getDataGenerator()->create_course();
-        $course = $this->getDataGenerator()->create_course();
+        $qbank1 = self::getDataGenerator()->create_module('qbank', ['course' => $course->id]);
-        $context = \context_course::instance($course->id);
+        $qbank1context = \context_module::instance($qbank1->cmid);
-        $othercourse = $this->getDataGenerator()->create_course();
+        $qbank2 = self::getDataGenerator()->create_module('qbank', ['course' => $course->id]);
-        $othercontext = \context_course::instance($othercourse->id);
+        $qbank2context = \context_module::instance($qbank2->cmid);
         // Create a couple of questions.
         $questiongenerator = $this->getDataGenerator()->get_plugin_generator('core_question');
         $cat = $questiongenerator->create_question_category([
-            'contextid' => $context->id,
+            'contextid' => $qbank1context->id,
         ]);
         $othercat = $questiongenerator->create_question_category([
-            'contextid' => $othercontext->id,
+            'contextid' => $qbank2context->id,
         ]);
         $q5 = $questiongenerator->create_question('shortanswer', null, ['category' => $othercat->id]);
         $approvedcontextlist = new \core_privacy\tests\request\approved_contextlist(
             $user,
             'core_question',
-            [$context->id]
+            [$qbank1context->id]
         );
         $user = \core_user::get_user_by_username('admin');
         $otheruser = $this->getDataGenerator()->create_user();
+        $course = $this->getDataGenerator()->create_course();
-        $course = $this->getDataGenerator()->create_course();
+        $qbank1 = self::getDataGenerator()->create_module('qbank', ['course' => $course->id]);
-        $context = \context_course::instance($course->id);
+        $qbank1context = \context_module::instance($qbank1->cmid);
-        $othercourse = $this->getDataGenerator()->create_course();
+        $qbank2 = self::getDataGenerator()->create_module('qbank', ['course' => $course->id]);
-        $othercontext = \context_course::instance($othercourse->id);
+        $qbank2context = \context_module::instance($qbank2->cmid);
         // Create a couple of questions.
         $questiongenerator = $this->getDataGenerator()->get_plugin_generator('core_question');
         $cat = $questiongenerator->create_question_category([
-            'contextid' => $context->id,
+            'contextid' => $qbank1context->id,
         ]);
         $othercat = $questiongenerator->create_question_category([
-            'contextid' => $othercontext->id,
+            'contextid' => $qbank2context->id,
         ]);
         // Find out how many questions are in the question bank to start with.
         $questioncount = $DB->count_records('question');
         // Delete the data and check it is removed.
         $this->setUser();
-        provider::delete_data_for_all_users_in_context($context);
+        provider::delete_data_for_all_users_in_context($qbank1context);
         $this->assertEquals($questioncount, $DB->count_records('question'));
         $this->setUser($user1);
         $user1data = $questiongenerator->setup_course_and_questions();
         $this->setUser($user2);
         $user2data = $questiongenerator->setup_course_and_questions();
-        $course1context = \context_course::instance($user1data[1]->id);
+        $qbankcontext = \context_module::instance($user1data[4]->cmid);
-        $course1questions = $user1data[3];
+        $questions = $user1data[3];
         // Log in as user3 and update the questions in course1.
         $this->setUser($user3);
-        foreach ($course1questions as $question) {
+        foreach ($questions as $question) {
             $questiongenerator->update_question($question);
+        }
-        $userlist = new \core_privacy\local\request\userlist($course1context, 'core_question');
+        $userlist = new \core_privacy\local\request\userlist($qbankcontext, 'core_question');
         $user2 = $this->getDataGenerator()->create_user();
         $user3 = $this->getDataGenerator()->create_user();
         // Create one question as each user in different contexts.
         $this->setUser($user1);
-        $course1data = $questiongenerator->setup_course_and_questions();
+        $coursedata = $questiongenerator->setup_course_and_questions();
-        $course1 = $course1data[1];
+        $qbank = $coursedata[4];
-        $course1qcat = $course1data[2];
+        $course1qcat = $coursedata[2];
-        $course1questions = $course1data[3];
+        $questions = $coursedata[3];
-        $course1context = \context_course::instance($course1->id);
+        $qbankcontext = \context_module::instance($qbank->cmid);
         // Log in as user2 and update the questions in course1.
         $this->setUser($user2);
-        foreach ($course1questions as $question) {
+        foreach ($questions as $question) {
             $questiongenerator->update_question($question);
         // Now, log in as user1 again, and then create a new course and add questions to that.
         $this->setUser($user1);
         $questiongenerator->setup_course_and_questions();
-        $approveduserlist = new \core_privacy\local\request\approved_userlist($course1context, 'core_question',
+        $approveduserlist = new \core_privacy\local\request\approved_userlist($qbankcontext, 'core_question',
                 [$user1->id, $user2->id]);
         provider::delete_data_for_users($approveduserlist);
                                           JOIN {question_versions} qv ON qv.questionid = q.id
                                           JOIN {question_bank_entries} qbe ON qbe.id = qv.questionbankentryid
                                           JOIN {question_categories} qc ON qc.id = qbe.questioncategoryid
                                          WHERE qc.contextid = ?
                                            AND (q.createdby = ? OR q.modifiedby = ? OR q.createdby = ? OR q.modifiedby = ?)",
-                        [$course1context->id, $user1->id, $user1->id, $user2->id, $user2->id])
+                        [$qbankcontext->id, $user1->id, $user1->id, $user2->id, $user2->id])
         );
         // User3 data in course1 should not change.
         $this->assertEquals(2,
                 $DB->count_records_sql("SELECT COUNT(q.id)
                                           FROM {question} q
                                           JOIN {question_versions} qv ON qv.questionid = q.id
                                           JOIN {question_bank_entries} qbe ON qbe.id = qv.questionbankentryid
                                           JOIN {question_categories} qc ON qc.id = qbe.questioncategoryid
                                          WHERE qc.contextid = ? AND (q.createdby = ? OR q.modifiedby = ?)",
-                        [$course1context->id, $user3->id, $user3->id])
+                        [$qbankcontext->id, $user3->id, $user3->id])
         );
         // User1 has authored 2 questions in another course.

Proyectos de Subversion Moodle

(root)/question/tests/privacy/provider_test.php – Rev 11 → 1441