Skip to content

feat(tests): Test that mtime is the same after move or rename #43689

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
come-nc wants to merge 9 commits into
base: master
Choose a base branch
from
+222 −53
Draft

feat(tests): Test that mtime is the same after move or rename #43689

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
531ec42
feat(tests): Test that mtime is the same after move or rename
come-nc Feb 20, 2024
56918bf
fix(tests): Alter mtime to be in the past before checking it was pres…
come-nc Feb 20, 2024
d176d3c
fix: Fix mtime preservation on rename in KnownMtime wrapper
come-nc Feb 20, 2024
ad68b06
feat(tests): Add test for moveFromStorage
come-nc Feb 22, 2024
afcbdf5
fix(tests): Fix moveFromStorage test for encryption wrapper
come-nc Feb 22, 2024
a9c3a14
fix(files): Fix mtime preservation in Local and Common storages
come-nc Feb 22, 2024
f9b4cf4
fix(files): Preserve directory mtime on rename and move
come-nc Feb 22, 2024
d37436a
fix: Update children classes of Common to respect copy signature
come-nc Feb 22, 2024
c412f34
fixup! fix: Update children classes of Common to respect copy signature
come-nc Feb 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
feat(tests): Test that mtime is the same after move or rename 
Signed-off-by: Côme Chilliet <[email protected]>
  • Loading branch information
@come-nc
come-nc committed Feb 20, 2024
commit 531ec42b64a64ca309e5b40af004298a2a944ea6
10 changes: 10 additions & 0 deletions tests/lib/Files/Storage/Storage.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -243,12 +243,14 @@ public function testCopy($source, $target) {
public function testMove($source, $target) {
$this->initSourceAndTarget($source);

$mTime = $this->instance->filemtime($source);
$this->instance->rename($source, $target);

$this->wait();
$this->assertTrue($this->instance->file_exists($target), $target . ' was not created');
$this->assertFalse($this->instance->file_exists($source), $source . ' still exists');
$this->assertSameAsLorem($target);
$this->assertEquals($mTime, $this->instance->filemtime($target), 'mtime was not preserved by move');
}

/**
Expand All @@ -271,11 +273,13 @@ public function testCopyOverwrite($source, $target) {
public function testMoveOverwrite($source, $target) {
$this->initSourceAndTarget($source, $target);

$mTime = $this->instance->filemtime($source);
$this->instance->rename($source, $target);

$this->assertTrue($this->instance->file_exists($target), $target . ' was not created');
$this->assertFalse($this->instance->file_exists($source), $source . ' still exists');
$this->assertSameAsLorem($target);
$this->assertEquals($mTime, $this->instance->filemtime($target), 'mtime was not preserved by move');
}

public function testLocal() {
Expand Down Expand Up @@ -485,6 +489,10 @@ public function testRenameDirectory() {
$this->instance->file_put_contents('source/test2.txt', 'qwerty');
$this->instance->mkdir('source/subfolder');
$this->instance->file_put_contents('source/subfolder/test.txt', 'bar');

$mTimeDirectory = $this->instance->filemtime('source');
$mTimeTestFile = $this->instance->filemtime('source/subfolder/test.txt');

$this->instance->rename('source', 'target');

$this->assertFalse($this->instance->file_exists('source'));
Expand All @@ -505,6 +513,8 @@ public function testRenameDirectory() {
$this->assertEquals('foo', $this->instance->file_get_contents('target/test1.txt'));
$this->assertEquals('qwerty', $this->instance->file_get_contents('target/test2.txt'));
$this->assertEquals('bar', $this->instance->file_get_contents('target/subfolder/test.txt'));
$this->assertEquals($mTimeDirectory, $this->instance->filemtime('target'), 'directory mtime was not preserved by rename');
$this->assertEquals($mTimeTestFile, $this->instance->filemtime('target/subfolder/test.txt'), 'file mtime was not preserved by rename');
}

public function testRenameOverWriteDirectory() {
Expand Down