Bugfix: assertions were being thrown if class names were changed code…

… minifiers.

README.md

@@ -17,13 +17,13 @@ A [fluent API](https://en.wikipedia.org/wiki/Fluent_interface) for enforcing
 To get started, add this dependency:
 
 ```shell
-npm install --save @cowwoc/[email protected].1
+npm install --save @cowwoc/[email protected].2
 ```
 
 or [pnpm](https://pnpm.io/):
 
 ```shell
-pnpm add @cowwoc/[email protected].1
+pnpm add @cowwoc/[email protected].2
 ```
 
 The contents of the API classes depend on which [modules](wiki/Supported_Libraries.md) are enabled.

docs/Changelog.md

@@ -1,9 +1,13 @@
 Minor updates involving cosmetic changes have been omitted from this list. See
 https://github.com/cowwoc/requirements.java/commits/master for a full list.
 
+## Version 3.3.2 - 2023/11/30
+
+* Bugfix: Assertions were being thrown if class names were changed code minifiers.
+
 ## Version 3.3.1 - 2023/11/29
 
-* Bugfix: Output ESM format to browsers instead of IIFE
+* Bugfix: Output ESM format to browsers instead of IIFE.
 
 ## Version 3.3.0 - 2023/11/29
 

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@cowwoc/requirements",
-	"version": "3.3.1",
+	"version": "3.3.2",
 	"keywords": [
 		"preconditions",
 		"postconditions",

src/Configuration.mts

@@ -33,7 +33,7 @@ class Configuration
 		typeToStringConverter = new Map<string, (input: unknown) => string>())
 	{
 		this.globalConfiguration = globalConfiguration;
-		Objects.assertThatObjectOf(context, "context", "Map");
+		Objects.assertThatInstanceOf(context, "context", Map);
 		this.context = context;
 		if (typeof (assertionsEnabled) === "undefined")
 			assertionsEnabled = globalConfiguration.assertionsAreEnabled();

src/ValidationFailure.mts

@@ -1,7 +1,7 @@
-import type {Configuration} from "./internal/internal.mjs";
 import {
 	ContextLine,
-	Objects
+	Objects,
+	Configuration
 } from "./internal/internal.mjs";
 
 /**
@@ -27,7 +27,7 @@ class ValidationFailure
 	 */
 	constructor(configuration: Configuration, exceptionType: new (message: string) => Error, message: string)
 	{
-		Objects.assertThatObjectOf(configuration, "configuration", "Configuration");
+		Objects.assertThatInstanceOf(configuration, "configuration", Configuration);
 		Objects.requireThatInstanceOf(exceptionType, "exceptionType", Function);
 		Objects.requireThatStringIsNotEmpty(message, "message");
 
@@ -111,7 +111,7 @@ class ValidationFailure
 		const existingKeys = new Set();
 		for (const entry of this.context)
 		{
-			Objects.requireThatObjectOf(entry, "entry", ContextLine.name);
+			Objects.requireThatInstanceOf(entry, "entry", ContextLine);
 			mergedContext.push(entry);
 			if (entry.key !== "")
 				existingKeys.add(entry.key);

src/internal/Objects.mts

@@ -207,42 +207,6 @@ class Objects
 		return true;
 	}
 
-	/**
-	 * Requires that a value is an object whose class has the specified name if assertions are enabled. We
-	 * assume that <code>Objects.assert()</code> will be stripped out at build-time if assertions are disabled.
-	 *
-	 * @param value - the value of a parameter
-	 * @param name - the name of the parameter
-	 * @param clazz - he expected class name of <code>value</code>
-	 * @returns <code>true</code>
-	 * @throws TypeError if <code>value</code> is not an object of type <code>clazz</code>. If
-	 * <code>name</code> is not a string.
-	 */
-	static requireThatObjectOf(value: unknown, name: string, clazz: string)
-	{
-		const nameInfo = Objects.getTypeInfo(name);
-		if (nameInfo.type !== "string")
-		{
-			throw new TypeError("name must be a string.\n" +
-				"Actual: " + Objects.toString(name) + "\n" +
-				"Type  : " + nameInfo.toString());
-		}
-		const valueInfo = Objects.getTypeInfo(value);
-		if (valueInfo.type !== "object")
-		{
-			throw new TypeError("value must be an object.\n" +
-				"Actual: " + Objects.toString(value) + "\n" +
-				"Type  : " + valueInfo.toString());
-		}
-		if (valueInfo.name !== clazz)
-		{
-			throw new TypeError(name + " must be an object of type " + clazz + ".\n" +
-				"Actual: " + Objects.toString(value) + "\n" +
-				"Type  : " + valueInfo.toString());
-		}
-		return true;
-	}
-
 	/**
 	 * Requires that an object is an instance of <code>type</code>.
 	 *
@@ -290,21 +254,6 @@ class Objects
 		Objects.assert(this.requireThatTypeOf(value, name, type));
 	}
 
-	/**
-	 * Requires that a value is an object whose class has the specified name if assertions are enabled. We
-	 * assume that <code>Objects.assert()</code> will be stripped out at build-time if assertions are disabled.
-	 *
-	 * @param value - the value of a parameter
-	 * @param name - the name of the parameter
-	 * @param type - the expected class name of <code>value</code>
-	 * @throws TypeError if <code>value</code> is not an object of type <code>type</code>. If
-	 * <code>name</code> is not a string.
-	 */
-	static assertThatObjectOf(value: unknown, name: string, type: string): void
-	{
-		Objects.assert(this.requireThatObjectOf(value, name, type));
-	}
-
 	/**
 	 * Requires that an object is an instance of the expected type.
 	 *

src/internal/Strings.mts

@@ -65,7 +65,7 @@ class Strings
 	static lastIndexOf(source: string, target: RegExp): SearchResult | null
 	{
 		Objects.assertThatTypeOf(source, "source", "string");
-		Objects.assertThatObjectOf(target, "target", "RegExp");
+		Objects.assertThatInstanceOf(target, "target", RegExp);
 
 		// RegExp is stateful: https://stackoverflow.com/a/11477448/14731
 		let flags = target.flags;

src/internal/diff/ContextGenerator.mts

@@ -1,6 +1,6 @@
 import isEqual from "lodash/isEqual.js";
-import type {Configuration} from "../internal.mjs";
 import {
+	Configuration,
 	ContextLine,
 	DiffGenerator,
 	Objects,
@@ -29,7 +29,7 @@ class ContextGenerator
 	 */
 	constructor(configuration: Configuration)
 	{
-		Objects.assertThatObjectOf(configuration, "configuration", "Configuration");
+		Objects.assertThatInstanceOf(configuration, "configuration", Configuration);
 
 		this.config = configuration;
 		this.diffGenerator = new DiffGenerator(configuration.getGlobalConfiguration().getTerminalEncoding());