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

Unified Diff: components/proximity_auth/cryptauth/cryptauth_api_call_flow_unittest.cc

Issue 1066453002: Refactor CryptAuth component to be more testable. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@cryptauth_securemessage
Patch Set: rename tests Created 5 years, 8 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/proximity_auth/cryptauth/cryptauth_api_call_flow_unittest.cc
diff --git a/components/proximity_auth/cryptauth/cryptauth_api_call_flow_unittest.cc b/components/proximity_auth/cryptauth/cryptauth_api_call_flow_unittest.cc
index 53417e724249443f91fb9a51f48b800769b59723..8cc2384cfb734ac56020af11968d15d5fb27df98 100644
--- a/components/proximity_auth/cryptauth/cryptauth_api_call_flow_unittest.cc
+++ b/components/proximity_auth/cryptauth/cryptauth_api_call_flow_unittest.cc
@@ -25,8 +25,7 @@ class ProximityAuthCryptAuthApiCallFlowTest
protected:
ProximityAuthCryptAuthApiCallFlowTest()
: url_request_context_getter_(new net::TestURLRequestContextGetter(
- new base::TestSimpleTaskRunner())),
- flow_(GURL(kRequestUrl)) {}
+ new base::TestSimpleTaskRunner())) {}
void SetUp() override {
// The TestURLFetcherFactory will override the global URLFetcherFactory for
@@ -36,9 +35,8 @@ class ProximityAuthCryptAuthApiCallFlowTest
}
void StartApiCallFlow() {
- flow_.Start(url_request_context_getter_.get(),
- "access_token",
- kSerializedRequestProto,
+ flow_.Start(GURL(kRequestUrl), url_request_context_getter_.get(),
+ "access_token", kSerializedRequestProto,
base::Bind(&ProximityAuthCryptAuthApiCallFlowTest::OnResult,
base::Unretained(this)),
base::Bind(&ProximityAuthCryptAuthApiCallFlowTest::OnError,
« no previous file with comments | « components/proximity_auth/cryptauth/cryptauth_api_call_flow.cc ('k') | components/proximity_auth/cryptauth/cryptauth_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698