From 757178a551c2dcc24730dccb6ed1cd11005b3b8f Mon Sep 17 00:00:00 2001 From: Ed Tanous Date: Wed, 3 Apr 2024 14:32:38 -0700 Subject: Refactor tftp parser This function in the next patch will be used for more than just TFTP, so rename it to match intent, and refactor to use non-TFTP specific types. Tested: Rename only. Need help on TFTP setups if we need it. Change-Id: Ifc7485aa60ec53407c38b3d1bec530bdacf50075 Signed-off-by: Ed Tanous --- test/redfish-core/lib/update_service_test.cpp | 39 ++++++++++++++++----------- 1 file changed, 23 insertions(+), 16 deletions(-) (limited to 'test') diff --git a/test/redfish-core/lib/update_service_test.cpp b/test/redfish-core/lib/update_service_test.cpp index d56d709f5f..540313154f 100644 --- a/test/redfish-core/lib/update_service_test.cpp +++ b/test/redfish-core/lib/update_service_test.cpp @@ -16,38 +16,42 @@ TEST(UpdateService, ParseTFTPPostitive) crow::Response res; { // No protocol, schema on url - std::optional ret = parseTftpUrl("tftp://1.1.1.1/path", - std::nullopt, res); + std::optional ret = + parseSimpleUpdateUrl("tftp://1.1.1.1/path", std::nullopt, res); ASSERT_TRUE(ret); if (!ret) { return; } - EXPECT_EQ(ret->tftpServer, "1.1.1.1"); - EXPECT_EQ(ret->fwFile, "path"); + EXPECT_EQ(ret->encoded_host_and_port(), "1.1.1.1"); + EXPECT_EQ(ret->encoded_path(), "/path"); + EXPECT_EQ(ret->scheme(), "tftp"); } { // Protocol, no schema on url - std::optional ret = parseTftpUrl("1.1.1.1/path", "TFTP", res); + std::optional ret = + parseSimpleUpdateUrl("1.1.1.1/path", "TFTP", res); ASSERT_TRUE(ret); if (!ret) { return; } - EXPECT_EQ(ret->tftpServer, "1.1.1.1"); - EXPECT_EQ(ret->fwFile, "path"); + EXPECT_EQ(ret->encoded_host_and_port(), "1.1.1.1"); + EXPECT_EQ(ret->encoded_path(), "/path"); + EXPECT_EQ(ret->scheme(), "tftp"); } { // Both protocl and schema on url - std::optional ret = parseTftpUrl("tftp://1.1.1.1/path", "TFTP", - res); + std::optional ret = + parseSimpleUpdateUrl("tftp://1.1.1.1/path", "TFTP", res); ASSERT_TRUE(ret); if (!ret) { return; } - EXPECT_EQ(ret->tftpServer, "1.1.1.1"); - EXPECT_EQ(ret->fwFile, "path"); + EXPECT_EQ(ret->encoded_host_and_port(), "1.1.1.1"); + EXPECT_EQ(ret->encoded_path(), "/path"); + EXPECT_EQ(ret->scheme(), "tftp"); } } @@ -55,16 +59,19 @@ TEST(UpdateService, ParseTFTPNegative) { crow::Response res; // No protocol, no schema - ASSERT_EQ(parseTftpUrl("1.1.1.1/path", std::nullopt, res), std::nullopt); + ASSERT_EQ(parseSimpleUpdateUrl("1.1.1.1/path", std::nullopt, res), + std::nullopt); // No host - ASSERT_EQ(parseTftpUrl("/path", "TFTP", res), std::nullopt); + ASSERT_EQ(parseSimpleUpdateUrl("/path", "TFTP", res), std::nullopt); // No host - ASSERT_EQ(parseTftpUrl("path", "TFTP", res), std::nullopt); + ASSERT_EQ(parseSimpleUpdateUrl("path", "TFTP", res), std::nullopt); // No path - ASSERT_EQ(parseTftpUrl("tftp://1.1.1.1", "TFTP", res), std::nullopt); - ASSERT_EQ(parseTftpUrl("tftp://1.1.1.1/", "TFTP", res), std::nullopt); + ASSERT_EQ(parseSimpleUpdateUrl("tftp://1.1.1.1", "TFTP", res), + std::nullopt); + ASSERT_EQ(parseSimpleUpdateUrl("tftp://1.1.1.1/", "TFTP", res), + std::nullopt); } } // namespace } // namespace redfish -- cgit v1.2.3