JsonPathMatcher does not match json body nested objects

[timurnes]

Describe the bug

JsonPathMatcher example in official documentation does not match json body with nested fields:
Pattern: $.things[?(@.name == 'RequiredThing')]
Json: { "things": { "name": "RequiredThing" } }

First-level fields match works correctly

Expected behavior:

JsonPathMatcher correctly matches json body with nested fields

Test to reproduce

[Fact]
public void JsonPathMatcher_IsMatch_JsonStringWithNestedObjects()
{
	// Assign
	var json = "{ \"things\": { \"name\": \"RequiredThing\" } }";
	var matcher = new JsonPathMatcher("$.things[?(@.name == 'RequiredThing')]");

	// Act 
	double match = matcher.IsMatch(json);

	// Assert 
	Check.That(match).IsEqualTo(1);
}

The checked value is different from the expected one.
The checked value:
	[0]
The expected value:
	[1]

Other related info

The same pattern works correctly with another json from example:
{ "things": [ { "name": "RequiredThing" }, { "name": "Wiremock" } ] }

[StefH]

#966

[StefH]

@timurnes
You can check version
1.5.31-ci-17598

https://github.com/WireMock-Net/WireMock.Net/wiki/MyGet-preview-versions

[timurnes]

@StefH
Thanks, 1.5.31-ci-17598 works correctly. Waiting for release package version :)

Update: It doesn't. Please take a look on my next comment

[timurnes]

@StefH
The fix is working only with first-level nested objects. It seems that the root cause is deeper (looking on your commits - Newtonsoft.Json?).
This test fails:

[Fact]
public void JsonPathMatcher_IsMatch_JsonStringWithNestedObjects()
{
	// Assign
	var json = "{ \"things\": { \"nested\": { \"name\": \"RequiredThing\" } } }";
	var matcher = new JsonPathMatcher("$.things.nested[?(@.name == 'RequiredThing')]");

	// Act 
	double match = matcher.IsMatch(json);

	// Assert 
	Check.That(match).IsEqualTo(1);
}

[StefH]

@timurnes
You are correct. My fix is just a workaround for the first level.
To make it work for all levels, I need to call the code recursive. I'll check this.

BTW you know that you also can use a https://github.com/WireMock-Net/WireMock.Net/wiki/Request-Matching#dynamic-linq-linqmatcher

Which is much easier to use:

"things.nested.name == \"RequiredThing\""

[timurnes]

@StefH
Sure, I used JsonPartialMatcher when I discovered that JsonPathMatcher does not work correctly - it looks better for my case

[StefH]

@timurnes
Making the fix recursive is not that easy, so for now I'll merge this PR as-is.