Branch looks good, but please add some test that checks that the json is being built right, and that both unicode and json special characters are escaped properly.
For this, it might make sense to refactor into a new function the bits that create the json, including the root.toStyledString(), then add a few tests for this function, plus one test that checks that Reviews::submit_review calls it right.
Branch looks good, but please add some test that checks that the json is being built right, and that both unicode and json special characters are escaped properly.
For this, it might make sense to refactor into a new function the bits that create the json, including the root.toStyledSt ring(), then add a few tests for this function, plus one test that checks that Reviews: :submit_ review calls it right.