Rev 11 | AutorÃa | Comparar con el anterior | Ultima modificación | Ver Log |
<?php// This file is part of Moodle - http://moodle.org///// Moodle is free software: you can redistribute it and/or modify// it under the terms of the GNU General Public License as published by// the Free Software Foundation, either version 3 of the License, or// (at your option) any later version.//// Moodle is distributed in the hope that it will be useful,// but WITHOUT ANY WARRANTY; without even the implied warranty of// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the// GNU General Public License for more details.//// You should have received a copy of the GNU General Public License// along with Moodle. If not, see <http://www.gnu.org/licenses/>.namespace mod_h5pactivity\local;use context_module;use stdClass;/*** Manager tests class for mod_h5pactivity.** @package mod_h5pactivity* @covers \mod_h5pactivity\local\manager* @category test* @copyright 2020 Ferran Recio <ferran@moodle.com>* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later*/final class manager_test extends \advanced_testcase {/*** Test for static create methods.*/public function test_create(): void {$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity', ['course' => $course]);$cm = get_coursemodule_from_id('h5pactivity', $activity->cmid, 0, false, MUST_EXIST);$context = context_module::instance($cm->id);$manager = manager::create_from_instance($activity);$manageractivity = $manager->get_instance();$this->assertEquals($activity->id, $manageractivity->id);$managercm = $manager->get_coursemodule();$this->assertEquals($cm->id, $managercm->id);$managercontext = $manager->get_context();$this->assertEquals($context->id, $managercontext->id);$manager = manager::create_from_coursemodule($cm);$manageractivity = $manager->get_instance();$this->assertEquals($activity->id, $manageractivity->id);$managercm = $manager->get_coursemodule();$this->assertEquals($cm->id, $managercm->id);$managercontext = $manager->get_context();$this->assertEquals($context->id, $managercontext->id);}/*** Test for is_tracking_enabled and can_submit methods.** @covers ::is_tracking_enabled* @covers ::can_submit* @dataProvider is_tracking_enabled_data* @param bool $login if the user is logged in* @param string $role user role in course* @param int $enabletracking if tracking is enabled* @param bool $expectedtracking expected result for is_tracking_enabled()* @param bool $expectedsubmit expected result for can_submit()*/public function test_is_tracking_enabled_and_can_submit(bool $login, string $role, int $enabletracking, bool $expectedtracking,bool $expectedsubmit): void {$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity',['course' => $course, 'enabletracking' => $enabletracking]);$user = $this->getDataGenerator()->create_and_enrol($course, $role);if ($login) {$this->setUser($user);$param = null;} else {$param = $user;}$manager = manager::create_from_instance($activity);$this->assertEquals($expectedtracking, $manager->is_tracking_enabled());$this->assertEquals($expectedsubmit, $manager->can_submit($param));}/*** Data provider for test_is_tracking_enabled_and_can_submit.** @return array*/public static function is_tracking_enabled_data(): array {return ['Logged student, tracking enabled' => [true, 'student', 1, true, true,],'Logged student, tracking disabled' => [true, 'student', 0, false, true,],'Logged teacher, tracking enabled' => [true, 'editingteacher', 1, true, false,],'Logged teacher, tracking disabled' => [true, 'editingteacher', 0, false, false,],'No logged student, tracking enabled' => [true, 'student', 1, true, true,],'No logged student, tracking disabled' => [true, 'student', 0, false, true,],'No logged teacher, tracking enabled' => [true, 'editingteacher', 1, true, false,],'No logged teacher, tracking disabled' => [true, 'editingteacher', 0, false, false,],];}/*** Test for get_users_scaled_score.** @dataProvider get_users_scaled_score_data* @param int $enabletracking if tracking is enabled* @param int $gradingmethod new grading method* @param array $result1 student 1 results (scaled, timemodified, attempt number)* @param array $result2 student 2 results (scaled, timemodified, attempt number)*/public function test_get_users_scaled_score(int $enabletracking, int $gradingmethod, array $result1, array $result2): void {global $DB;$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity',['course' => $course, 'enabletracking' => $enabletracking, 'grademethod' => $gradingmethod]);// Generate two users with 4 attempts each.$user1 = $this->getDataGenerator()->create_and_enrol($course, 'student');$this->generate_fake_attempts($activity, $user1, 1);$user2 = $this->getDataGenerator()->create_and_enrol($course, 'student');$this->generate_fake_attempts($activity, $user2, 2);$manager = manager::create_from_instance($activity);// Get all users scaled scores.$scaleds = $manager->get_users_scaled_score();// No results will be returned if tracking is dsabled or manual grading method is defined.if (empty($result1)) {$this->assertNull($scaleds);return;}$this->assertCount(2, $scaleds);// Check expected user1 scaled score.$scaled = $scaleds[$user1->id];$this->assertEquals($user1->id, $scaled->userid);$this->assertEquals($result1[0], $scaled->scaled);$this->assertEquals($result1[1], $scaled->timemodified);if ($result1[2]) {$attempt = $DB->get_record('h5pactivity_attempts', ['id' => $scaled->attemptid]);$this->assertEquals($attempt->h5pactivityid, $activity->id);$this->assertEquals($attempt->userid, $scaled->userid);$this->assertEquals($attempt->scaled, round($scaled->scaled, 5));$this->assertEquals($attempt->timemodified, $scaled->timemodified);$this->assertEquals($result1[2], $attempt->attempt);} else {$this->assertEquals(0, $scaled->attemptid);}// Check expected user2 scaled score.$scaled = $scaleds[$user2->id];$this->assertEquals($user2->id, $scaled->userid);$this->assertEquals($result2[0], round($scaled->scaled, 5));$this->assertEquals($result2[1], $scaled->timemodified);if ($result2[2]) {$attempt = $DB->get_record('h5pactivity_attempts', ['id' => $scaled->attemptid]);$this->assertEquals($attempt->h5pactivityid, $activity->id);$this->assertEquals($attempt->userid, $scaled->userid);$this->assertEquals($attempt->scaled, $scaled->scaled);$this->assertEquals($attempt->timemodified, $scaled->timemodified);$this->assertEquals($result2[2], $attempt->attempt);} else {$this->assertEquals(0, $scaled->attemptid);}// Now check a single user record.$scaleds = $manager->get_users_scaled_score($user2->id);$this->assertCount(1, $scaleds);$scaled2 = $scaleds[$user2->id];$this->assertEquals($scaled->userid, $scaled2->userid);$this->assertEquals($scaled->scaled, $scaled2->scaled);$this->assertEquals($scaled->attemptid, $scaled2->attemptid);$this->assertEquals($scaled->timemodified, $scaled2->timemodified);}/*** Data provider for get_users_scaled_score.** @return array*/public static function get_users_scaled_score_data(): array {return ['Tracking with max attempt method' => [1, manager::GRADEHIGHESTATTEMPT, [1.00000, 31, 2], [0.66667, 32, 2]],'Tracking with average attempt method' => [1, manager::GRADEAVERAGEATTEMPT, [0.61111, 51, 0], [0.52222, 52, 0]],'Tracking with last attempt method' => [1, manager::GRADELASTATTEMPT, [0.33333, 51, 3], [0.40000, 52, 3]],'Tracking with first attempt method' => [1, manager::GRADEFIRSTATTEMPT, [0.50000, 11, 1], [0.50000, 12, 1]],'Tracking with manual attempt grading' => [1, manager::GRADEMANUAL, [], []],'No tracking with max attempt method' => [0, manager::GRADEHIGHESTATTEMPT, [], []],'No tracking with average attempt method' => [0, manager::GRADEAVERAGEATTEMPT, [], []],'No tracking with last attempt method' => [0, manager::GRADELASTATTEMPT, [], []],'No tracking with first attempt method' => [0, manager::GRADEFIRSTATTEMPT, [], []],'No tracking with manual attempt grading' => [0, manager::GRADEMANUAL, [], []],];}/*** Test static get_grading_methods.*/public function test_get_grading_methods(): void {$methods = manager::get_grading_methods();$this->assertCount(5, $methods);$this->assertNotEmpty($methods[manager::GRADEHIGHESTATTEMPT]);$this->assertNotEmpty($methods[manager::GRADEAVERAGEATTEMPT]);$this->assertNotEmpty($methods[manager::GRADELASTATTEMPT]);$this->assertNotEmpty($methods[manager::GRADEFIRSTATTEMPT]);$this->assertNotEmpty($methods[manager::GRADEMANUAL]);}/*** Test static get_selected_attempt.** @dataProvider get_selected_attempt_data* @param int $enabletracking if tracking is enabled* @param int $gradingmethod new grading method* @param int $result the expected result*/public function test_get_selected_attempt(int $enabletracking, int $gradingmethod, int $result): void {$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity',['course' => $course, 'enabletracking' => $enabletracking, 'grademethod' => $gradingmethod]);$manager = manager::create_from_instance($activity);$selected = $manager->get_selected_attempt();$this->assertEquals($result, $selected[0]);$this->assertNotEmpty($selected[1]);}/*** Data provider for get_users_scaled_score.** @return array*/public static function get_selected_attempt_data(): array {return ['Tracking with max attempt method' => [1, manager::GRADEHIGHESTATTEMPT, manager::GRADEHIGHESTATTEMPT],'Tracking with average attempt method' => [1, manager::GRADEAVERAGEATTEMPT, manager::GRADEAVERAGEATTEMPT],'Tracking with last attempt method' => [1, manager::GRADELASTATTEMPT, manager::GRADELASTATTEMPT],'Tracking with first attempt method' => [1, manager::GRADEFIRSTATTEMPT, manager::GRADEFIRSTATTEMPT],'Tracking with manual attempt grading' => [1, manager::GRADEMANUAL, manager::GRADEMANUAL],'No tracking with max attempt method' => [0, manager::GRADEHIGHESTATTEMPT, manager::GRADEMANUAL],'No tracking with average attempt method' => [0, manager::GRADEAVERAGEATTEMPT, manager::GRADEMANUAL],'No tracking with last attempt method' => [0, manager::GRADELASTATTEMPT, manager::GRADEMANUAL],'No tracking with first attempt method' => [0, manager::GRADEFIRSTATTEMPT, manager::GRADEMANUAL],'No tracking with manual attempt grading' => [0, manager::GRADEMANUAL, manager::GRADEMANUAL],];}/*** Test static get_review_modes.*/public function test_get_review_modes(): void {$methods = manager::get_review_modes();$this->assertCount(2, $methods);$this->assertNotEmpty($methods[manager::REVIEWCOMPLETION]);$this->assertNotEmpty($methods[manager::REVIEWNONE]);}/*** Test get_grader method.*/public function test_get_grader(): void {$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity', ['course' => $course]);$cm = get_coursemodule_from_id('h5pactivity', $activity->cmid, 0, false, MUST_EXIST);$context = context_module::instance($cm->id);$manager = manager::create_from_instance($activity);$grader = $manager->get_grader();$this->assertInstanceOf('mod_h5pactivity\local\grader', $grader);}/*** Test static can_view_all_attempts.** @dataProvider can_view_all_attempts_data* @param int $enabletracking if tracking is enabled* @param bool $usestudent if test must be done with a user role* @param bool $useloggedin if test must be done with the loggedin user* @param bool $result the expected result*/public function test_can_view_all_attempts(int $enabletracking, bool $usestudent, bool $useloggedin, bool $result): void {global $USER;$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity',['course' => $course, 'enabletracking' => $enabletracking]);$manager = manager::create_from_instance($activity);$user = $this->getDataGenerator()->create_and_enrol($course, 'student');$loggedin = $USER;// We want to test what when the method is called to check a different user than $USER.if (!$usestudent) {$loggedin = $user;$user = $USER;}if ($useloggedin) {$this->setUser($user);$user = null;} else {$this->setUser($loggedin);}$this->assertEquals($result, $manager->can_view_all_attempts($user));}/*** Data provider for test_can_view_all_attempts.** @return array*/public static function can_view_all_attempts_data(): array {return [// No tracking cases.'No tracking with admin using $USER' => [0, false, false, false],'No tracking with student using $USER' => [0, true, false, false],'No tracking with admin loggedin' => [0, false, true, false],'No tracking with student loggedin' => [0, true, true, false],// Tracking enabled cases.'Tracking with admin using $USER' => [1, false, false, true],'Tracking with student using $USER' => [1, true, false, false],'Tracking with admin loggedin' => [1, false, true, true],'Tracking with student loggedin' => [1, true, true, false],];}/*** Test static can_view_own_attempts.** @dataProvider can_view_own_attempts_data* @param int $enabletracking if tracking is enabled* @param int $reviewmode the attempt review mode* @param bool $useloggedin if test must be done with the loggedin user* @param bool $hasattempts if the student have attempts* @param bool $result the expected result*/public function test_can_view_own_attempts(int $enabletracking, int $reviewmode,bool $useloggedin, bool $hasattempts, bool $result): void {$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity',['course' => $course, 'enabletracking' => $enabletracking, 'reviewmode' => $reviewmode]);$manager = manager::create_from_instance($activity);$user = $this->getDataGenerator()->create_and_enrol($course, 'student');if ($hasattempts) {$this->generate_fake_attempts($activity, $user, 1);}if ($useloggedin) {$this->setUser($user);$user = null;}$this->assertEquals($result, $manager->can_view_own_attempts($user));}/*** Data provider for test_can_view_own_attempts.** @return array*/public static function can_view_own_attempts_data(): array {return [// No tracking cases.'No tracking, review none, using $USER, without attempts' => [0, manager::REVIEWNONE, false, false, false],'No tracking, review enabled, using $USER, without attempts' => [0, manager::REVIEWCOMPLETION, false, false, false],'No tracking, review none, loggedin, without attempts' => [0, manager::REVIEWNONE, true, false, false],'No tracking, review enabled, loggedin, without attempts' => [0, manager::REVIEWCOMPLETION, true, false, false],'No tracking, review none, using $USER, with attempts' => [0, manager::REVIEWNONE, false, true, false],'No tracking, review enabled, using $USER, with attempts' => [0, manager::REVIEWCOMPLETION, false, true, false],'No tracking, review none, loggedin, with attempts' => [0, manager::REVIEWNONE, true, true, false],'No tracking, review enabled, loggedin, with attempts' => [0, manager::REVIEWCOMPLETION, true, true, false],// Tracking enabled cases.'Tracking enabled, review none, using $USER, without attempts' => [1, manager::REVIEWNONE, false, false, false],'Tracking enabled, review enabled, using $USER, without attempts' => [1, manager::REVIEWCOMPLETION, false, false, true],'Tracking enabled, review none, loggedin, without attempts' => [1, manager::REVIEWNONE, true, false, false],'Tracking enabled, review enabled, loggedin, without attempts' => [1, manager::REVIEWCOMPLETION, true, false, true],'Tracking enabled, review none, using $USER, with attempts' => [1, manager::REVIEWNONE, false, true, false],'Tracking enabled, review enabled, using $USER, with attempts' => [1, manager::REVIEWCOMPLETION, false, true, true],'Tracking enabled, review none, loggedin, with attempts' => [1, manager::REVIEWNONE, true, true, false],'Tracking enabled, review enabled, loggedin, with attempts' => [1, manager::REVIEWCOMPLETION, true, true, true],];}/*** Test static count_attempts of one user.*/public function test_count_attempts(): void {$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity',['course' => $course]);$manager = manager::create_from_instance($activity);// User without attempts.$user1 = $this->getDataGenerator()->create_and_enrol($course, 'student');// User with 1 attempt.$user2 = $this->getDataGenerator()->create_and_enrol($course, 'student');$this->generate_fake_attempts($activity, $user2, 1);// User with 2 attempts.$user3 = $this->getDataGenerator()->create_and_enrol($course, 'student');$this->generate_fake_attempts($activity, $user3, 1);// Incomplete user2 and 3 has only 3 attempts completed.$this->assertEquals(0, $manager->count_attempts($user1->id));$this->assertEquals(3, $manager->count_attempts($user2->id));$this->assertEquals(3, $manager->count_attempts($user3->id));}/*** Test static count_attempts of all active participants.** @dataProvider count_attempts_all_data* @param bool $canview if the student role has mod_h5pactivity/view capability* @param bool $cansubmit if the student role has mod_h5pactivity/submit capability* @param bool $extrarole if an extra role without submit capability is required* @param int $result the expected result*/public function test_count_attempts_all(bool $canview, bool $cansubmit, bool $extrarole, int $result): void {global $DB;$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity',['course' => $course]);$manager = manager::create_from_instance($activity);$roleid = $DB->get_field('role', 'id', ['shortname' => 'student']);$newcap = ($canview) ? CAP_ALLOW : CAP_PROHIBIT;role_change_permission($roleid, $manager->get_context(), 'mod/h5pactivity:view', $newcap);$newcap = ($cansubmit) ? CAP_ALLOW : CAP_PROHIBIT;role_change_permission($roleid, $manager->get_context(), 'mod/h5pactivity:submit', $newcap);// Teacher with review capability and attempts (should not be listed).if ($extrarole) {$user1 = $this->getDataGenerator()->create_and_enrol($course, 'editingteacher');$this->generate_fake_attempts($activity, $user1, 1);}// Student with attempts.$user2 = $this->getDataGenerator()->create_and_enrol($course, 'student');$this->generate_fake_attempts($activity, $user2, 1);// Another student with attempts.$user3 = $this->getDataGenerator()->create_and_enrol($course, 'student');$this->generate_fake_attempts($activity, $user3, 1);$this->assertEquals($result, $manager->count_attempts());}/*** Data provider for test_count_attempts_all.** @return array*/public static function count_attempts_all_data(): array {return ['Students with both view and submit capability' => [true, true, false, 6],'Students without view but with submit capability' => [false, true, false, 0],'Students with view but without submit capability' => [true, false, false, 6],'Students without both view and submit capability' => [false, false, false, 0],'Students with both view and submit capability and extra role' => [true, true, true, 6],'Students without view but with submit capability and extra role' => [false, true, true, 0],'Students with view but without submit capability and extra role' => [true, false, true, 6],'Students without both view and submit capability and extra role' => [false, false, true, 0],];}/*** Test static test_get_active_users_join of all active participants.** Most method scenarios are tested in test_count_attempts_all so we only* need to test the with $allpotentialusers true and false.** @dataProvider get_active_users_join_data* @param bool $allpotentialusers if the join should return all potential users or only the submitted ones.* @param int $result the expected result*/public function test_get_active_users_join(bool $allpotentialusers, int $result): void {global $DB;$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity',['course' => $course]);$manager = manager::create_from_instance($activity);$user1 = $this->getDataGenerator()->create_and_enrol($course, 'editingteacher');$this->generate_fake_attempts($activity, $user1, 1);// Student with attempts.$user2 = $this->getDataGenerator()->create_and_enrol($course, 'student');$this->generate_fake_attempts($activity, $user2, 1);// 2 more students without attempts.$this->getDataGenerator()->create_and_enrol($course, 'student');$this->getDataGenerator()->create_and_enrol($course, 'student');$usersjoin = $manager->get_active_users_join($allpotentialusers);// Final SQL.$num = $DB->count_records_sql("SELECT COUNT(DISTINCT u.id)FROM {user} u $usersjoin->joinsWHERE $usersjoin->wheres",array_merge($usersjoin->params));$this->assertEquals($result, $num);}/*** Data provider for test_get_active_users_join.** @return array*/public static function get_active_users_join_data(): array {return ['All potential users' => ['allpotentialusers' => true,'result' => 3,],'Users with attempts' => ['allpotentialusers' => false,'result' => 1,],];}/*** Test active users joins returns appropriate results for groups*/public function test_get_active_users_join_groupmode(): void {global $DB;$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course(['groupmode' => SEPARATEGROUPS, 'groupmodeforce' => 1]);// Teacher/user one in group one.$teacher = $this->getDataGenerator()->create_and_enrol($course, 'teacher');$userone = $this->getDataGenerator()->create_and_enrol($course, 'student');$groupone = $this->getDataGenerator()->create_group(['courseid' => $course->id]);$this->getDataGenerator()->create_group_member(['groupid' => $groupone->id, 'userid' => $teacher->id]);$this->getDataGenerator()->create_group_member(['groupid' => $groupone->id, 'userid' => $userone->id]);// User two in group two.$usertwo = $this->getDataGenerator()->create_and_enrol($course, 'student');$grouptwo = $this->getDataGenerator()->create_group(['courseid' => $course->id]);$this->getDataGenerator()->create_group_member(['groupid' => $grouptwo->id, 'userid' => $usertwo->id]);// User three in no group.$userthree = $this->getDataGenerator()->create_and_enrol($course, 'student');// User four in a non-participation group.$userfour = $this->getDataGenerator()->create_and_enrol($course, 'student');$groupthree = $this->getDataGenerator()->create_group(['courseid' => $course->id, 'participation' => 0]);$this->getDataGenerator()->create_group_member(['groupid' => $groupthree->id, 'userid' => $userfour->id]);// Editing teacher in no group.$editingteacher = $this->getDataGenerator()->create_and_enrol($course, 'editingteacher');$activity = $this->getDataGenerator()->create_module('h5pactivity', ['course' => $course]);$manager = manager::create_from_instance($activity);// Admin user can view all participants (any group and none).$usersjoin = $manager->get_active_users_join(true, 0);$users = $DB->get_fieldset_sql("SELECT u.username FROM {user} u {$usersjoin->joins} WHERE {$usersjoin->wheres}",$usersjoin->params);$this->assertEqualsCanonicalizing([$userone->username, $usertwo->username, $userthree->username, $userfour->username], $users);// Switch to teacher, who cannot view all participants.$this->setUser($teacher);$usersjoin = $manager->get_active_users_join(true, 0);$users = $DB->get_fieldset_sql("SELECT u.username FROM {user} u {$usersjoin->joins} WHERE {$usersjoin->wheres}",$usersjoin->params);$this->assertEmpty($users);// Teacher can view participants inside group.$usersjoin = $manager->get_active_users_join(true, $groupone->id);$users = $DB->get_fieldset_sql("SELECT u.username FROM {user} u {$usersjoin->joins} WHERE {$usersjoin->wheres}",$usersjoin->params);$this->assertEqualsCanonicalizing([$userone->username], $users);// Switch to editing teacher, who can view all participants.$this->setUser($editingteacher);$usersjoin = $manager->get_active_users_join(true, 0);$users = $DB->get_fieldset_sql("SELECT u.username FROM {user} u {$usersjoin->joins} WHERE {$usersjoin->wheres}",$usersjoin->params);$this->assertEqualsCanonicalizing([$userone->username, $usertwo->username, $userthree->username, $userfour->username], $users);}/*** Test getting active users join where there are no roles with 'mod/h5pactivity:reviewattempts' capability*/public function test_get_active_users_join_no_reviewers(): void {global $DB;$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity', ['course' => $course]);$user = $this->getDataGenerator()->create_and_enrol($course, 'student');$manager = manager::create_from_instance($activity);// By default manager and editingteacher can review attempts, prohibit both.$rolemanager = $DB->get_field('role', 'id', ['shortname' => 'manager']);role_change_permission($rolemanager, $manager->get_context(), 'mod/h5pactivity:reviewattempts', CAP_PROHIBIT);$roleeditingteacher = $DB->get_field('role', 'id', ['shortname' => 'editingteacher']);role_change_permission($roleeditingteacher, $manager->get_context(), 'mod/h5pactivity:reviewattempts', CAP_PROHIBIT);// Generate users join SQL to find matching users.$usersjoin = $manager->get_active_users_join(true);$usernames = $DB->get_fieldset_sql("SELECT u.usernameFROM {user} u{$usersjoin->joins}WHERE {$usersjoin->wheres}",$usersjoin->params);$this->assertEquals([$user->username], $usernames);}/*** Test static count_attempts.*/public function test_count_users_attempts(): void {$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity',['course' => $course]);$manager = manager::create_from_instance($activity);// User without attempts.$user1 = $this->getDataGenerator()->create_and_enrol($course, 'student');// User with 1 attempt.$user2 = $this->getDataGenerator()->create_and_enrol($course, 'student');$this->generate_fake_attempts($activity, $user2, 1);// User with 2 attempts.$user3 = $this->getDataGenerator()->create_and_enrol($course, 'student');$this->generate_fake_attempts($activity, $user3, 1);$attempts = $manager->count_users_attempts();$this->assertArrayNotHasKey($user1->id, $attempts);$this->assertArrayHasKey($user2->id, $attempts);$this->assertEquals(4, $attempts[$user2->id]);$this->assertArrayHasKey($user3->id, $attempts);$this->assertEquals(4, $attempts[$user3->id]);}/*** Test static get_report.** @dataProvider get_report_data* @param int $enabletracking if tracking is enabled* @param int $reviewmode the attempt review mode* @param bool $createattempts if the student have attempts* @param string $role the user role (student or editingteacher)* @param array $results the expected classname (or null)*/public function test_get_report(int $enabletracking, int $reviewmode, bool $createattempts,string $role, array $results): void {$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity',['course' => $course, 'enabletracking' => $enabletracking, 'reviewmode' => $reviewmode]);$manager = manager::create_from_instance($activity);$cm = get_coursemodule_from_id('h5pactivity', $activity->cmid, 0, false, MUST_EXIST);$users = ['editingteacher' => $this->getDataGenerator()->create_and_enrol($course, 'editingteacher'),'student' => $this->getDataGenerator()->create_and_enrol($course, 'student'),'otheruser' => $this->getDataGenerator()->create_and_enrol($course, 'student'),];$attempts = [];if ($createattempts) {$this->generate_fake_attempts($activity, $users['student'], 1);$this->generate_fake_attempts($activity, $users['otheruser'], 2);$attempts['student'] = attempt::last_attempt($users['student'], $cm);$attempts['otheruser'] = attempt::last_attempt($users['otheruser'], $cm);}$classnamebase = 'mod_h5pactivity\\local\\report\\';$attemptid = null;if (isset($attempts['student'])) {$attemptid = $attempts['student']->get_id() ?? null;}$userid = $users['student']->id;// Check reports.$this->setUser($users[$role]);$report = $manager->get_report(null, null);if ($results[0] === null) {$this->assertNull($report);} else {$this->assertEquals($classnamebase.$results[0], get_class($report));}$report = $manager->get_report($userid, null);if ($results[1] === null) {$this->assertNull($report);} else {$this->assertEquals($classnamebase.$results[1], get_class($report));}$report = $manager->get_report($userid, $attemptid);if ($results[2] === null) {$this->assertNull($report);} else {$this->assertEquals($classnamebase.$results[2], get_class($report));}// Check that student cannot access another student reports.if ($role == 'student') {$attemptid = null;if (isset($attempts['otheruser'])) {$attemptid = $attempts['otheruser']->get_id() ?? null;}$userid = $users['otheruser']->id;$report = $manager->get_report($userid, null);$this->assertNull($report);$report = $manager->get_report($userid, $attemptid);$this->assertNull($report);}}/*** Data provider for test_get_report.** @return array*/public static function get_report_data(): array {return [// No tracking scenarios.'No tracking, review none, no attempts, teacher' => [0, manager::REVIEWNONE, false, 'editingteacher', [null, null, null]],'No tracking, review own, no attempts, teacher' => [0, manager::REVIEWCOMPLETION, false, 'editingteacher', [null, null, null]],'No tracking, review none, no attempts, student' => [0, manager::REVIEWNONE, false, 'student', [null, null, null]],'No tracking, review own, no attempts, student' => [0, manager::REVIEWCOMPLETION, false, 'student', [null, null, null]],'No tracking, review none, with attempts, teacher' => [0, manager::REVIEWNONE, true, 'editingteacher', [null, null, null]],'No tracking, review own, with attempts, teacher' => [0, manager::REVIEWCOMPLETION, true, 'editingteacher', [null, null, null]],'No tracking, review none, with attempts, student' => [0, manager::REVIEWNONE, true, 'student', [null, null, null]],'No tracking, review own, with attempts, student' => [0, manager::REVIEWCOMPLETION, true, 'student', [null, null, null]],// Tracking enabled scenarios.'Tracking enabled, review none, no attempts, teacher' => [1, manager::REVIEWNONE, false, 'editingteacher', ['participants', 'attempts', 'attempts']],'Tracking enabled, review own, no attempts, teacher' => [1, manager::REVIEWCOMPLETION, false, 'editingteacher', ['participants', 'attempts', 'attempts']],'Tracking enabled, review none, no attempts, student' => [1, manager::REVIEWNONE, false, 'student', [null, null, null]],'Tracking enabled, review own, no attempts, student' => [1, manager::REVIEWCOMPLETION, false, 'student', ['attempts', 'attempts', 'attempts']],'Tracking enabled, review none, with attempts, teacher' => [1, manager::REVIEWNONE, true, 'editingteacher', ['participants', 'attempts', 'results']],'Tracking enabled, review own, with attempts, teacher' => [1, manager::REVIEWCOMPLETION, true, 'editingteacher', ['participants', 'attempts', 'results']],'Tracking enabled, review none, with attempts, student' => [1, manager::REVIEWNONE, true, 'student', [null, null, null]],'Tracking enabled, review own, with attempts, student' => [1, manager::REVIEWCOMPLETION, true, 'student', ['attempts', 'attempts', 'results']],];}/*** Test teacher access to student reports (get_report) when course groupmode is SEPARATEGROUPS.* @covers ::get_report()* @dataProvider get_report_data_groupmode** @param bool $activitygroupmode Course or activity groupmode*/public function test_get_report_groupmode(bool $activitygroupmode): void {global $DB;$this->resetAfterTest();$this->setAdminUser();if ($activitygroupmode) {$course = $this->getDataGenerator()->create_course(['groupmode' => NOGROUPS, 'groupmodeforce' => 0]);$activitysettings = ['course' => $course, 'groupmode' => SEPARATEGROUPS];} else {$course = $this->getDataGenerator()->create_course(['groupmode' => SEPARATEGROUPS, 'groupmodeforce' => 1]);$activitysettings = ['course' => $course];}$activity = $this->getDataGenerator()->create_module('h5pactivity', $activitysettings);// Grant mod/h5pactivity:reviewattempts to non-editing teacher.// At the time of writing this is not set by default (see MDL-80028).$teacherrole = $DB->get_record('role', ['shortname' => 'teacher']);role_change_permission($teacherrole->id,\context_course::instance($course->id), 'mod/h5pactivity:reviewattempts', CAP_ALLOW);$manager = manager::create_from_instance($activity);$editingteacher = $this->getDataGenerator()->create_and_enrol($course, 'editingteacher');$teacher1 = $this->getDataGenerator()->create_and_enrol($course, 'teacher');$teacher2 = $this->getDataGenerator()->create_and_enrol($course, 'teacher');$student1 = $this->getDataGenerator()->create_and_enrol($course);$student2 = $this->getDataGenerator()->create_and_enrol($course);$student3 = $this->getDataGenerator()->create_and_enrol($course);$group1 = $this->getDataGenerator()->create_group(['courseid' => $course->id]);$this->getDataGenerator()->create_group_member(['groupid' => $group1->id, 'userid' => $teacher1->id]);$this->getDataGenerator()->create_group_member(['groupid' => $group1->id, 'userid' => $student1->id]);$group2 = $this->getDataGenerator()->create_group(['courseid' => $course->id]);$this->getDataGenerator()->create_group_member(['groupid' => $group2->id, 'userid' => $student2->id]);// Check reports.// Editing teachers can view all users, those in any group or no group.$this->setUser($editingteacher);$report = $manager->get_report($student1->id);$this->assertNotNull($report);$report = $manager->get_report($student3->id);$this->assertNotNull($report);// Non-editing teacher can view student, both members of same group.$this->setUser($teacher1);$report = $manager->get_report($student1->id);$this->assertNotNull($report);// Non-editing teacher cannot view student in no group.$report = $manager->get_report($student3->id);$this->assertNull($report);// Non-editing teacher cannot view student in different group.$report = $manager->get_report($student2->id);$this->assertNull($report);// Non-editing teacher in no group can view no one.$this->setUser($teacher2);$report = $manager->get_report($student1->id);$this->assertNull($report);$report = $manager->get_report($student3->id);$this->assertNull($report);}/*** Data provider for test_get_report_groupmode.** @return array*/public static function get_report_data_groupmode(): array {return [// No tracking scenarios.'course groupmode is SEPARATEGROUPS' => [false],'course groupmode is NOGROUPS, activity groupmode is SEPARATEGROUPS' => [true],];}/*** Test get_attempt method.** @dataProvider get_attempt_data* @param string $attemptname the attempt to use* @param string|null $result the expected attempt ID or null for none*/public function test_get_attempt(string $attemptname, ?string $result): void {$this->resetAfterTest();$this->setAdminUser();$course = $this->getDataGenerator()->create_course();$activity = $this->getDataGenerator()->create_module('h5pactivity', ['course' => $course]);$cm = get_coursemodule_from_id('h5pactivity', $activity->cmid, 0, false, MUST_EXIST);$otheractivity = $this->getDataGenerator()->create_module('h5pactivity', ['course' => $course]);$othercm = get_coursemodule_from_id('h5pactivity', $otheractivity->cmid, 0, false, MUST_EXIST);$manager = manager::create_from_instance($activity);$user = $this->getDataGenerator()->create_and_enrol($course, 'student');$attempts = ['inexistent' => 0];$this->generate_fake_attempts($activity, $user, 1);$attempt = attempt::last_attempt($user, $cm);$attempts['current'] = $attempt->get_id();$this->generate_fake_attempts($otheractivity, $user, 1);$attempt = attempt::last_attempt($user, $othercm);$attempts['other'] = $attempt->get_id();$attempt = $manager->get_attempt($attempts[$attemptname]);if ($result === null) {$this->assertNull($attempt);} else {$this->assertEquals($attempts[$attemptname], $attempt->get_id());$this->assertEquals($activity->id, $attempt->get_h5pactivityid());$this->assertEquals($user->id, $attempt->get_userid());$this->assertEquals(4, $attempt->get_attempt());}}/*** Data provider for test_get_attempt.** @return array*/public static function get_attempt_data(): array {return ['Get the current activity attempt' => ['current', 'current'],'Try to get another activity attempt' => ['other', null],'Try to get an inexistent attempt' => ['inexistent', null],];}/*** Insert fake attempt data into h5pactiviyt_attempts.** This function insert 4 attempts. 3 of them finished with different gradings* and timestamps and 1 unfinished.** @param stdClass $activity the activity record* @param stdClass $user user record* @param int $basescore a score to be used to generate all attempts*/private function generate_fake_attempts(stdClass $activity, stdClass $user, int $basescore) {global $DB;$attempt = (object)['h5pactivityid' => $activity->id,'userid' => $user->id,'timecreated' => $basescore,'timemodified' => ($basescore + 10),'attempt' => 1,'rawscore' => $basescore,'maxscore' => ($basescore + $basescore),'duration' => $basescore,'completion' => 1,'success' => 1,];$attempt->scaled = $attempt->rawscore / $attempt->maxscore;$DB->insert_record('h5pactivity_attempts', $attempt);$attempt = (object)['h5pactivityid' => $activity->id,'userid' => $user->id,'timecreated' => ($basescore + 20),'timemodified' => ($basescore + 30),'attempt' => 2,'rawscore' => $basescore,'maxscore' => ($basescore + $basescore - 1),'duration' => $basescore,'completion' => 1,'success' => 1,];$attempt->scaled = $attempt->rawscore / $attempt->maxscore;$DB->insert_record('h5pactivity_attempts', $attempt);$attempt = (object)['h5pactivityid' => $activity->id,'userid' => $user->id,'timecreated' => ($basescore + 40),'timemodified' => ($basescore + 50),'attempt' => 3,'rawscore' => $basescore,'maxscore' => ($basescore + $basescore + 1),'duration' => $basescore,'completion' => 1,'success' => 0,];$attempt->scaled = $attempt->rawscore / $attempt->maxscore;$DB->insert_record('h5pactivity_attempts', $attempt);// Unfinished attempt.$attempt = (object)['h5pactivityid' => $activity->id,'userid' => $user->id,'timecreated' => ($basescore + 60),'timemodified' => ($basescore + 60),'attempt' => 4,'rawscore' => $basescore,'maxscore' => $basescore,'duration' => $basescore,];$DB->insert_record('h5pactivity_attempts', $attempt);}}