See Release Notes
Long Term Support Release
<?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/>. /** * Unit tests for assignfeedback_comments. * * @package assignfeedback_comments * @copyright 2018 Adrian Greeve <adrian@moodle.com> * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later */ defined('MOODLE_INTERNAL') || die(); global $CFG; require_once($CFG->dirroot . '/mod/assign/locallib.php'); require_once($CFG->dirroot . '/mod/assign/tests/privacy_test.php'); /** * Unit tests for mod/assign/feedback/comments/classes/privacy/ * * @copyright 2018 Adrian Greeve <adrian@moodle.com> * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later */ class assignfeedback_comments_privacy_testcase extends \mod_assign\tests\mod_assign_privacy_testcase { /** * Convenience function for creating feedback data. * * @param object $assign assign object * @param stdClass $student user object * @param stdClass $teacher user object * @param string $submissiontext Submission text * @param string $feedbacktext Feedback text * @return array Feedback plugin object and the grade object. */ protected function create_feedback($assign, $student, $teacher, $submissiontext, $feedbacktext) { global $CFG; $submission = new \stdClass(); $submission->assignment = $assign->get_instance()->id; $submission->userid = $student->id; $submission->timecreated = time(); $submission->onlinetext_editor = ['text' => $submissiontext, 'format' => FORMAT_MOODLE]; $this->setUser($student); $notices = []; $assign->save_submission($submission, $notices); $grade = $assign->get_user_grade($student->id, true); $this->setUser($teacher); $context = context_user::instance($teacher->id); $draftitemid = file_get_unused_draft_itemid(); file_prepare_draft_area($draftitemid, $context->id, ASSIGNFEEDBACK_COMMENTS_COMPONENT, ASSIGNFEEDBACK_COMMENTS_FILEAREA, $grade->id); $dummy = array( 'contextid' => $context->id, 'component' => 'user', 'filearea' => 'draft', 'itemid' => $draftitemid, 'filepath' => '/', 'filename' => 'feedback1.txt' ); $fs = get_file_storage(); $fs->create_file_from_string($dummy, $feedbacktext); $feedbacktext = $feedbacktext . " <img src='{$CFG->wwwroot}/draftfile.php/{$context->id}/user/draft/{$draftitemid}/feedback1.txt.png>"; $plugin = $assign->get_feedback_plugin_by_type('comments'); $feedbackdata = new \stdClass(); $feedbackdata->assignfeedbackcomments_editor = [ 'text' => $feedbacktext, 'format' => FORMAT_HTML, 'itemid' => $draftitemid ]; $plugin->save($grade, $feedbackdata); return [$plugin, $grade]; } /** * Quick test to make sure that get_metadata returns something. */ public function test_get_metadata() { $collection = new \core_privacy\local\metadata\collection('assignfeedback_comments'); $collection = \assignfeedback_comments\privacy\provider::get_metadata($collection); $this->assertNotEmpty($collection); } /** * Test that feedback comments are exported for a user. */ public function test_export_feedback_user_data() { $this->resetAfterTest(); // Create course, assignment, submission, and then a feedback comment. $course = $this->getDataGenerator()->create_course(); // Student. $user1 = $this->getDataGenerator()->create_user(); // Teacher. $user2 = $this->getDataGenerator()->create_user(); $this->getDataGenerator()->enrol_user($user1->id, $course->id, 'student'); $this->getDataGenerator()->enrol_user($user2->id, $course->id, 'editingteacher'); $assign = $this->create_instance(['course' => $course]); $context = $assign->get_context(); $feedbacktext = '<p>first comment for this test</p>'; list($plugin, $grade) = $this->create_feedback($assign, $user1, $user2, 'Submission text', $feedbacktext); $writer = \core_privacy\local\request\writer::with_context($context); $this->assertFalse($writer->has_any_data()); // The student should be able to see the teachers feedback. $exportdata = new \mod_assign\privacy\assign_plugin_request_data($context, $assign, $grade, [], $user1); \assignfeedback_comments\privacy\provider::export_feedback_user_data($exportdata);< $this->assertContains($feedbacktext, $writer->get_data(['Feedback comments'])->commenttext);> $this->assertStringContainsString($feedbacktext, $writer->get_data(['Feedback comments'])->commenttext);$filespath = []; $filespath[] = 'Feedback comments'; $feedbackfile = $writer->get_files($filespath)['feedback1.txt']; $this->assertInstanceOf('stored_file', $feedbackfile); $this->assertEquals('feedback1.txt', $feedbackfile->get_filename()); // The teacher should also be able to see the feedback that they provided. $exportdata = new \mod_assign\privacy\assign_plugin_request_data($context, $assign, $grade, [], $user2); \assignfeedback_comments\privacy\provider::export_feedback_user_data($exportdata);< $this->assertContains($feedbacktext, $writer->get_data(['Feedback comments'])->commenttext);> $this->assertStringContainsString($feedbacktext, $writer->get_data(['Feedback comments'])->commenttext);$feedbackfile = $writer->get_files($filespath)['feedback1.txt']; $this->assertInstanceOf('stored_file', $feedbackfile); $this->assertEquals('feedback1.txt', $feedbackfile->get_filename()); } /** * Test that all feedback is deleted for a context. */ public function test_delete_feedback_for_context() { $this->resetAfterTest(); // Create course, assignment, submission, and then a feedback comment. $course = $this->getDataGenerator()->create_course(); // Student. $user1 = $this->getDataGenerator()->create_user(); $user2 = $this->getDataGenerator()->create_user(); // Teacher. $user3 = $this->getDataGenerator()->create_user(); $this->getDataGenerator()->enrol_user($user1->id, $course->id, 'student'); $this->getDataGenerator()->enrol_user($user2->id, $course->id, 'student'); $this->getDataGenerator()->enrol_user($user3->id, $course->id, 'editingteacher'); $assign = $this->create_instance(['course' => $course]); $context = $assign->get_context(); $feedbacktext = '<p>first comment for this test</p>'; list($plugin1, $grade1) = $this->create_feedback($assign, $user1, $user3, 'Submission text', $feedbacktext); $feedbacktext = '<p>Comment for second student.</p>'; list($plugin2, $grade2) = $this->create_feedback($assign, $user2, $user3, 'Submission text', $feedbacktext); // Check that we have data. $feedbackcomments = $plugin1->get_feedback_comments($grade1->id); $this->assertNotEmpty($feedbackcomments); $feedbackcomments = $plugin1->get_feedback_comments($grade2->id); $this->assertNotEmpty($feedbackcomments); $fs = new file_storage(); $files = $fs->get_area_files($assign->get_context()->id, ASSIGNFEEDBACK_COMMENTS_COMPONENT, ASSIGNFEEDBACK_COMMENTS_FILEAREA); // 4 including directories. $this->assertEquals(4, count($files)); // Delete all comments for this context. $requestdata = new \mod_assign\privacy\assign_plugin_request_data($context, $assign); assignfeedback_comments\privacy\provider::delete_feedback_for_context($requestdata); // Check that the data is now gone. $feedbackcomments = $plugin1->get_feedback_comments($grade1->id); $this->assertEmpty($feedbackcomments); $feedbackcomments = $plugin1->get_feedback_comments($grade2->id); $this->assertEmpty($feedbackcomments); $fs = new file_storage(); $files = $fs->get_area_files($assign->get_context()->id, ASSIGNFEEDBACK_COMMENTS_COMPONENT, ASSIGNFEEDBACK_COMMENTS_FILEAREA); $this->assertEquals(0, count($files)); } /** * Test that a grade item is deleted for a user. */ public function test_delete_feedback_for_grade() { $this->resetAfterTest(); // Create course, assignment, submission, and then a feedback comment. $course = $this->getDataGenerator()->create_course(); // Student. $user1 = $this->getDataGenerator()->create_user(); $user2 = $this->getDataGenerator()->create_user(); // Teacher. $user3 = $this->getDataGenerator()->create_user(); $this->getDataGenerator()->enrol_user($user1->id, $course->id, 'student'); $this->getDataGenerator()->enrol_user($user2->id, $course->id, 'student'); $this->getDataGenerator()->enrol_user($user3->id, $course->id, 'editingteacher'); $assign = $this->create_instance(['course' => $course]); $context = $assign->get_context(); $feedbacktext = '<p>first comment for this test</p>'; list($plugin1, $grade1) = $this->create_feedback($assign, $user1, $user3, 'Submission text', $feedbacktext); $feedbacktext = '<p>Comment for second student.</p>'; list($plugin2, $grade2) = $this->create_feedback($assign, $user2, $user3, 'Submission text', $feedbacktext); // Check that we have data. $feedbackcomments = $plugin1->get_feedback_comments($grade1->id); $this->assertNotEmpty($feedbackcomments); $feedbackcomments = $plugin1->get_feedback_comments($grade2->id); $this->assertNotEmpty($feedbackcomments); $fs = new file_storage(); $files = $fs->get_area_files($assign->get_context()->id, ASSIGNFEEDBACK_COMMENTS_COMPONENT, ASSIGNFEEDBACK_COMMENTS_FILEAREA); // 4 including directories. $this->assertEquals(4, count($files)); // Delete all comments for this grade object. $requestdata = new \mod_assign\privacy\assign_plugin_request_data($context, $assign, $grade1, [], $user1); assignfeedback_comments\privacy\provider::delete_feedback_for_grade($requestdata); // These comments should be empty. $feedbackcomments = $plugin1->get_feedback_comments($grade1->id); $this->assertEmpty($feedbackcomments); // These comments should not. $feedbackcomments = $plugin1->get_feedback_comments($grade2->id); $this->assertNotEmpty($feedbackcomments); $fs = new file_storage(); $files = $fs->get_area_files($assign->get_context()->id, ASSIGNFEEDBACK_COMMENTS_COMPONENT, ASSIGNFEEDBACK_COMMENTS_FILEAREA); // 2 files that were not deleted. $this->assertEquals(2, count($files)); array_shift($files); $file = array_shift($files); $this->assertInstanceOf('stored_file', $file); $this->assertEquals('feedback1.txt', $file->get_filename()); $this->assertEquals($grade2->id, $file->get_itemid()); } /** * Test that a grade item is deleted for a user. */ public function test_delete_feedback_for_grades() { $this->resetAfterTest(); // Create course, assignment, submission, and then a feedback comment. $course = $this->getDataGenerator()->create_course(); // Student. $user1 = $this->getDataGenerator()->create_user(); $user2 = $this->getDataGenerator()->create_user(); $user3 = $this->getDataGenerator()->create_user(); $user4 = $this->getDataGenerator()->create_user(); // Teacher. $user5 = $this->getDataGenerator()->create_user(); $this->getDataGenerator()->enrol_user($user1->id, $course->id, 'student'); $this->getDataGenerator()->enrol_user($user2->id, $course->id, 'student'); $this->getDataGenerator()->enrol_user($user3->id, $course->id, 'student'); $this->getDataGenerator()->enrol_user($user4->id, $course->id, 'student'); $this->getDataGenerator()->enrol_user($user5->id, $course->id, 'editingteacher'); $assign1 = $this->create_instance(['course' => $course]); $assign2 = $this->create_instance(['course' => $course]); $feedbacktext = '<p>first comment for this test</p>'; list($plugin1, $grade1) = $this->create_feedback($assign1, $user1, $user5, 'Submission text', $feedbacktext); $feedbacktext = '<p>Comment for second student.</p>'; list($plugin2, $grade2) = $this->create_feedback($assign1, $user2, $user5, 'Submission text', $feedbacktext); $feedbacktext = '<p>Comment for third student.</p>'; list($plugin3, $grade3) = $this->create_feedback($assign1, $user3, $user5, 'Submission text', $feedbacktext); $feedbacktext = '<p>Comment for third student in the second assignment.</p>'; list($plugin4, $grade4) = $this->create_feedback($assign2, $user3, $user5, 'Submission text', $feedbacktext); $feedbacktext = '<p>Comment for fourth student in the second assignment.</p>'; list($plugin5, $grade5) = $this->create_feedback($assign2, $user4, $user5, 'Submission text', $feedbacktext); // Check that we have data. $feedbackcomments = $plugin1->get_feedback_comments($grade1->id); $this->assertNotEmpty($feedbackcomments); $feedbackcomments = $plugin2->get_feedback_comments($grade2->id); $this->assertNotEmpty($feedbackcomments); $feedbackcomments = $plugin3->get_feedback_comments($grade3->id); $this->assertNotEmpty($feedbackcomments); $feedbackcomments = $plugin4->get_feedback_comments($grade4->id); $this->assertNotEmpty($feedbackcomments); $feedbackcomments = $plugin5->get_feedback_comments($grade5->id); $this->assertNotEmpty($feedbackcomments); $fs = new file_storage(); // 6 including directories for assign 1. // 4 including directories for assign 2. $this->assertCount(6, $fs->get_area_files($assign1->get_context()->id, ASSIGNFEEDBACK_COMMENTS_COMPONENT, ASSIGNFEEDBACK_COMMENTS_FILEAREA)); $this->assertCount(4, $fs->get_area_files($assign2->get_context()->id, ASSIGNFEEDBACK_COMMENTS_COMPONENT, ASSIGNFEEDBACK_COMMENTS_FILEAREA)); $deletedata = new \mod_assign\privacy\assign_plugin_request_data($assign1->get_context(), $assign1); $deletedata->set_userids([$user1->id, $user3->id]); $deletedata->populate_submissions_and_grades(); assignfeedback_comments\privacy\provider::delete_feedback_for_grades($deletedata); // Check that grade 1 and grade 3 have been removed. $feedbackcomments = $plugin1->get_feedback_comments($grade1->id); $this->assertEmpty($feedbackcomments); $feedbackcomments = $plugin2->get_feedback_comments($grade2->id); $this->assertNotEmpty($feedbackcomments); $feedbackcomments = $plugin3->get_feedback_comments($grade3->id); $this->assertEmpty($feedbackcomments); $feedbackcomments = $plugin4->get_feedback_comments($grade4->id); $this->assertNotEmpty($feedbackcomments); $feedbackcomments = $plugin5->get_feedback_comments($grade5->id); $this->assertNotEmpty($feedbackcomments); // We have deleted two from assign 1, and none from assign 2. // 2 including directories for assign 1. // 4 including directories for assign 2. $this->assertCount(2, $fs->get_area_files($assign1->get_context()->id, ASSIGNFEEDBACK_COMMENTS_COMPONENT, ASSIGNFEEDBACK_COMMENTS_FILEAREA)); $this->assertCount(4, $fs->get_area_files($assign2->get_context()->id, ASSIGNFEEDBACK_COMMENTS_COMPONENT, ASSIGNFEEDBACK_COMMENTS_FILEAREA)); } }