Disabling wallet and mocking out more of WalletClient while testing.

components/autofill/content/browser/wallet/test_wallet_client.h

Index: components/autofill/content/browser/wallet/test_wallet_client.h
diff --git a/components/autofill/content/browser/wallet/test_wallet_client.h b/components/autofill/content/browser/wallet/test_wallet_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..381035d6f8d5d083779c6b5a7bab444390d544ed
--- /dev/null
+++ b/components/autofill/content/browser/wallet/test_wallet_client.h
@@ -0,0 +1,78 @@
+// Copyright 2013 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.
+
+#ifndef COMPONENTS_AUTOFILL_CONTENT_BROWSER_WALLET_TEST_WALLET_CLIENT_H_
+#define COMPONENTS_AUTOFILL_CONTENT_BROWSER_WALLET_TEST_WALLET_CLIENT_H_
+
+#include "components/autofill/content/browser/wallet/instrument.h"
+#include "components/autofill/content/browser/wallet/wallet_client.h"
+#include "testing/gmock/include/gmock/gmock.h"
+
+namespace autofill {
+namespace wallet {
+
+class TestWalletClient : public WalletClient {

[Ilya Sherman, 2013/07/11 01:06:30]

nit: This class should probably be named "MockWalletClient" rather than
"TestWalletClient", since it's mostly got lots of methods mocked out.

[Ilya Sherman, 2013/07/11 01:06:30]

nit: Might be nice to document this class, including mentioning that methods
that would make network requests are all mocked out.

[Dan Beam, 2013/07/11 01:29:36]

Done.

[Dan Beam, 2013/07/11 01:29:36]

Done.

+ public:
+  TestWalletClient(net::URLRequestContextGetter* context,
+                   WalletClientDelegate* delegate);
+  virtual ~TestWalletClient();
+
+  MOCK_METHOD1(GetWalletItems, void(const GURL& source_url));
+
+  MOCK_METHOD3(AcceptLegalDocuments,
+      void(const std::vector<WalletItems::LegalDocument*>& documents,
+           const std::string& google_transaction_id,
+           const GURL& source_url));
+
+  MOCK_METHOD3(AuthenticateInstrument,
+      void(const std::string& instrument_id,
+           const std::string& card_verification_number,
+           const std::string& obfuscated_gaia_id));
+
+  MOCK_METHOD1(GetFullWallet,
+      void(const WalletClient::FullWalletRequest& request));
+
+  MOCK_METHOD2(SaveAddress,
+      void(const Address& address, const GURL& source_url));
+
+  MOCK_METHOD3(SaveInstrument,
+      void(const Instrument& instrument,
+           const std::string& obfuscated_gaia_id,
+           const GURL& source_url));
+
+  MOCK_METHOD4(SaveInstrumentAndAddress,
+      void(const Instrument& instrument,
+           const Address& address,
+           const std::string& obfuscated_gaia_id,
+           const GURL& source_url));
+
+  MOCK_METHOD4(SendAutocheckoutStatus,
+      void(autofill::AutocheckoutStatus status,
+           const GURL& source_url,
+           const std::vector<AutocheckoutStatistic>& latency_statistics,
+           const std::string& google_transaction_id));
+
+  MOCK_METHOD2(UpdateAddress,
+      void(const Address& address, const GURL& source_url));
+
+  virtual void UpdateInstrument(
+      const WalletClient::UpdateInstrumentRequest& update_request,
+      scoped_ptr<Address> billing_address) {

[Ilya Sherman, 2013/07/11 01:06:30]

nit: OVERRIDE

[Dan Beam, 2013/07/11 01:29:36]

Done.

+    updated_billing_address_ = billing_address.Pass();
+  }
+
+  const Address* updated_billing_address() {
+    return updated_billing_address_.get();
+  }
+
+ private:
+  scoped_ptr<Address> updated_billing_address_;
+
+  DISALLOW_COPY_AND_ASSIGN(TestWalletClient);
+};
+
+}  // namespace wallet
+}  // namespace autofill
+
+#endif  // COMPONENTS_AUTOFILL_CONTENT_BROWSER_WALLET_TEST_WALLET_CLIENT_H_