[Suggestions Service] Add support for expiring the SuggestionsStore

components/suggestions/suggestions_service_unittest.cc

 // Copyright 2014 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 "components/suggestions/suggestions_service.h"
 
 #include <map>
 #include <sstream>
 #include <string>
 
@@ skipping 76 matching lines @@
 namespace suggestions {
 
 scoped_ptr<SuggestionsProfile> CreateSuggestionsProfile() {
   scoped_ptr<SuggestionsProfile> profile(new SuggestionsProfile());
   ChromeSuggestion* suggestion = profile->add_suggestions();
   suggestion->set_title(kTestTitle);
   suggestion->set_url(kTestUrl);
   return profile.Pass();
 }
 
+// create one suggestion with expiry timestamp and one without
+SuggestionsProfile CreateSuggestionsProfile_MultipleSuggestions() {

[manzagop (departed), 2014/07/31 15:31:50]

CreateSuggestionsProfileWithSomeExpiryTimestamps?

[gayane -on leave until 09-2017, 2014/08/04 13:46:30]

Done.

+  int64 now = (base::Time::NowFromSystemTime()
+              -base::Time::UnixEpoch()).ToInternalValue();

[manzagop (departed), 2014/07/31 15:31:50]

indent. See other comments for the details.

[gayane -on leave until 09-2017, 2014/08/04 13:46:30]

Done.

+
+  SuggestionsProfile profile;
+  ChromeSuggestion* suggestion = profile.add_suggestions();
+  suggestion->set_title(kTestTitle);
+  suggestion->set_url(kTestUrl);
+  suggestion->set_expiry_ts(now);
+
+  suggestion = profile.add_suggestions();
+  suggestion->set_title(kTestTitle);
+  suggestion->set_url(kTestUrl);
+
+  return profile;
+}
+
 class MockSuggestionsStore : public suggestions::SuggestionsStore {
  public:
   MOCK_METHOD1(LoadSuggestions, bool(SuggestionsProfile*));
   MOCK_METHOD1(StoreSuggestions, bool(const SuggestionsProfile&));
   MOCK_METHOD0(ClearSuggestions, void());
 };
 
 class MockImageManager : public suggestions::ImageManager {
  public:
   MockImageManager() {}
@@ skipping 383 matching lines @@
 
   // Delay increases on failure.
   suggestions_service->UpdateBlacklistDelay(false);
   EXPECT_GT(suggestions_service->blacklist_delay(), initial_delay);
 
   // Delay resets on success.
   suggestions_service->UpdateBlacklistDelay(true);
   EXPECT_EQ(initial_delay, suggestions_service->blacklist_delay());
 }
 
+TEST_F(SuggestionsServiceTest, CheckDefaultTimeStamps) {
+
+  scoped_ptr<SuggestionsService> suggestions_service(
+      CreateSuggestionsServiceWithMocks());
+
+  SuggestionsProfile suggestions =
+                      CreateSuggestionsProfile_MultipleSuggestions();
+  suggestions_service->AddDefaultExpiryTimestamps(&suggestions);
+
+  int64 now = (base::Time::NowFromSystemTime()
+              -base::Time::UnixEpoch()).ToInternalValue();
+
+  EXPECT_LT(suggestions.suggestions(0).expiry_ts(), now);

[manzagop (departed), 2014/07/31 15:31:50]

You could simplify testing by having AddDefaultExpiryTimestamps take in the
expiry timestamps as a parameter. Then you'd check you get the exact values you
expect. Of course this wouldn't cover the computation of the expiry timestamp,
but that could be done seperatly.

[gayane -on leave until 09-2017, 2014/08/04 13:46:30]

----
1. added a const private field in suggestions_service.h 
const int64 default_expiry_usec = 72 * base::Time.kMicrosecondsPerHour;
2. changed AddDefaultExpiryTimestamps(suggestions) to
SetDefaultExpiryTimestamps(suggestions, timestamp)
3.  changed unittest to check that first suggestions is not equal to passed
timestamp while the second one is equal
4. changed OnURLFetchComplete to pass now_usec + default_expiry_usec to
SetDefaultExpiryTimestamps method

+  EXPECT_GT(suggestions.suggestions(1).expiry_ts(), now);
+}
+
 }  // namespace suggestions