Skip to content

Commit

Permalink
Merge pull request #1942 from Expensify/main
Browse files Browse the repository at this point in the history
Update expensify_prod branch
  • Loading branch information
rafecolton authored Nov 5, 2024
2 parents 1ef8352 + 0b3defd commit 113ab5a
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
2 changes: 2 additions & 0 deletions libstuff/SLog.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,8 @@ static const set<string> PARAMS_WHITELIST = {
"policyAccountID",
"policyID",
"reportID",
"shouldCompleteOnboarding",
"shouldDismissHybridAppOnboarding",
"token",
"transactionID",
"type",
Expand Down
9 changes: 8 additions & 1 deletion test/lib/BedrockTester.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -576,7 +576,14 @@ bool BedrockTester::readDB(const string& query, SQResult& result, bool online)
SData command("Query");
command["Query"] = fixedQuery;
command["Format"] = "JSON";
auto row0 = SParseJSONObject(executeWaitMultipleData({command})[0].content)["rows"];
auto commandResult = executeWaitMultipleData({command});
auto row0 = SParseJSONObject(commandResult[0].content)["rows"];
auto headerString = SParseJSONObject(commandResult[0].content)["headers"];

list<string> headers = SParseJSONArray(headerString);
for (const auto& h : headers) {
result.headers.push_back(h);
}

list<string> rows = SParseJSONArray(row0);
for (const string& rowStr : rows) {
Expand Down

0 comments on commit 113ab5a

Please sign in to comment.