Awesome, we should keep this when we merge the two of our PRs as I haven't yet added tests for these types.  I have a forthcoming PR that's dependent on my JSON PR that adds a bunch of tests.  I'm hoping that we have the proper test coverage by the end of the week and we can start requiring them on PRs then.