Merge pull request #1236 from nextcloud-libraries/fix/rename-mtime

Author: skjnldsv

__tests__/files/file.spec.ts

@@ -103,47 +103,37 @@ describe('File data change', () => {
 			source: 'https://cloud.domain.com/remote.php/dav/files/emma/Photos/picture.jpg',
 			mime: 'image/jpeg',
 			owner: 'emma',
-			mtime: new Date(Date.UTC(2023, 0, 1, 0, 0, 0)),
 		})
 
 		expect(file.basename).toBe('picture.jpg')
 		expect(file.dirname).toBe('/')
 		expect(file.root).toBe('/files/emma/Photos')
-		expect(file.mtime?.toISOString()).toBe('2023-01-01T00:00:00.000Z')
 
 		file.rename('picture-old.jpg')
 
 		expect(file.basename).toBe('picture-old.jpg')
 		expect(file.dirname).toBe('/')
 		expect(file.source).toBe('https://cloud.domain.com/remote.php/dav/files/emma/Photos/picture-old.jpg')
 		expect(file.root).toBe('/files/emma/Photos')
-
-		// Check that mtime has been updated
-		expect(file.mtime?.getDate()).toBe(new Date().getDate())
 	})
 
 	test('Moving a file', () => {
 		const file = new File({
 			source: 'https://cloud.domain.com/remote.php/dav/files/emma/Photos/picture.jpg',
 			mime: 'image/jpeg',
 			owner: 'emma',
-			mtime: new Date(Date.UTC(2023, 0, 1, 0, 0, 0)),
 		})
 
 		expect(file.basename).toBe('picture.jpg')
 		expect(file.dirname).toBe('/')
 		expect(file.root).toBe('/files/emma/Photos')
-		expect(file.mtime?.toISOString()).toBe('2023-01-01T00:00:00.000Z')
 
 		file.move('https://cloud.domain.com/remote.php/dav/files/emma/Pictures/picture-old.jpg')
 
 		expect(file.basename).toBe('picture-old.jpg')
 		expect(file.dirname).toBe('/')
 		expect(file.source).toBe('https://cloud.domain.com/remote.php/dav/files/emma/Pictures/picture-old.jpg')
 		expect(file.root).toBe('/files/emma/Pictures')
-
-		// Check that mtime has been updated
-		expect(file.mtime?.getDate()).toBe(new Date().getDate())
 	})
 
 	test('Moving a file to an invalid destination throws', () => {

__tests__/files/node.spec.ts

@@ -640,7 +640,6 @@ describe('Root and paths detection', () => {
 })
 
 describe('Move and rename of a node', () => {
-
 	test('Move updates the basename', () => {
 		const file = new File({
 			source: 'https://cloud.example.com/dav/files/images/emma.jpeg',

lib/files/node.ts

@@ -372,7 +372,6 @@ export abstract class Node {
 			// this can not be guaranteed, so to be sure users should better refetch the node
 			this.displayname = this.basename
 		}
-		this.updateMtime()
 	}
 
 	/**