Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(970)

Unified Diff: ios/chrome/browser/ui/history/history_entry_item_unittest.mm

Issue 2590473002: Upstream Chrome on iOS source code [5/11]. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ios/chrome/browser/ui/history/history_entry_item_unittest.mm
diff --git a/ios/chrome/browser/ui/history/history_entry_item_unittest.mm b/ios/chrome/browser/ui/history/history_entry_item_unittest.mm
new file mode 100644
index 0000000000000000000000000000000000000000..28f8423460f154ae6e900e929d7e800e6f4c0570
--- /dev/null
+++ b/ios/chrome/browser/ui/history/history_entry_item_unittest.mm
@@ -0,0 +1,72 @@
+// 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.
+
+#import "ios/chrome/browser/ui/history/history_entry_item.h"
+
+#include "base/i18n/time_formatting.h"
+#include "base/mac/scoped_nsobject.h"
+#include "base/strings/sys_string_conversions.h"
+#include "base/strings/utf_string_conversions.h"
+#include "base/time/time.h"
+#include "ios/chrome/browser/ui/history/history_entry.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "testing/gtest_mac.h"
+
+namespace {
+const char kTestUrl[] = "http://test/";
+const char kTestUrl2[] = "http://test2/";
+const char kTestTitle[] = "Test";
+}
+
+HistoryEntryItem* GetHistoryEntryItem(const GURL& url,
+ const char title[],
+ base::Time timestamp) {
+ history::HistoryEntry entry = history::HistoryEntry(
+ history::HistoryEntry::LOCAL_ENTRY, GURL(url), base::UTF8ToUTF16(title),
+ timestamp, "", false, base::string16(), false);
+ HistoryEntryItem* item =
+ [[[HistoryEntryItem alloc] initWithType:0
+ historyEntry:entry
+ browserState:nil
+ delegate:nil] autorelease];
+ return item;
+}
+
+// Tests that -[HistoryEntryItem configureCell:] sets the cell's textLabel text
+// to the item title, the detailTextLabel text to the URL, and the timeLabel
+// text to the timestamp.
+TEST(HistoryEntryItemTest, ConfigureCell) {
+ base::Time timestamp = base::Time::Now();
+ HistoryEntryItem* item =
+ GetHistoryEntryItem(GURL(kTestUrl), kTestTitle, timestamp);
+
+ base::scoped_nsobject<HistoryEntryCell> cell([[[item cellClass] alloc] init]);
+ EXPECT_TRUE([cell isMemberOfClass:[HistoryEntryCell class]]);
+ [item configureCell:cell];
+ EXPECT_NSEQ(base::SysUTF8ToNSString(kTestTitle), cell.get().textLabel.text);
+ EXPECT_NSEQ(base::SysUTF8ToNSString(kTestUrl),
+ cell.get().detailTextLabel.text);
+ EXPECT_NSEQ(base::SysUTF16ToNSString(base::TimeFormatTimeOfDay(timestamp)),
+ cell.get().timeLabel.text);
+}
+
+// Tests that -[HistoryItem isEqualToHistoryItem:] returns YES if the two items
+// have the same URL and timestamp, and NO otherwise.
+TEST(HistoryEntryItemTest, IsEqual) {
+ base::Time timestamp = base::Time::Now();
+ base::Time timestamp2 = timestamp - base::TimeDelta::FromMinutes(1);
+ HistoryEntryItem* history_entry =
+ GetHistoryEntryItem(GURL(kTestUrl), kTestTitle, timestamp);
+ HistoryEntryItem* same_entry =
+ GetHistoryEntryItem(GURL(kTestUrl), kTestTitle, timestamp);
+
+ HistoryEntryItem* different_time_entry =
+ GetHistoryEntryItem(GURL(kTestUrl), kTestTitle, timestamp2);
+ HistoryEntryItem* different_url_entry =
+ GetHistoryEntryItem(GURL(kTestUrl2), kTestTitle, timestamp);
+
+ EXPECT_TRUE([history_entry isEqualToHistoryEntryItem:same_entry]);
+ EXPECT_FALSE([history_entry isEqualToHistoryEntryItem:different_time_entry]);
+ EXPECT_FALSE([history_entry isEqualToHistoryEntryItem:different_url_entry]);
+}
« no previous file with comments | « ios/chrome/browser/ui/history/history_entry_item.mm ('k') | ios/chrome/browser/ui/history/history_panel_view_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698