You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by an...@apache.org on 2017/04/04 17:49:46 UTC

[2/3] mesos git commit: Fixed the signatures of some test helpers.

Fixed the signatures of some test helpers.

This patch fixes some test helpers for
'CombinedAuthenticatorTest.MultipleAuthenticators' to
accept their parameters as const ref.

Review: https://reviews.apache.org/r/58174/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/c2ce0cc3
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/c2ce0cc3
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/c2ce0cc3

Branch: refs/heads/master
Commit: c2ce0cc3f2c41e0c57ba30c8be06c1faaa324a2e
Parents: 6fc22d1
Author: Greg Mann <gr...@mesosphere.io>
Authored: Tue Apr 4 10:43:04 2017 -0700
Committer: Anand Mazumdar <an...@apache.org>
Committed: Tue Apr 4 10:48:54 2017 -0700

----------------------------------------------------------------------
 src/tests/http_authentication_tests.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/c2ce0cc3/src/tests/http_authentication_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/http_authentication_tests.cpp b/src/tests/http_authentication_tests.cpp
index 36d2b73..99bc257 100644
--- a/src/tests/http_authentication_tests.cpp
+++ b/src/tests/http_authentication_tests.cpp
@@ -278,7 +278,7 @@ private:
 };
 
 
-AuthenticationResult createUnauthorized(MockAuthenticator& authenticator)
+AuthenticationResult createUnauthorized(const MockAuthenticator& authenticator)
 {
   AuthenticationResult result;
   result.unauthorized = Unauthorized(
@@ -299,7 +299,7 @@ AuthenticationResult createForbidden(MockAuthenticator& authenticator)
 
 
 AuthenticationResult createCombinedUnauthorized(
-    vector<MockAuthenticator> authenticators)
+    const vector<MockAuthenticator>& authenticators)
 {
   AuthenticationResult result;
   vector<string> headers;
@@ -321,7 +321,7 @@ AuthenticationResult createCombinedUnauthorized(
 
 
 AuthenticationResult createCombinedForbidden(
-    vector<MockAuthenticator> authenticators)
+    const vector<MockAuthenticator>& authenticators)
 {
   AuthenticationResult result;
   vector<string> bodies;