Refactor multiprocess test code to allow multiple child processes to be launched.

test/win/win_multiprocess_test.cc

 // Copyright 2015 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
 #include "test/win/win_multiprocess.h"
 
 #include "base/basictypes.h"
 #include "gtest/gtest.h"
 
 namespace crashpad {
 namespace test {
 namespace {
 
+template <int exit_code>

[scottmg, 2015/06/25 04:13:25]

exit_code -> ExitCode

[erikwright (departed), 2015/06/25 14:36:02]

Done.

 class TestWinMultiprocess final : public WinMultiprocess {
  public:
-  explicit TestWinMultiprocess(unsigned int exit_code)
-      : WinMultiprocess(), exit_code_(exit_code) {}
-
+  TestWinMultiprocess() {}
   ~TestWinMultiprocess() {}
 
  private:
   // WinMultiprocess will have already exercised the pipes.
-  void WinMultiprocessParent() override {}
+  void WinMultiprocessParent() override { SetExpectedChildExitCode(exit_code); }
 
   void WinMultiprocessChild() override {
-    exit(exit_code_);
+    exit(exit_code);
   }
 
-  unsigned int exit_code_;
-
   DISALLOW_COPY_AND_ASSIGN(TestWinMultiprocess);
 };
 
 enum class FailureType {
   kExpect,
   kAssert,
 };
 
+template <FailureType failure_type>

[scottmg, 2015/06/25 04:13:25]

similar

[erikwright (departed), 2015/06/25 14:36:02]

In the end I thought a non-template approach might be less obscure.

 class TestWinMultiprocessChildFails final : public WinMultiprocess {
  public:
-  explicit TestWinMultiprocessChildFails(FailureType failure_type)
-      : WinMultiprocess(), failure_type_(failure_type) {}
+  TestWinMultiprocessChildFails() {}
   ~TestWinMultiprocessChildFails() {}
 
  private:
-  void WinMultiprocessParent() override {}
+  void WinMultiprocessParent() override { SetExpectedChildExitCode(255); }
   void WinMultiprocessChild() override {
-    switch (failure_type_) {
+    switch (failure_type) {
       case FailureType::kExpect:
         EXPECT_FALSE(true);
         break;
       case FailureType::kAssert:
         ASSERT_FALSE(true);
         break;
     }
   }
 
-  FailureType failure_type_;
-
   DISALLOW_COPY_AND_ASSIGN(TestWinMultiprocessChildFails);
 };
 
 
 TEST(WinMultiprocess, WinMultiprocess) {
-  TestWinMultiprocess win_multiprocess(0);
-  win_multiprocess.Run();
+  WinMultiprocess::Run<TestWinMultiprocess<0>>();
 }
 
 TEST(WinMultiprocess, WinMultiprocessNonSuccessExitCode) {
-  TestWinMultiprocess win_multiprocess(100);
-  win_multiprocess.SetExpectedChildExitCode(100);
-  win_multiprocess.Run();
+  WinMultiprocess::Run<TestWinMultiprocess<100>>();
 }
 
 TEST(WinMultiprocessChildFails, ChildExpectFailure) {
-  TestWinMultiprocessChildFails multiprocess_failing_child(
-      FailureType::kExpect);
-  multiprocess_failing_child.SetExpectedChildExitCode(255);
-  multiprocess_failing_child.Run();
+  WinMultiprocess::Run<TestWinMultiprocessChildFails<FailureType::kExpect>>();
 }
 
 TEST(WinMultiprocessChildFails, ChildAssertFailure) {
-  TestWinMultiprocessChildFails multiprocess_failing_child(
-      FailureType::kAssert);
-  multiprocess_failing_child.SetExpectedChildExitCode(255);
-  multiprocess_failing_child.Run();
+  WinMultiprocess::Run<TestWinMultiprocessChildFails<FailureType::kAssert>>();
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace crashpad