Removed all instances of '.Times(1)' from libprocess tests. Using `.Times(1)` with `EXPECT_CALL` is redundant, as this is implied by default. This patch removes all such remaining occurrences from the libprocess tests.
Review: https://reviews.apache.org/r/56188/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/da60d1c4 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/da60d1c4 Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/da60d1c4 Branch: refs/heads/master Commit: da60d1c4e8a2e9e885b1d7ef554c8411712ad1c6 Parents: ad72a33 Author: Greg Mann <[email protected]> Authored: Fri Feb 17 11:48:05 2017 -0800 Committer: Vinod Kone <[email protected]> Committed: Fri Feb 17 11:48:05 2017 -0800 ---------------------------------------------------------------------- 3rdparty/libprocess/src/tests/process_tests.cpp | 21 +++++++------------- 1 file changed, 7 insertions(+), 14 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/da60d1c4/3rdparty/libprocess/src/tests/process_tests.cpp ---------------------------------------------------------------------- diff --git a/3rdparty/libprocess/src/tests/process_tests.cpp b/3rdparty/libprocess/src/tests/process_tests.cpp index cd159b6..287eb55 100644 --- a/3rdparty/libprocess/src/tests/process_tests.cpp +++ b/3rdparty/libprocess/src/tests/process_tests.cpp @@ -132,11 +132,9 @@ TEST_TEMP_DISABLED_ON_WINDOWS(ProcessTest, Spawn) SpawnProcess process; - EXPECT_CALL(process, initialize()) - .Times(1); + EXPECT_CALL(process, initialize()); - EXPECT_CALL(process, finalize()) - .Times(1); + EXPECT_CALL(process, finalize()); PID<SpawnProcess> pid = spawn(process); @@ -167,8 +165,7 @@ TEST_TEMP_DISABLED_ON_WINDOWS(ProcessTest, Dispatch) DispatchProcess process; - EXPECT_CALL(process, func0()) - .Times(1); + EXPECT_CALL(process, func0()); EXPECT_CALL(process, func1(_)) .WillOnce(ReturnArg<0>()); @@ -204,8 +201,7 @@ TEST_TEMP_DISABLED_ON_WINDOWS(ProcessTest, Defer1) DispatchProcess process; - EXPECT_CALL(process, func0()) - .Times(1); + EXPECT_CALL(process, func0()); EXPECT_CALL(process, func1(_)) .WillOnce(ReturnArg<0>()); @@ -482,8 +478,7 @@ TEST_TEMP_DISABLED_ON_WINDOWS(ProcessTest, Inheritance) EXPECT_CALL(process, func()) .Times(2); - EXPECT_CALL(process, foo()) - .Times(1); + EXPECT_CALL(process, foo()); PID<DerivedProcess> pid1 = spawn(&process); @@ -1236,11 +1231,9 @@ TEST_TEMP_DISABLED_ON_WINDOWS(ProcessTest, Listener) MultipleListenerProcess process; - EXPECT_CALL(process, event1()) - .Times(1); + EXPECT_CALL(process, event1()); - EXPECT_CALL(process, event2()) - .Times(1); + EXPECT_CALL(process, event2()); spawn(process);
