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

Unified Diff: components/autofill/browser/wallet/wallet_client_unittest.cc

Issue 12457033: Implements SendAutocheckoutStatus API calls for stats tracking. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: DCHECK Created 7 years, 9 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: components/autofill/browser/wallet/wallet_client_unittest.cc
diff --git a/components/autofill/browser/wallet/wallet_client_unittest.cc b/components/autofill/browser/wallet/wallet_client_unittest.cc
index 41546af5ade7ad84141b840fa40e8e0f7bf2e13e..abdb1d49b89a60a9bae77518a48ed75cae24849b 100644
--- a/components/autofill/browser/wallet/wallet_client_unittest.cc
+++ b/components/autofill/browser/wallet/wallet_client_unittest.cc
@@ -547,7 +547,6 @@ class MockWalletClientDelegate : public WalletClientDelegate {
void(const std::string& instrument_id,
const std::string& shipping_address_id,
const std::vector<RequiredAction>& required_actions));
- MOCK_METHOD0(OnDidSendAutocheckoutStatus, void());
MOCK_METHOD2(OnDidUpdateAddress,
void(const std::string& address_id,
const std::vector<RequiredAction>& required_actions));
@@ -1372,31 +1371,21 @@ TEST_F(WalletClientTest, UpdateInstrumentMalformedResponse) {
}
TEST_F(WalletClientTest, SendAutocheckoutOfStatusSuccess) {
- EXPECT_CALL(delegate_, OnDidSendAutocheckoutStatus()).Times(1);
-
wallet_client_->SendAutocheckoutStatus(autofill::SUCCESS,
GURL(kMerchantUrl),
"google_transaction_id");
- net::TestURLFetcher* fetcher = factory_.GetFetcherByID(0);
- ASSERT_TRUE(fetcher);
- EXPECT_EQ(kSendAutocheckoutStatusOfSuccessValidRequest, GetData(fetcher));
- fetcher->SetResponseString(")]}'"); // Invalid JSON. Should be ignored.
- fetcher->set_response_code(net::HTTP_OK);
- fetcher->delegate()->OnURLFetchComplete(fetcher);
+ VerifyAndFinishRequest(net::HTTP_OK,
+ kSendAutocheckoutStatusOfSuccessValidRequest,
+ ")]}"); // Invalid JSON. Should be ignored.
}
TEST_F(WalletClientTest, SendAutocheckoutStatusOfFailure) {
- EXPECT_CALL(delegate_, OnDidSendAutocheckoutStatus()).Times(1);
-
wallet_client_->SendAutocheckoutStatus(autofill::CANNOT_PROCEED,
GURL(kMerchantUrl),
"google_transaction_id");
- net::TestURLFetcher* fetcher = factory_.GetFetcherByID(0);
- ASSERT_TRUE(fetcher);
- EXPECT_EQ(kSendAutocheckoutStatusOfFailureValidRequest, GetData(fetcher));
- fetcher->set_response_code(net::HTTP_OK);
- fetcher->SetResponseString(")]}'"); // Invalid JSON. Should be ignored.
- fetcher->delegate()->OnURLFetchComplete(fetcher);
+ VerifyAndFinishRequest(net::HTTP_OK,
+ kSendAutocheckoutStatusOfFailureValidRequest,
+ ")]}"); // Invalid JSON. Should be ignored.
}
TEST_F(WalletClientTest, HasRequestInProgress) {

Powered by Google App Engine
This is Rietveld 408576698