refactor: Rename createParser to configureTestParser

- module name kebab case
- document it's usefulness and parameters

test/configure-test-parser.js

@@ -0,0 +1,38 @@
+var {DOMParser} = require('../lib/dom-parser')
+
+/**
+ * @typedef ErrorLevel {'warn' | 'error' | 'fatal'}
+ */
+/**
+ * Helper method for configuring an instance of DOMParser.
+ * Calling it without any arguments allows to assert on `errors` after using the parser.
+ * every field of the first argument is options and allows to specify test specific behavior.
+ * - `errorHandler`: The `locator` to pass to DOMParser constructor options,
+ * 									default stores a list of all entries per `key` in `errors` and does not log or throw.
+ * - `errors`: the object for the `errorHandler` to use,
+ * 						is also returned with the same name for later assertions,
+ * 						default is an empty object
+ * - `locator`: The `locator` to pass to DOMParser constructor options,
+ * 			  		 default is an empty object
+ *
+ * @param options {{
+ * 					errorHandler?: function (key: ErrorLevel, msg: string),
+ * 					errors?: Partial<Record<ErrorLevel, string[]>>,
+ * 					locator?: Object
+ *				}}
+ * @returns {{parser: DOMParser, errors: Partial<Record<ErrorLevel, string[]>>}}
+ */
+function configureTestParser ({errorHandler, errors = {}, locator = {}} = {}) {
+	errorHandler = errorHandler || ((key, msg) => {
+		if (!errors[key]) errors[key] = []
+		errors[key].push(msg)
+	})
+	return {
+		errors,
+		parser: new DOMParser({errorHandler, locator})
+	}
+}
+
+module.exports = {
+	configureTestParser
+}

test/xmltest/not-wf.test.js

@@ -1,5 +1,5 @@
 const xmltest = require('xmltest')
-const {createParser} = require('../createParser')
+const {configureTestParser} = require('../configure-test-parser')
 
 describe('xmltest/not-wellformed', () => {
 	describe('standalone', () => {
@@ -19,7 +19,7 @@ describe('xmltest/not-wellformed', () => {
 				// TODO: The DOCTYPE totally confuses xmldom :sic:
 				// for now we remove it and any newlines after it so we have reasonable tests
 				.replace(/^<!DOCTYPE doc \[[^\]]+]>[\r\n]*/m, '')
-			const {errors, parser} = createParser();
+			const {errors, parser} = configureTestParser();
 
 			// for 050.xml the result is undefined so `.toString()` needs to be careful
       const actual = parser.parseFromString(input)?.toString()

test/xmltest/valid.test.js

@@ -1,5 +1,5 @@
 const xmltest = require('xmltest')
-const {createParser} = require('../createParser')
+const {configureTestParser} = require('../configure-test-parser')
 
 
 describe('xmltest/valid', () => {
@@ -16,7 +16,7 @@ describe('xmltest/valid', () => {
 					// for now we remove it and any newlines after it so we have reasonable tests
 					.replace(/^<!DOCTYPE doc \[[^\]]+]>[\r\n]*/m, '')
 				const expected = await xmltest.getContent(xmltest.RELATED.out(pathInZip))
-        const {errors, parser} = createParser();
+        const {errors, parser} = configureTestParser();
 
         const actual = parser.parseFromString(input).toString()