fix(node): allow negative file ids

__tests__/files/node.spec.ts

@@ -52,6 +52,17 @@ describe('FileId attribute', () => {
 		expect(file.fileid).toBe(1234)
 	})
 
+	// Mostly used when a node is unavailable
+	test('FileId negative fallback', () => {
+		const file = new File({
+			source: 'https://cloud.domain.com/remote.php/dav/picture.jpg',
+			mime: 'image/jpeg',
+			owner: 'emma',
+			id: -1234,
+		})
+		expect(file.fileid).toBe(-1234)
+	})
+
 	test('FileId attributes fallback', () => {
 		const file = new Folder({
 			source: 'https://cloud.domain.com/remote.php/dav/files/emma/Photos/',
@@ -73,13 +84,6 @@ describe('Sanity checks', () => {
 			owner: 'emma',
 			id: '1234' as unknown as number,
 		})).toThrowError('Invalid id type of value')
-
-		expect(() => new File({
-			source: 'https://cloud.domain.com/remote.php/dav/files/emma/Photos/picture.jpg',
-			mime: 'image/jpeg',
-			owner: 'emma',
-			id: -1234,
-		})).toThrowError('Invalid id type of value')
 	})
 
 	test('Invalid source', () => {

lib/files/nodeData.ts

@@ -23,7 +23,7 @@
 import { join } from 'path'
 import { Permission } from '../permissions'
 
 export interface Attribute { [key: string]: any }
 
 export interface NodeData {
 	/** Unique ID */
@@ -81,7 +81,7 @@
  * @param davService Pattern to check if source is DAV ressource
  */
 export const validateData = (data: NodeData, davService: RegExp) => {
-	if (data.id && (typeof data.id !== 'number' || data.id < 0)) {
+	if (data.id && typeof data.id !== 'number') {
 		throw new Error('Invalid id type of value')
 	}
 
@@ -151,7 +151,7 @@
 	}
 
 	if (data.root && isDavRessource(data.source, davService)) {
 		const service = data.source.match(davService)![0]
 		if (!data.source.includes(join(service, data.root))) {
 			throw new Error('The root must be relative to the service. e.g /files/emma')
 		}