Added zenfs-test --clean

Removed deprecated APIs from coverage

Author: james-pre

scripts/test.js

@@ -19,6 +19,7 @@ const { values: options, positionals } = parseArgs({
 		preserve: { short: 'p', type: 'boolean' },
 		'exit-on-fail': { short: 'e', type: 'boolean' },
 		report: { type: 'boolean' },
+		clean: { type: 'boolean' },
 	},
 	allowPositionals: true,
 });
@@ -42,7 +43,8 @@ Options:
 Coverage:
     --coverage <dir>    Override the default coverage data directory
     -p,--preserve       Do not delete or report coverage data
-	--report            ONLY report coverage`);
+	--report            ONLY report coverage
+    --clean             ONLY clean up coverage directory`);
 	process.exit();
 }
 
@@ -53,9 +55,14 @@ if (options.quiet && options.verbose) {
 
 process.env.NODE_V8_COVERAGE = options.coverage;
 
+if (options.clean) {
+	rmSync(options.coverage, { recursive: true, force: true });
+	process.exit();
+}
+
 if (options.report) {
 	execSync('npx c8 report --reporter=text', { stdio: 'inherit' });
-	rmSync(options.coverage, { recursive: true });
+	rmSync(options.coverage, { recursive: true, force: true });
 	process.exit();
 }
 

src/backends/store/fs.ts

@@ -46,6 +46,7 @@ export class StoreFS<T extends Store = Store> extends FileSystem {
 		};
 	}
 
+	/* node:coverage disable */
 	/**
 	 * Delete all contents stored in the file system.
 	 * @deprecated
@@ -65,6 +66,7 @@ export class StoreFS<T extends Store = Store> extends FileSystem {
 		// Root always exists.
 		this.checkRootSync();
 	}
+	/* node:coverage enable */
 
 	/**
 	 * Load an index into the StoreFS.

src/backends/store/index_fs.ts

@@ -15,6 +15,7 @@ import type { Store } from './store.js';
  * Implementors: You *must* populate the underlying store for read operations to work!
  * @deprecated
  */
+/* node:coverage disable */
 export abstract class IndexFS<T extends Store> extends StoreFS<T> {
 	protected readonly index: Index = new Index();
 
@@ -79,3 +80,4 @@ export abstract class IndexFS<T extends Store> extends StoreFS<T> {
 		super.syncSync(path, data, stats);
 	}
 }
+/* node:coverage enable */

src/devices.ts

@@ -236,6 +236,7 @@ export class DeviceFS extends StoreFS<InMemoryStore> {
 	 * Creates a new device at `path` relative to the `DeviceFS` root.
 	 * @deprecated
 	 */
+	/* node:coverage disable */
 	public createDevice<TData = any>(path: string, driver: DeviceDriver<TData>, options: object = {}): Device<TData | Record<string, never>> {
 		if (this.existsSync(path)) {
 			throw ErrnoError.With('EEXIST', path, 'mknod');
@@ -255,6 +256,7 @@ export class DeviceFS extends StoreFS<InMemoryStore> {
 		this.devices.set(path, dev);
 		return dev;
 	}
+	/* node:coverage enable */
 
 	protected devicesWithDriver(driver: DeviceDriver<unknown> | string, forceIdentity?: boolean): Device[] {
 		if (forceIdentity && typeof driver == 'string') {

src/file.ts

@@ -562,6 +562,7 @@ export class PreloadFile<FS extends FileSystem> extends File<FS> {
  * For the file systems which do not sync to anything.
  * @deprecated
  */
+/* node:coverage disable */
 export class NoSyncFile<T extends FileSystem> extends PreloadFile<T> {
 	public sync(): Promise<void> {
 		return Promise.resolve();
@@ -575,6 +576,7 @@ export class NoSyncFile<T extends FileSystem> extends PreloadFile<T> {
 
 	public closeSync(): void {}
 }
+/* node:coverage enable */
 
 /**
  * An implementation of `File` that uses the FS

src/stats.ts

@@ -113,12 +113,14 @@ export abstract class StatsCommon<T extends number | bigint> implements Node.Sta
 	 */
 	public gid: T = this._convert(0);
 
+	/* node:coverage disable */
 	/**
 	 * Some file systems stash data on stats objects.
 	 * @todo [BREAKING] Remove this
 	 * @deprecated @hidden
 	 */
 	public fileData?: unknown;
+	/* node:coverage enable */
 
 	/**
 	 * Time of last access, since epoch
@@ -270,11 +272,12 @@ export abstract class StatsCommon<T extends number | bigint> implements Node.Sta
 		return (perm & mode) === mode;
 	}
 
+	/* node:coverage disable */
 	/**
 	 * Change the mode of the file.
 	 * We use this helper function to prevent messing up the type of the file.
-	 * @internal
-	 * @deprecated This will be removed in the next minor release since it is internal
+	 * @internal @deprecated
+	 * @todo [BREAKING] Remove
 	 */
 	public chmod(mode: number): void {
 		this.mode = this._convert((this.mode & c.S_IFMT) | mode);
@@ -283,8 +286,8 @@ export abstract class StatsCommon<T extends number | bigint> implements Node.Sta
 	/**
 	 * Change the owner user/group of the file.
 	 * This function makes sure it is a valid UID/GID (that is, a 32 unsigned int)
-	 * @internal
-	 * @deprecated This will be removed in the next minor release since it is internal
+	 * @internal @deprecated
+	 * @todo [BREAKING] Remove
 	 */
 	public chown(uid: number, gid: number): void {
 		uid = Number(uid);
@@ -296,6 +299,7 @@ export abstract class StatsCommon<T extends number | bigint> implements Node.Sta
 			this.gid = this._convert(gid);
 		}
 	}
+	/* node:coverage enable */
 
 	public get atimeNs(): bigint {
 		return BigInt(this.atimeMs) * n1000;

src/utils.ts

@@ -48,7 +48,9 @@ export function encodeUTF8(input: string): Uint8Array {
 	return encoder.encode(input);
 }
 
+/* node:coverage disable */
 export { /** @deprecated @hidden */ encodeUTF8 as encode };
+/* node:coverage enable */
 
 const decoder = new TextDecoder();
 
@@ -64,7 +66,9 @@ export function decodeUTF8(input?: Uint8Array): string {
 	return decoder.decode(input);
 }
 
+/* node:coverage disable */
 export { /** @deprecated @hidden */ decodeUTF8 as decode };
+/* node:coverage enable */
 
 /**
  * Decodes a directory listing

src/vfs/index.ts

@@ -7,6 +7,7 @@ export {
 	chroot,
 	mount,
 	mountObject,
+	/* node:coverage disable */
 	/**
 	 * The map of mount points.
 	 * Using `fs.mounts` instead of the `mounts` export is a security issue and not recommended.
@@ -15,6 +16,7 @@ export {
 	 * @deprecated Use the `mounts` export that isn't an `fs` property!
 	 */
 	mounts,
+	/* node:coverage enable */
 	umount,
 } from './shared.js';
 export * from './streams.js';

src/vfs/shared.ts

@@ -144,6 +144,7 @@ export function fixError<E extends ErrnoError>(e: E, paths: Record<string, strin
 	return e;
 }
 
+/* node:coverage disable */
 /**
  * @internal @deprecated
  */
@@ -155,6 +156,7 @@ export function mountObject(mounts: MountObject): void {
 		mount(point, fs);
 	}
 }
+/* node:coverage enable */
 
 /**
  * @internal @hidden