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

Unified Diff: chrome/browser/ui/app_list/search/app_search_provider_unittest.cc

Issue 379333005: Allow AppSearchProvider to provide recommendations. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address comments Created 6 years, 5 months 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: chrome/browser/ui/app_list/search/app_search_provider_unittest.cc
diff --git a/chrome/browser/ui/app_list/search/app_search_provider_unittest.cc b/chrome/browser/ui/app_list/search/app_search_provider_unittest.cc
index 451eb5b7d669516bc4ae1676e49b6443dd25c58b..3d405dc3b8c35563f23239d94897f3c6cc92eec5 100644
--- a/chrome/browser/ui/app_list/search/app_search_provider_unittest.cc
+++ b/chrome/browser/ui/app_list/search/app_search_provider_unittest.cc
@@ -14,12 +14,15 @@
#include "chrome/browser/ui/app_list/search/chrome_search_result.h"
#include "chrome/common/chrome_constants.h"
#include "chrome/test/base/testing_profile.h"
+#include "extensions/browser/extension_prefs.h"
#include "extensions/common/extension_set.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace app_list {
namespace test {
+const base::Time kTestCurrentTime = base::Time::FromInternalValue(1000);
+
class AppSearchProviderTest : public AppListTestBase {
public:
AppSearchProviderTest() {}
@@ -33,8 +36,7 @@ class AppSearchProviderTest : public AppListTestBase {
}
std::string RunQuery(const std::string& query) {
- app_search_->Start(base::UTF8ToUTF16(query));
- app_search_->Stop();
+ app_search_->StartImpl(kTestCurrentTime, base::UTF8ToUTF16(query));
std::string result_str;
const SearchProvider::Results& results = app_search_->results();
@@ -47,14 +49,12 @@ class AppSearchProviderTest : public AppListTestBase {
return result_str;
}
- private:
scoped_ptr<AppSearchProvider> app_search_;
DISALLOW_COPY_AND_ASSIGN(AppSearchProviderTest);
};
TEST_F(AppSearchProviderTest, Basic) {
- EXPECT_EQ("", RunQuery(""));
EXPECT_EQ("", RunQuery("!@#$-,-_"));
EXPECT_EQ("", RunQuery("unmatched query"));
@@ -92,5 +92,20 @@ TEST_F(AppSearchProviderTest, Uninstall) {
base::RunLoop().RunUntilIdle();
}
+TEST_F(AppSearchProviderTest, FetchRecommendations) {
+ extensions::ExtensionPrefs* prefs =
+ extensions::ExtensionPrefs::Get(profile_.get());
+
+ prefs->SetLastLaunchTime(kHostedAppId, base::Time::FromInternalValue(20));
+ prefs->SetLastLaunchTime(kPackagedApp1Id, base::Time::FromInternalValue(10));
+ prefs->SetLastLaunchTime(kPackagedApp2Id, base::Time::FromInternalValue(0));
+ EXPECT_EQ("Hosted App,Packaged App 1,Packaged App 2", RunQuery(""));
+
+ prefs->SetLastLaunchTime(kHostedAppId, base::Time::FromInternalValue(0));
+ prefs->SetLastLaunchTime(kPackagedApp1Id, base::Time::FromInternalValue(10));
+ prefs->SetLastLaunchTime(kPackagedApp2Id, base::Time::FromInternalValue(20));
+ EXPECT_EQ("Packaged App 2,Packaged App 1,Hosted App", RunQuery(""));
+}
+
} // namespace test
} // namespace app_list

Powered by Google App Engine
This is Rietveld 408576698