fix: relax maskValue

.changeset/chilly-lies-decide.md

@@ -0,0 +1,11 @@
+---
+'@mocktomata/framework': patch
+---
+
+Allow `maskValue()` to be called after `spec()`.
+
+This is needed for `zucchini` step definitions.
+Instead a warning will be generated.
+
+Fixes [#530](https://github.com/mocktomata/mocktomata/issues/530)
+

packages/framework/ts/spec/masking.spec.ts

@@ -1,8 +1,6 @@
-import { a } from 'assertron'
 import { logLevels } from 'standard-log'
 import { incubator } from '../incubator/index.js'
 import { createTestAxios } from '../test_artifacts/test_subjects.js'
-import { InvokeMetaMethodAfterSpec } from './errors.js'
 
 afterAll(incubator.cleanup)
 
@@ -24,9 +22,11 @@ describe(`maskValue(string)`, () => {
 	incubator.save('throws if called after spec', { logLevel: logLevels.all }, (specName, spec, reporter) => {
 		test(specName, async () => {
 			await spec(() => {})
-			a.throws(() => spec.maskValue('secret'), InvokeMetaMethodAfterSpec)
+			spec.maskValue('secret')
+			await spec.done()
+			expect(reporter.getLogMessage()).not.toContain('secret')
+			expect(reporter.getLogMessage()).toContain('maskValue is called after spec is invoked')
 		})
-		expect(reporter.getLogMessage()).not.toContain('secret')
 	})
 
 	incubator(

packages/framework/ts/spec/spec_object.ts

@@ -36,8 +36,11 @@ export function createSpecFns(context: AsyncContext<Spec.Context>) {
 		if (s) throw new InvokeMetaMethodAfterSpec('ignoreMismatch')
 		else initState.ignoreValues.push(value)
 	}
+	let maskLog: Promise<void> | undefined
 	function maskValue(value: string | RegExp, replaceWith?: string) {
-		if (s) throw new InvokeMetaMethodAfterSpec('maskValue')
+		if (s) {
+			maskLog = context.get().then(({ log }) => log.warn('maskValue is called after spec is invoked'))
+		}
 		initState.maskCriteria.push({ value, replaceWith })
 	}
 
@@ -48,7 +51,8 @@ export function createSpecFns(context: AsyncContext<Spec.Context>) {
 		}
 	}
 	let actualSpec: Spec
-	function done() {
+	async function done() {
+		if (maskLog) await maskLog
 		if (actualSpec) return actualSpec.done()
 		// spec can be not used at all,
 		// e.g. in `scenario`.