Merge lp:~dobey/unity-scope-click/unsign-reviews into lp:unity-scope-click

[dobey]

Must not sign the call to get the existing reviews for a package.

[Charles Kerr (charlesk)]

review: Approve

=== modified file 'libclickscope/click/reviews.cpp'
--- libclickscope/click/reviews.cpp	2016-01-27 18:25:47 +0000
+++ libclickscope/click/reviews.cpp	2016-03-25 14:11:50 +0000
@@ -131,7 +131,8 @@
     click::web::CallParams params;
     params.add(click::REVIEWS_QUERY_ARGNAME, package_name.c_str());
     QSharedPointer<click::web::Response> response = client->call
-        (get_base_url() + click::REVIEWS_API_PATH, params);
+        (get_base_url() + click::REVIEWS_API_PATH, "GET", false,
+         std::map<std::string, std::string>{}, "", params);
 
     QObject::connect(response.data(), &click::web::Response::finished,
                 [=](QString reply) {

=== modified file 'libclickscope/tests/test_reviews.cpp'
--- libclickscope/tests/test_reviews.cpp	2016-02-29 21:08:09 +0000
+++ libclickscope/tests/test_reviews.cpp	2016-03-25 14:11:50 +0000
@@ -150,12 +150,12 @@
                                      click::Reviews::Error) {});
 }
 
-TEST_F(ReviewsTest, testFetchReviewsSignsCall)
+TEST_F(ReviewsTest, testFetchReviewsDoesNotSignCall)
 {
     LifetimeHelper<click::network::Reply, MockNetworkReply> reply;
     auto response = responseForReply(reply.asSharedPtr());
 
-    EXPECT_CALL(*clientPtr, callImpl(_, _, true, _, _, _))
+    EXPECT_CALL(*clientPtr, callImpl(_, _, false, _, _, _))
             .Times(1)
             .WillOnce(Return(response));