[BUG] ArgumentException: Object serialized to String. JObject instance expected. about snapper

[theramis]

Hey @viceice ,

Sorry for the late reply, you're on a role with all these bugs that you keep finding!
In this case the bug is happening because you are setting the default settings for newtonsoft and it broke in the one place I missed overriding the settings.
https://github.com/theramis/Snapper/blob/2.2.2/project/Snapper/Json/JsonSnapshotStore.cs#L39

The fix is simple, I just need to use the JObjectHelper class in the solution and then the generated snapshot looks like this

{
  "Boundary": null,
  "CharSet": null,
  "MediaType": "text/html",
  "Name": null,
  "Parameters": []
}

I'll make a PR with the change and release it soon.

Thanks again for the bug report!

[viceice]

I've started using this lib more and more on a big project. 😅

[theramis]

I've started using this lib more and more on a big project. 😅

Ahh, I see!

Anyways the change is going out in v2.2.3.

Should be up in nuget in the next 10/15 mins 😄

[viceice]

Workaround: Convert to JObject before passing to Assert.

object ToJson(object obj) {
  var res = JToken.FromObject(obj);
  return res is JObject ? res : new JObject { { "autoGenerated", res } };
}