BlockDevice constructor takes an options argument

Change-type: major

lib/scanner/adapters/block-device.ts

@@ -115,12 +115,12 @@ export class BlockDeviceAdapter extends Adapter {
 			}
 			for (const added of difference(newDevices, oldDevices)) {
 				const drive = drives.get(added);
-				const blockDevice = new BlockDevice(
-					drive!,
-					this.unmountOnSuccess,
-					this.oWrite,
-					this.oDirect,
-				);
+				const blockDevice = new BlockDevice({
+					drive: drive!,
+					unmountOnSuccess: this.unmountOnSuccess,
+					write: this.oWrite,
+					direct: this.oDirect,
+				});
 				this.emit('attach', blockDevice);
 				this.drives.set(added, blockDevice);
 			}

lib/source-destination/block-device.ts

@@ -43,16 +43,27 @@ const UNMOUNT_ON_SUCCESS_TIMEOUT_MS = 2000;
 const WIN32_FIRST_BYTES_TO_KEEP = 64 * 1024;
 
 export class BlockDevice extends File implements AdapterSourceDestination {
+	private drive: DrivelistDrive;
+	private unmountOnSuccess: boolean;
+	public readonly oDirect: boolean;
 	public emitsProgress = false;
 	public readonly alignment: number;
 
-	constructor(
-		private drive: DrivelistDrive,
-		private unmountOnSuccess = false,
-		oWrite = false,
-		public readonly oDirect = true,
-	) {
-		super({ path: drive.raw, write: oWrite });
+	constructor({
+		drive,
+		unmountOnSuccess = false,
+		write = false,
+		direct = true,
+	}: {
+		drive: DrivelistDrive;
+		unmountOnSuccess?: boolean;
+		write?: boolean;
+		direct?: boolean;
+	}) {
+		super({ path: drive.raw, write });
+		this.drive = drive;
+		this.unmountOnSuccess = unmountOnSuccess;
+		this.oDirect = direct;
 		this.alignment = drive.blockSize || DEFAULT_ALIGNMENT;
 	}
 

tests/tester.ts

@@ -67,7 +67,12 @@ export async function blockDeviceFromFile(
 		isVirtual: false,
 		logicalBlockSize: 512,
 	};
-	return new FakeBlockDevice(drive, false, true, false);
+	return new FakeBlockDevice({
+		drive,
+		unmountOnSuccess: false,
+		write: true,
+		direct: false,
+	});
 }
 
 export async function testImageNoIt(