Fixed `delete` usage in authorization_test.cpp. Additionally this review fixes some minor formatting and typo issues.
Review: https://reviews.apache.org/r/49575/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/2c858800 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/2c858800 Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/2c858800 Branch: refs/heads/master Commit: 2c85880025d925788621f8755e109391e63d1523 Parents: 3e78054 Author: Joerg Schad <jo...@mesosphere.io> Authored: Tue Jul 5 00:28:25 2016 -0500 Committer: Vinod Kone <vinodk...@gmail.com> Committed: Tue Jul 5 00:28:25 2016 -0500 ---------------------------------------------------------------------- src/tests/authorization_tests.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/2c858800/src/tests/authorization_tests.cpp ---------------------------------------------------------------------- diff --git a/src/tests/authorization_tests.cpp b/src/tests/authorization_tests.cpp index ec14984..38885ca 100644 --- a/src/tests/authorization_tests.cpp +++ b/src/tests/authorization_tests.cpp @@ -2322,14 +2322,14 @@ TYPED_TEST(AuthorizationTest, ViewFlags) AWAIT_EXPECT_TRUE(authorizer.get()->authorized(request)); } - { + { authorization::Request request; request.set_action(authorization::VIEW_FLAGS); request.mutable_subject()->set_value("bar"); AWAIT_EXPECT_FALSE(authorizer.get()->authorized(request)); } - // Test that no authorzer is created with invalid flags. + // Test that no authorizer is created with invalid flags. { ACLs invalid; @@ -2384,7 +2384,7 @@ TYPED_TEST(AuthorizationTest, ValidateEndpoints) // Create an `Authorizer` with the ACLs. Try<Authorizer*> create = TypeParam::create(parameterize(acls)); - EXPECT_SOME(create); + ASSERT_SOME(create); delete create.get(); } }
