L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'MDL-75448-MOODLE_311' of https://github.com/golenkovm/moodle into MOODLE_311_STABLE

Browse Source
This commit is contained in:
Sara Arjona
2022-11-10 13:19:48 +01:00
parent 01011e49dd e5e8bc4b22
commit 72e9f115ab
4 changed files with 266 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/filestorage/tests/fixtures/testimage_rotated.jpg
Vendored
BIN
View File
Binary file not shown.
Side by Side

After

Width:  |  Height:  |  Size: 116 KiB

lib/filestorage/tests/stored_file_test.php
+94
View File
@@ -0,0 +1,94 @@
<?php
// This file is part of Moodle - https://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 core;
use advanced_testcase;
use context_system;
/**
* Unit tests for lib/filestorage/stored_file.php.
*
* @package core_files
* @category test
* @covers \stored_file
* @copyright 2022 Mikhail Golenkov <[email protected]>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
class stored_file_test extends advanced_testcase {
/**
* Test that the rotate_image() method does not rotate
* an image that is not supposed to be rotated.
* @covers ::rotate_image()
*/
public function test_rotate_image_does_not_rotate_image() {
global $CFG;
$this->resetAfterTest();
$filename = 'testimage.jpg';
$filepath = $CFG->dirroot . '/lib/filestorage/tests/fixtures/' . $filename;
$filerecord = [
'contextid' => context_system::instance()->id,
'component' => 'core',
'filearea' => 'unittest',
'itemid' => 0,
'filepath' => '/',
'filename' => $filename,
];
$fs = get_file_storage();
$storedfile = $fs->create_file_from_pathname($filerecord, $filepath);
$result = $storedfile->rotate_image();
$this->assertIsArray($result);
$this->assertCount(2, $result);
$this->assertFalse($result[0]);
$this->assertFalse($result[1]);
}
/**
* Test that the rotate_image() method rotates an image
* that is supposed to be rotated.
* @covers ::rotate_image()
*/
public function test_rotate_image_rotates_image() {
global $CFG;
$this->resetAfterTest();
// This image was manually rotated to be upside down. Also, Orientation, ExifImageWidth
// and ExifImageLength EXIF tags were written into its metadata.
// This is needed to make sure that this image will be rotated by stored_file::rotate_image()
// and stored as a new rotated file.
$filename = 'testimage_rotated.jpg';
$filepath = $CFG->dirroot . '/lib/filestorage/tests/fixtures/' . $filename;
$filerecord = [
'contextid' => context_system::instance()->id,
'component' => 'core',
'filearea' => 'unittest',
'itemid' => 0,
'filepath' => '/',
'filename' => $filename,
];
$fs = get_file_storage();
$storedfile = $fs->create_file_from_pathname($filerecord, $filepath);
list ($rotateddata, $size) = $storedfile->rotate_image();
$this->assertNotFalse($rotateddata);
$this->assertIsArray($size);
$this->assertEquals(1200, $size['width']);
$this->assertEquals(297, $size['height']);
}
}
mod/assign/feedback/editpdf/classes/document_services.php
+7 -4
View File
@@ -916,10 +916,13 @@ EOD;
$oldfile = $fs->get_file($record->contextid, $record->component, $record->filearea,
$record->itemid, $record->filepath, $record->filename);
$newhash = sha1($newfilepath);
// Delete old file if exists.
if ($oldfile && $newhash !== $oldfile->get_contenthash()) {
if ($oldfile) {
$newhash = \file_storage::hash_from_path($newfilepath);
if ($newhash === $oldfile->get_contenthash()) {
// Use existing file if contenthash match.
return $oldfile;
}
// Delete existing file.
$oldfile->delete();
}
mod/assign/feedback/editpdf/tests/document_services_test.php
+165
View File
@@ -0,0 +1,165 @@
<?php
// This file is part of Moodle - https://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 assignfeedback_editpdf;
use mod_assign_test_generator;
use advanced_testcase;
use ReflectionMethod;
defined('MOODLE_INTERNAL') || die();
global $CFG;
require_once($CFG->dirroot . '/mod/assign/tests/generator.php');
/**
* Unit tests for document services.
*
* @package assignfeedback_editpdf
* @category test
* @covers \assignfeedback_editpdf\document_services
* @copyright 2022 Mikhail Golenkov <[email protected]>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
class document_services_test extends advanced_testcase {
use mod_assign_test_generator;
/**
* Test that the save file method saves the file.
* @covers ::save_file()
*/
public function test_save_file_saves_the_file() {
global $DB;
$this->resetAfterTest();
$course = $this->getDataGenerator()->create_course();
$assign = $this->create_instance($course);
$user = $this->getDataGenerator()->create_user();
$this->getDataGenerator()->enrol_user($user->id, $course->id, 'student');
$method = new ReflectionMethod('\assignfeedback_editpdf\document_services', 'save_file');
$method->setAccessible(true);
$filearea = document_services::TMP_ROTATED_JPG_FILEAREA;
$content = 'some random content';
$tempfile = make_temp_directory('assignfeedback_editpdf') . DIRECTORY_SEPARATOR . 'mock.file';
file_put_contents($tempfile, $content);
// Invoke the method and confirm, that the file is saved.
$file1 = $method->invoke(null, $assign, $user->id, 1, $filearea, $tempfile);
$this->assertInstanceOf('stored_file', $file1);
$this->assertEquals(1, $DB->count_records('files', ['id' => $file1->get_id()]));
// Invoke the method again and confirm, that exising file is returned.
$file2 = $method->invoke(null, $assign, $user->id, 1, $filearea, $tempfile);
$this->assertEquals($file1->get_id(), $file2->get_id());
}
/**
* Test that save_rotated_image_file() method saves the file.
* @covers ::save_rotated_image_file()
*/
public function test_save_rotated_image_file_saves_the_file() {
global $CFG, $DB;
$this->resetAfterTest();
$course = $this->getDataGenerator()->create_course();
$assign = $this->create_instance($course);
$user = $this->getDataGenerator()->create_user();
$this->getDataGenerator()->enrol_user($user->id, $course->id, 'student');
$method = new ReflectionMethod('\assignfeedback_editpdf\document_services', 'save_rotated_image_file');
$method->setAccessible(true);
$imagecontent = file_get_contents($CFG->dirroot . '/lib/filestorage/tests/fixtures/testimage.png');
$imageresource = imagecreatefromstring($imagecontent);
// Invoke the method and confirm, that the file is saved.
$file1 = $method->invoke(null, $assign, $user->id, 1, $imageresource, 'testimage.png');
$this->assertInstanceOf('stored_file', $file1);
$this->assertEquals(1, $DB->count_records('files', ['id' => $file1->get_id()]));
// Invoke the method again and confirm, that exising file is returned.
$file2 = $method->invoke(null, $assign, $user->id, 1, $imageresource, 'testimage.png');
$this->assertEquals($file1->get_id(), $file2->get_id());
}
/**
* Test that get_combined_document_for_attempt() method rotates the image only once.
* @covers ::get_combined_document_for_attempt()
*/
public function test_get_combined_document_for_attempt_rotates_image() {
global $CFG, $DB;
$this->resetAfterTest();
$course = $this->getDataGenerator()->create_course();
$assignparams = [
'assignsubmission_file_enabled' => 1,
'assignsubmission_file_maxfiles' => 1,
'assignsubmission_file_maxsizebytes' => 1024 * 1024,
];
$assign = $this->create_instance($course, $assignparams);
$student = $this->getDataGenerator()->create_user();
$this->getDataGenerator()->enrol_user($student->id, $course->id, 'student');
$this->setUser($student);
$notices = [];
$submission = $assign->get_user_submission($student->id, true, 1);
$data = (object) ['files_filemanager' => $submission->id];
$assign->save_submission($data, $notices);
// This image was manually rotated to be upside down. Also, Orientation, ExifImageWidth
// and ExifImageLength EXIF tags were written into its metadata.
// This is needed to make sure that this image will be rotated by stored_file::rotate_image()
// and stored as a new rotated file.
$filename = 'testimage_rotated.jpg';
$filepath = $CFG->dirroot . '/lib/filestorage/tests/fixtures/' . $filename;
$filerecord = [
'contextid' => $assign->get_context()->id,
'component' => 'assignsubmission_file',
'filearea' => ASSIGNSUBMISSION_FILE_FILEAREA,
'itemid' => $submission->id,
'filepath' => '/',
'filename' => $filename,
];
$fs = get_file_storage();
$fs->create_file_from_pathname($filerecord, $filepath);
$params = [
'filearea' => document_services::TMP_ROTATED_JPG_FILEAREA,
'component' => document_services::COMPONENT,
'filename' => $filename,
];
// Combine the document and get the rotated file.
document_services::get_combined_document_for_attempt($assign, $student->id, 1);
$records = $DB->get_records('files', $params);
$this->assertCount(1, $records);
$record1 = reset($records);
// Polling file converters do this twice: one call to start a conversion and another one
// to poll the converted file. So we combine the document again here.
document_services::get_combined_document_for_attempt($assign, $student->id, 1);
$records = $DB->get_records('files', $params);
$this->assertCount(1, $records);
$record2 = reset($records);
// Confirm, that the second get_combined_document_for_attempt() call doesn't create new
// rotated file and re-uses the one that was created as part of the first
// get_combined_document_for_attempt() call.
$this->assertEquals($record1->id, $record2->id);
}
}