Matcher includes is not working in expectJsonSnapshot

Discussed in https://github.com/pactumjs/pactum/discussions/98

<div type='discussions-op-text'>

^{Originally posted by captaincc423 December 1, 2021} I’m trying to get some matchers to work in expectJsonSnapshot and I’m running into a few problems:

1. I’m trying to use a variable variable with a string in an “includes()”, e.g. "id": includes(response.body.id) and it’s not working. It always says it doesn’t match.

Am I misusing these matchers?</div>

[olcortesb]

Yes, It’s true, is necessary evaluate the impact.

Another solution could be passing the functions as string in to include(), and use eval() o new Function()() for evaluate in to _validateJsonSnapshot(response) , for example:

...

"id": includes('return response.body.id')
...
// In _validateJasonSnapshot(response)
// Previews validations of return in string
value = new Function(value)(response)
...
const current_rules = jmv.getMatchingRules(value, '$.body')

[olcortesb]

@olcortesb Thank you for taking time to fix this issue.

The easiest resolution would be making the strict comparison configurable while validating json snapshots.

const strict = false;
const errors = jmv.validate(actual, expected, all_rules, '$.body', strict);

User needs to be cautious while disabling strict comparison. May be we can pass the true/false as the second or third argument.

expectJsonSnapshot(matchers, strict?);
expectJsonSnapshot(name, matchers, strict?);

OK, Interesting, I take your recommendation with strict comparison configurable and prepare a Pull Request! thanks