5etools-utils 0.1.11 → 0.1.14

package/lib/TestJson.js

@@ -17,15 +17,11 @@ class JsonTester {
 	constructor (
 		{
 			dirSchema,
-			dir = ".",
-			dirWhitelist = null,
 			tagLog = LOG_TAG,
 			fnGetSchemaId = () => "homebrew.json",
 		},
 	) {
 		this._dirSchema = dirSchema;
-		this._dir = dir;
-		this._dirWhitelist = dirWhitelist;
 		this._tagLog = tagLog;
 		this._fnGetSchemaId = fnGetSchemaId;
 
@@ -46,6 +42,8 @@ class JsonTester {
 			},
 		);
 		// endregion
+
+		this._isSchemaLoaded = false;
 	}
 
 	/**
@@ -94,8 +92,8 @@ class JsonTester {
 		errors.push(error);
 	}
 
-	getErrors () {
-		um.info(this._tagLog, `Validating JSON against schema`);
+	_doLoadSchema () {
+		if (this._isSchemaLoaded) return;
 
 		uf.listJsonFiles(this._dirSchema)
 			.forEach(filePath => {
@@ -108,25 +106,49 @@ class JsonTester {
 				this._ajv.addSchema(contents, relativeFilePath);
 			});
 
+		this._isSchemaLoaded = true;
+	}
+
+	_doRunOnDir ({dir, errors, errorsFull, ...opts} = {}) {
+		uf.listJsonFiles(dir, opts)
+			.forEach(filePath => {
+				um.info(this._tagLog, `\tValidating "${filePath}"...`);
+
+				const data = uf.readJSON(filePath);
+				this._addImplicits(data);
+				const valid = this._ajv.validate(this._fnGetSchemaId(filePath), data);
+
+				if (!valid) this._handleError({filePath, errors, errorsFull, data});
+			});
+	}
+
+	/**
+	 * @param dir
+	 * @param [opts]
+	 * @param [opts.dirBlacklist]
+	 */
+	getErrors (dir, opts = {}) {
+		um.info(this._tagLog, `Validating JSON against schema`);
+
+		this._doLoadSchema();
+
 		const errors = [];
 		const errorsFull = [];
 
-		uf.runOnDirs(
-			(dir) => {
-				uf.listJsonFiles(dir)
-					.forEach(filePath => {
-						const data = uf.readJSON(filePath);
-						this._addImplicits(data);
-						const valid = this._ajv.validate(this._fnGetSchemaId(filePath), data);
+		this._doRunOnDir({...opts, errors, errorsFull, dir});
 
-						if (!valid) this._handleError({filePath, errors, errorsFull, data});
-					});
-			},
-			{
-				root: this._dir,
-				dirWhitelist: this._dirWhitelist,
-			},
-		);
+		return {errors, errorsFull};
+	}
+
+	getErrorsOnDirs () {
+		um.info(this._tagLog, `Validating JSON against schema`);
+
+		this._doLoadSchema();
+
+		const errors = [];
+		const errorsFull = [];
+
+		uf.runOnDirs((dir) => this._doRunOnDir({errors, errorsFull, dir}));
 
 		return {errors, errorsFull};
 	}

package/lib/UtilFs.js

@@ -28,23 +28,30 @@ function readJSON (path, {isIncludeRaw = false} = {}) {
 	}
 }
 
-function listJsonFiles (dir) {
+/**
+ * @param dir
+ * @param [opts]
+ * @param [opts.dirBlacklist]
+ */
+function listJsonFiles (dir, opts) {
+	const {dirBlacklist = null} = opts || {};
+
 	const dirContent = fs.readdirSync(dir, "utf8")
 		.map(file => dir === "." ? file : `${dir}/${file}`);
 	return dirContent.reduce((acc, file) => {
-		if (isDirectory(file)) acc.push(...listJsonFiles(file));
-		else {
+		if (isDirectory(file)) {
+			if (dirBlacklist == null || !dirBlacklist.has(file)) acc.push(...listJsonFiles(file, opts));
+		} else {
 			if (file.toLowerCase().endsWith(".json")) acc.push(file);
 		}
 		return acc;
 	}, []);
 }
 
-function runOnDirs (fn, {root = ".", dirWhitelist = null} = {}) {
-	fs.readdirSync(root, "utf8")
-		.map(dir => ({dir, dirPath: root === "." ? dir : `${root}/${dir}`}))
-		.filter(({dir, dirPath}) => isDirectory(dirPath) && !dir.startsWith(".") && !dir.startsWith("_") && dir !== "node_modules" && (!dirWhitelist || dirWhitelist.has(dir)))
-		.forEach(({dirPath}) => fn(dirPath));
+function runOnDirs (fn) {
+	fs.readdirSync(".", "utf8")
+		.filter(dir => isDirectory(dir) && !dir.startsWith(".") && !dir.startsWith("_") && dir !== "node_modules")
+		.forEach(dir => fn(dir));
 }
 
 function mkDirs (pathToCreate) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "5etools-utils",
-	"version": "0.1.11",
+	"version": "0.1.14",
 	"description": "Shared utilities for the 5etools ecosystem.",
 	"type": "module",
 	"main": "lib/Api.js",