-
Notifications
You must be signed in to change notification settings - Fork 871
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(json): support JSON.GET pretty print options #1832
feat(json): support JSON.GET pretty print options #1832
Conversation
0b39d0a
to
25f25d4
Compare
25f25d4
to
e5852d6
Compare
Signed-off-by: iko1 <me@remotecpp.dev>
c5a8076
to
1342e3d
Compare
@romange, there is a way to exclude the patch file in this commit from the clang-format validator? |
Signed-off-by: iko1 <me@remotecpp.dev>
1342e3d
to
005acf5
Compare
Hey @iko1 thanks for fixing this issue! Yes, you can check |
src/server/json_family_test.cc
Outdated
@@ -110,7 +110,9 @@ TEST_F(JsonFamilyTest, SetGetFromPhonebook) { | |||
EXPECT_EQ(resp, R"(["New York","NY","21 2nd Street","10021-3100"])"); | |||
|
|||
resp = Run({"JSON.GET", "json", "$.firstName", "$.age", "$.lastName"}); | |||
EXPECT_EQ(resp, R"({"$.age":[27],"$.firstName":["John"],"$.lastName":["Smith"]})"); | |||
EXPECT_EQ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you please add few tests covering those cases?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reverted the change that affected the "JSON.GET" command when formatting args are not supplied with it.
Can you elaborate on which tests are currently missing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added a few tests based on the previous JSON.GET cases that cover the feature functionality.
Signed-off-by: iko1 <me@remotecpp.dev>
edb065e
to
22f65da
Compare
Signed-off-by: iko1 <me@remotecpp.dev>
f1a63d0
to
64e93bb
Compare
Signed-off-by: iko1 <me@remotecpp.dev>
64e93bb
to
123cc97
Compare
Signed-off-by: iko1 <me@remotecpp.dev>
Fixes: #848