Adding the AccessTokenFetcher and Environment class to the app remoting test

remoting/test/mock_access_token_fetcher.h

Index: remoting/test/mock_access_token_fetcher.h
diff --git a/remoting/test/mock_access_token_fetcher.h b/remoting/test/mock_access_token_fetcher.h
new file mode 100644
index 0000000000000000000000000000000000000000..925a4967c0e6f4b06a2a82acaab6085f9be91976
--- /dev/null
+++ b/remoting/test/mock_access_token_fetcher.h
@@ -0,0 +1,47 @@
+// Copyright 2015 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 REMOTING_TEST_MOCK_ACCESS_TOKEN_FETCHER_H_
+#define REMOTING_TEST_MOCK_ACCESS_TOKEN_FETCHER_H_
+
+#include "remoting/test/access_token_fetcher.h"
+
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace remoting {
+namespace test {
+
+// Used for testing classes which rely on the AccessTokenFetcher and want to
+// simulate success and failure scenarios without using the actual class and
+// network connection.
+class MockAccessTokenFetcher : public AccessTokenFetcher {
+ public:
+  MockAccessTokenFetcher();
+  ~MockAccessTokenFetcher() override;
+
+  MOCK_METHOD2(GetAccessTokenFromAuthCode,
+               void(const std::string& auth_code,
+                    const AccessTokenCallback& callback));
+
+  MOCK_METHOD2(GetAccessTokenFromRefreshToken,
+               void(const std::string& refresh_token,
+                    const AccessTokenCallback& callback));
+
+  // Stores an access token fetcher object and wires up the mock methods to call
+  // through to the appropriate method on it.  This method is typically used to
+  // pass a fake access_token_fetcher object in but will also work for any

[Wez, 2015/02/24 03:10:19]

Why access_token_fetcher?  The class is called AccessTokenFetcher -
access_token_fetcher would be a variable name, since it is unix_style. You
probably mean "... to pass a FakeAccessTokenFetcher." (the rest of that line is
redundant since I can see from the interface, and the previous line of the
comment, that it accepts any AccessTokenFetcher.

[joedow1, 2015/02/24 05:17:41]

Done.

+  // AccessTokenFetcher instance.
+  void SetAccessTokenFetcher(scoped_ptr<AccessTokenFetcher> fetcher);
+
+ private:
+  scoped_ptr<AccessTokenFetcher> internal_access_token_fetcher_;
+
+  DISALLOW_COPY_AND_ASSIGN(MockAccessTokenFetcher);
+};
+
+}  // namespace test
+}  // namespace remoting
+
+#endif  // REMOTING_TEST_MOCK_ACCESS_TOKEN_FETCHER_H_