Serialize Notification.direction == "auto" in the database (2/3)

content/browser/notifications/notification_database_data_unittest.cc

 // Copyright 2015 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/strings/utf_string_conversions.h"
 #include "content/browser/notifications/notification_database_data.pb.h"
 #include "content/browser/notifications/notification_database_data_conversions.h"
 #include "content/public/browser/notification_database_data.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 16 matching lines @@
   std::vector<int> vibration_pattern(
       kNotificationVibrationPattern,
       kNotificationVibrationPattern + arraysize(kNotificationVibrationPattern));
 
   std::vector<char> developer_data(
       kNotificationData, kNotificationData + arraysize(kNotificationData));
 
   PlatformNotificationData notification_data;
   notification_data.title = base::ASCIIToUTF16(kNotificationTitle);
   notification_data.direction =
-      PlatformNotificationData::NotificationDirectionRightToLeft;
+      PlatformNotificationData::DIRECTION_RIGHT_TO_LEFT;
   notification_data.lang = kNotificationLang;
   notification_data.body = base::ASCIIToUTF16(kNotificationBody);
   notification_data.tag = kNotificationTag;
   notification_data.icon = GURL(kNotificationIconUrl);
   notification_data.vibration_pattern = vibration_pattern;
   notification_data.silent = true;
   notification_data.data = developer_data;
 
   NotificationDatabaseData database_data;
   database_data.notification_id = kNotificationId;
@@ skipping 31 matching lines @@
   EXPECT_THAT(copied_notification_data.vibration_pattern,
       testing::ElementsAreArray(kNotificationVibrationPattern));
 
   EXPECT_EQ(notification_data.silent, copied_notification_data.silent);
 
   ASSERT_EQ(developer_data.size(), copied_notification_data.data.size());
   for (size_t i = 0; i < developer_data.size(); ++i)
     EXPECT_EQ(developer_data[i], copied_notification_data.data[i]);
 }
 
+TEST(NotificationDatabaseDataTest, SerializeAndDeserializeDirections) {
+  PlatformNotificationData::Direction directions[] = {
+    PlatformNotificationData::DIRECTION_LEFT_TO_RIGHT,
+    PlatformNotificationData::DIRECTION_RIGHT_TO_LEFT,
+    PlatformNotificationData::DIRECTION_AUTO

[palmer, 2015/08/03 20:42:46]

Might be good to add a negative test case.

+  };
+
+  for (size_t i = 0; i < arraysize(directions); ++i) {
+    PlatformNotificationData notification_data;
+    notification_data.direction = directions[i];
+
+    NotificationDatabaseData database_data;
+    database_data.notification_data = notification_data;
+
+    std::string serialized_data;
+    ASSERT_TRUE(SerializeNotificationDatabaseData(database_data,
+                                                  &serialized_data));
+
+    NotificationDatabaseData copied_data;
+    ASSERT_TRUE(DeserializeNotificationDatabaseData(serialized_data,
+                                                    &copied_data));
+
+    EXPECT_EQ(directions[i], copied_data.notification_data.direction);
+  }
+}
+
 }  // namespace content