From 876d354d3245bb3431a6ceaa350045ef3ca2d67d Mon Sep 17 00:00:00 2001 From: Mathieu Tortuyaux Date: Thu, 12 Oct 2023 12:33:01 +0200 Subject: [PATCH] test/kv-store: add test case for quoted values Useful to test when the value contains whitespaces surrounded with quotes/double-quotes. Signed-off-by: Mathieu Tortuyaux --- src/update_engine/simple_key_value_store_unittest.cc | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/update_engine/simple_key_value_store_unittest.cc b/src/update_engine/simple_key_value_store_unittest.cc index 341bfd4..345bbda 100644 --- a/src/update_engine/simple_key_value_store_unittest.cc +++ b/src/update_engine/simple_key_value_store_unittest.cc @@ -18,7 +18,7 @@ namespace chromeos_update_engine { class SimpleKeyValueStoreTest : public ::testing::Test {}; TEST(SimpleKeyValueStoreTest, SimpleTest) { - string blob = "A=B\nC=\n=\nFOO=BAR=BAZ\nBAR=BAX\nMISSING=NEWLINE"; + string blob = "A=B\nC=\n=\nFOO=BAR=BAZ\nALIAS='my alias'\nTEST=\"this is a test\"\nBAR=BAX\nMISSING=NEWLINE"; map parts = simple_key_value_store::ParseString(blob); string combined = simple_key_value_store::AssembleString(parts); map combined_parts = @@ -26,13 +26,15 @@ TEST(SimpleKeyValueStoreTest, SimpleTest) { map* maps[] = { &parts, &combined_parts }; for (size_t i = 0; i < arraysize(maps); i++) { map* test_map = maps[i]; - EXPECT_EQ(6, test_map->size()) << "i = " << i; + EXPECT_EQ(8, test_map->size()) << "i = " << i; EXPECT_EQ("B", (*test_map)["A"]) << "i = " << i; EXPECT_EQ("", (*test_map)["C"]) << "i = " << i; EXPECT_EQ("", (*test_map)[""]) << "i = " << i; EXPECT_EQ("BAR=BAZ", (*test_map)["FOO"]) << "i = " << i; EXPECT_EQ("BAX", (*test_map)["BAR"]) << "i = " << i; EXPECT_EQ("NEWLINE", (*test_map)["MISSING"]) << "i = " << i; + EXPECT_EQ("my alias", (*test_map)["ALIAS"]) << "i = " << i; + EXPECT_EQ("this is a test", (*test_map)["TEST"]) << "i = " << i; } }