Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(242)

Side by Side Diff: base/test/launcher/test_launcher.h

Issue 441333002: Disallow breakaway from job for unit tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixes Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | base/test/launcher/test_launcher.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef BASE_TEST_LAUNCHER_TEST_LAUNCHER_H_ 5 #ifndef BASE_TEST_LAUNCHER_TEST_LAUNCHER_H_
6 #define BASE_TEST_LAUNCHER_TEST_LAUNCHER_H_ 6 #define BASE_TEST_LAUNCHER_TEST_LAUNCHER_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 10
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 class TestLauncher { 70 class TestLauncher {
71 public: 71 public:
72 // Constructor. |parallel_jobs| is the limit of simultaneous parallel test 72 // Constructor. |parallel_jobs| is the limit of simultaneous parallel test
73 // jobs. 73 // jobs.
74 TestLauncher(TestLauncherDelegate* launcher_delegate, size_t parallel_jobs); 74 TestLauncher(TestLauncherDelegate* launcher_delegate, size_t parallel_jobs);
75 ~TestLauncher(); 75 ~TestLauncher();
76 76
77 // Runs the launcher. Must be called at most once. 77 // Runs the launcher. Must be called at most once.
78 bool Run() WARN_UNUSED_RESULT; 78 bool Run() WARN_UNUSED_RESULT;
79 79
80 // Flags controlling behavior of LaunchChildGTestProcess.
sky 2014/08/07 16:19:57 nit: style guide says enums are first in a section
Paweł Hajdan Jr. 2014/08/08 10:19:09 Done.
81 enum LaunchChildGTestProcessFlags {
82 // Allows usage of job objects on Windows. Helps properly clean up child
83 // processes.
84 USE_JOB_OBJECTS = (1 << 0),
85
86 // Allows breakaway from job on Windows. May result in some child processes
87 // not being properly terminated after launcher dies if these processes
88 // fail to cooperate.
89 ALLOW_BREAKAWAY_FROM_JOB = (1 << 1),
90 };
91
80 // Callback called after a child process finishes. First argument is the exit 92 // Callback called after a child process finishes. First argument is the exit
81 // code, second one is child process elapsed time, third one is true if 93 // code, second one is child process elapsed time, third one is true if
82 // the child process was terminated because of a timeout, and fourth one 94 // the child process was terminated because of a timeout, and fourth one
83 // contains output of the child (stdout and stderr together). 95 // contains output of the child (stdout and stderr together).
84 typedef Callback<void(int, const TimeDelta&, bool, const std::string&)> 96 typedef Callback<void(int, const TimeDelta&, bool, const std::string&)>
85 LaunchChildGTestProcessCallback; 97 LaunchChildGTestProcessCallback;
86 98
87 // Launches a child process (assumed to be gtest-based binary) using 99 // Launches a child process (assumed to be gtest-based binary) using
88 // |command_line|. If |wrapper| is not empty, it is prepended to the final 100 // |command_line|. If |wrapper| is not empty, it is prepended to the final
89 // command line. If the child process is still running after |timeout|, it 101 // command line. If the child process is still running after |timeout|, it
90 // is terminated. |use_job_objects| determines whether job objects are used 102 // is terminated. After the child process finishes |callback| is called
91 // on Windows (if unsure pass true). After the child process finishes 103 // on the same thread this method was called.
92 // |callback| is called on the same thread this method was called.
93 void LaunchChildGTestProcess(const CommandLine& command_line, 104 void LaunchChildGTestProcess(const CommandLine& command_line,
94 const std::string& wrapper, 105 const std::string& wrapper,
95 base::TimeDelta timeout, 106 base::TimeDelta timeout,
96 bool use_job_objects, 107 int flags,
97 const LaunchChildGTestProcessCallback& callback); 108 const LaunchChildGTestProcessCallback& callback);
98 109
99 // Called when a test has finished running. 110 // Called when a test has finished running.
100 void OnTestFinished(const TestResult& result); 111 void OnTestFinished(const TestResult& result);
101 112
102 private: 113 private:
103 bool Init() WARN_UNUSED_RESULT; 114 bool Init() WARN_UNUSED_RESULT;
104 115
105 // Runs all tests in current iteration. Uses callbacks to communicate success. 116 // Runs all tests in current iteration. Uses callbacks to communicate success.
106 void RunTests(); 117 void RunTests();
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
187 const std::string& full_output); 198 const std::string& full_output);
188 199
189 // Returns command line command line after gtest-specific processing 200 // Returns command line command line after gtest-specific processing
190 // and applying |wrapper|. 201 // and applying |wrapper|.
191 CommandLine PrepareCommandLineForGTest(const CommandLine& command_line, 202 CommandLine PrepareCommandLineForGTest(const CommandLine& command_line,
192 const std::string& wrapper); 203 const std::string& wrapper);
193 204
194 } // namespace base 205 } // namespace base
195 206
196 #endif // BASE_TEST_LAUNCHER_TEST_LAUNCHER_H_ 207 #endif // BASE_TEST_LAUNCHER_TEST_LAUNCHER_H_
OLDNEW
« no previous file with comments | « no previous file | base/test/launcher/test_launcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698