Allow component installers to specify a map of installer attributes.

components/update_client/utils_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/files/file_path.h"
 #include "base/path_service.h"
 #include "components/update_client/utils.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
@@ skipping 57 matching lines @@
   EXPECT_FALSE(IsValidBrand(std::string("T")));      // Too short.
   EXPECT_FALSE(IsValidBrand(std::string("TE")));     //
   EXPECT_FALSE(IsValidBrand(std::string("TES")));    //
   EXPECT_FALSE(IsValidBrand(std::string("TESTS")));  // Too long.
   EXPECT_FALSE(IsValidBrand(std::string("TES1")));   // Has digit.
   EXPECT_FALSE(IsValidBrand(std::string(" TES")));   // Begins with white space.
   EXPECT_FALSE(IsValidBrand(std::string("TES ")));   // Ends with white space.
   EXPECT_FALSE(IsValidBrand(std::string("T ES")));   // Contains white space.
   EXPECT_FALSE(IsValidBrand(std::string("<TE")));    // Has <.
   EXPECT_FALSE(IsValidBrand(std::string("TE>")));    // Has >.
-  EXPECT_FALSE(IsValidAp(std::string("\"")));        // Has "
-  EXPECT_FALSE(IsValidAp(std::string("\\")));        // Has backslash.
+  EXPECT_FALSE(IsValidBrand(std::string("\"")));     // Has "
+  EXPECT_FALSE(IsValidBrand(std::string("\\")));     // Has backslash.
   EXPECT_FALSE(IsValidBrand(std::string("\xaa")));   // Has non-ASCII char.
 }
 
-// Tests that the ap matches ^[-+_=a-zA-Z0-9]{0,256}$
-TEST(UpdateClientUtils, IsValidAp) {
-  EXPECT_TRUE(IsValidAp(std::string("a=1")));
-  EXPECT_TRUE(IsValidAp(std::string("")));
-  EXPECT_TRUE(IsValidAp(std::string("A")));
-  EXPECT_TRUE(IsValidAp(std::string("Z")));
-  EXPECT_TRUE(IsValidAp(std::string("a")));
-  EXPECT_TRUE(IsValidAp(std::string("z")));
-  EXPECT_TRUE(IsValidAp(std::string("0")));
-  EXPECT_TRUE(IsValidAp(std::string("9")));
-  EXPECT_TRUE(IsValidAp(std::string(256, 'a')));
-  EXPECT_TRUE(IsValidAp(std::string("-+_=")));
+// Tests that the name of an InstallerAttribute matches ^[-_=a-zA-Z0-9]{1,256}$
+TEST(UpdateClientUtils, IsValidInstallerAttributeName) {
+  // Test the length boundaries.
+  EXPECT_FALSE(IsValidInstallerAttribute(
+      make_pair(std::string(0, 'a'), std::string("value"))));
+  EXPECT_TRUE(IsValidInstallerAttribute(
+      make_pair(std::string(1, 'a'), std::string("value"))));
+  EXPECT_TRUE(IsValidInstallerAttribute(
+      make_pair(std::string(256, 'a'), std::string("value"))));
+  EXPECT_FALSE(IsValidInstallerAttribute(
+      make_pair(std::string(257, 'a'), std::string("value"))));
 
-  EXPECT_FALSE(IsValidAp(std::string(257, 'a')));  // Too long.
-  EXPECT_FALSE(IsValidAp(std::string(" ap")));     // Begins with white space.
-  EXPECT_FALSE(IsValidAp(std::string("ap ")));     // Ends with white space.
-  EXPECT_FALSE(IsValidAp(std::string("a p")));     // Contains white space.
-  EXPECT_FALSE(IsValidAp(std::string("<ap")));     // Has <.
-  EXPECT_FALSE(IsValidAp(std::string("ap>")));     // Has >.
-  EXPECT_FALSE(IsValidAp(std::string("\"")));      // Has "
-  EXPECT_FALSE(IsValidAp(std::string("\\")));      // Has backspace.
-  EXPECT_FALSE(IsValidAp(std::string("\xaa")));    // Has non-ASCII char.
+  const char* const valid_names[] = {"A", "Z", "a", "a-b", "A_B",
+                                     "z", "0", "9", "-_"};
+  for (const auto& name : valid_names)
+    EXPECT_TRUE(IsValidInstallerAttribute(
+        make_pair(std::string(name), std::string("value"))));
+
+  const char* const invalid_names[] = {
+      "",   "a=1", " name", "name ", "na me", "<name", "name>",
+      "\"", "\\",  "\xaa",  ".",     ",",     ";",     "+"};
+  for (const auto& name : invalid_names)
+    EXPECT_FALSE(IsValidInstallerAttribute(
+        make_pair(std::string(name), std::string("value"))));
+}
+
+// Tests that the value of an InstallerAttribute matches
+// ^[-.,;+_=a-zA-Z0-9]{0,256}$
+TEST(UpdateClientUtils, IsValidInstallerAttributeValue) {
+  // Test the length boundaries.
+  EXPECT_TRUE(IsValidInstallerAttribute(
+      make_pair(std::string("name"), std::string(0, 'a'))));
+  EXPECT_TRUE(IsValidInstallerAttribute(
+      make_pair(std::string("name"), std::string(256, 'a'))));
+  EXPECT_FALSE(IsValidInstallerAttribute(
+      make_pair(std::string("name"), std::string(257, 'a'))));
+
+  const char* const valid_values[] = {"",  "a=1", "A", "Z",      "a",
+                                      "z", "0",   "9", "-.,;+_="};
+  for (const auto& value : valid_values)
+    EXPECT_TRUE(IsValidInstallerAttribute(
+        make_pair(std::string("name"), std::string(value))));
+
+  const char* const invalid_values[] = {" ap", "ap ", "a p", "<ap",
+                                        "ap>", "\"",  "\\",  "\xaa"};
+  for (const auto& value : invalid_values)
+    EXPECT_FALSE(IsValidInstallerAttribute(
+        make_pair(std::string("name"), std::string(value))));
 }
 
 TEST(UpdateClientUtils, RemoveUnsecureUrls) {
   const GURL test1[] = {GURL("http://foo"), GURL("https://foo")};
   std::vector<GURL> urls(std::begin(test1), std::end(test1));
   RemoveUnsecureUrls(&urls);
   EXPECT_EQ(1u, urls.size());
   EXPECT_EQ(urls[0], GURL("https://foo"));
 
   const GURL test2[] = {GURL("https://foo"), GURL("http://foo")};
@@ skipping 14 matching lines @@
   RemoveUnsecureUrls(&urls);
   EXPECT_EQ(0u, urls.size());
 
   const GURL test5[] = {GURL("http://foo"), GURL("http://bar")};
   urls.assign(std::begin(test5), std::end(test5));
   RemoveUnsecureUrls(&urls);
   EXPECT_EQ(0u, urls.size());
 }
 
 }  // namespace update_client