diff options
author | Myung Bae <myungbae@us.ibm.com> | 2024-04-01 17:27:39 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2024-04-02 02:22:31 +0300 |
commit | 1873a04f43bb414408d1da9a5a775c05474603d2 (patch) | |
tree | 01859da41d4c385874e95fe28f3643049a034609 /test | |
parent | 4393bdb981d97e8db1d2cf7700a507222f14e08b (diff) | |
download | bmcweb-1873a04f43bb414408d1da9a5a775c05474603d2.tar.xz |
Reduce multi-level calls of req.req members
Several places access the members of `req` indirectly like
`req.req.method()`. This can be simplified as `req.method()` .
This would also make the code clearer.
Tested:
- Compiles
- Redfish service validator passes
Change-Id: Ie129564ff907cdea7ac224b1e3d80cc0dedfbd7b
Signed-off-by: Myung Bae <myungbae@us.ibm.com>
Diffstat (limited to 'test')
-rw-r--r-- | test/redfish-core/include/utils/json_utils_test.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/test/redfish-core/include/utils/json_utils_test.cpp b/test/redfish-core/include/utils/json_utils_test.cpp index 9245f25d87..5ef80eeba6 100644 --- a/test/redfish-core/include/utils/json_utils_test.cpp +++ b/test/redfish-core/include/utils/json_utils_test.cpp @@ -297,7 +297,7 @@ TEST(ReadJsonPatch, ValidElementsReturnsTrueResponseOkValuesUnpackedCorrectly) crow::Request req("{\"integer\": 1}", ec); // Ignore errors intentionally - req.req.set(boost::beast::http::field::content_type, "application/json"); + req.addHeader(boost::beast::http::field::content_type, "application/json"); int64_t integer = 0; ASSERT_TRUE(readJsonPatch(req, res, "integer", integer)); @@ -324,7 +324,7 @@ TEST(ReadJsonPatch, OdataIgnored) crow::Response res; std::error_code ec; crow::Request req(R"({"@odata.etag": "etag", "integer": 1})", ec); - req.req.set(boost::beast::http::field::content_type, "application/json"); + req.addHeader(boost::beast::http::field::content_type, "application/json"); // Ignore errors intentionally std::optional<int64_t> integer = 0; @@ -352,7 +352,7 @@ TEST(ReadJsonAction, ValidElementsReturnsTrueResponseOkValuesUnpackedCorrectly) crow::Response res; std::error_code ec; crow::Request req("{\"integer\": 1}", ec); - req.req.set(boost::beast::http::field::content_type, "application/json"); + req.addHeader(boost::beast::http::field::content_type, "application/json"); // Ignore errors intentionally int64_t integer = 0; @@ -367,7 +367,7 @@ TEST(ReadJsonAction, EmptyObjectReturnsTrueResponseOk) crow::Response res; std::error_code ec; crow::Request req({"{}"}, ec); - req.req.set(boost::beast::http::field::content_type, "application/json"); + req.addHeader(boost::beast::http::field::content_type, "application/json"); // Ignore errors intentionally std::optional<int64_t> integer = 0; |