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

Side by Side Diff: chromeos/process_proxy/process_output_watcher_unittest.cc

Issue 407073010: Disable all the tests that are flaking more than 5% on ChromeOS builders. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review comments Created 6 years, 5 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include <gtest/gtest.h> 5 #include <gtest/gtest.h>
6 6
7 #include <queue> 7 #include <queue>
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
170 private: 170 private:
171 base::Closure test_case_done_callback_; 171 base::Closure test_case_done_callback_;
172 base::MessageLoop message_loop_; 172 base::MessageLoop message_loop_;
173 scoped_ptr<base::Thread> output_watch_thread_; 173 scoped_ptr<base::Thread> output_watch_thread_;
174 bool output_watch_thread_started_; 174 bool output_watch_thread_started_;
175 bool failed_; 175 bool failed_;
176 ProcessWatcherExpectations expectations_; 176 ProcessWatcherExpectations expectations_;
177 std::vector<TestCase> exp; 177 std::vector<TestCase> exp;
178 }; 178 };
179 179
180 180 // http://crbug.com/396496
181 TEST_F(ProcessOutputWatcherTest, OutputWatcher) { 181 TEST_F(ProcessOutputWatcherTest, DISABLED_OutputWatcher) {
182 std::vector<TestCase> test_cases; 182 std::vector<TestCase> test_cases;
183 test_cases.push_back(TestCase("t", false)); 183 test_cases.push_back(TestCase("t", false));
184 test_cases.push_back(TestCase("testing output\n", false)); 184 test_cases.push_back(TestCase("testing output\n", false));
185 test_cases.push_back(TestCase("testing error\n", false)); 185 test_cases.push_back(TestCase("testing error\n", false));
186 test_cases.push_back(TestCase("testing error1\n", false)); 186 test_cases.push_back(TestCase("testing error1\n", false));
187 test_cases.push_back(TestCase("testing output1\n", false)); 187 test_cases.push_back(TestCase("testing output1\n", false));
188 test_cases.push_back(TestCase("testing output2\n", false)); 188 test_cases.push_back(TestCase("testing output2\n", false));
189 test_cases.push_back(TestCase("testing output3\n", false)); 189 test_cases.push_back(TestCase("testing output3\n", false));
190 test_cases.push_back(TestCase(VeryLongString(), false)); 190 test_cases.push_back(TestCase(VeryLongString(), false));
191 test_cases.push_back(TestCase("testing error2\n", false)); 191 test_cases.push_back(TestCase("testing error2\n", false));
192 192
193 RunTest(test_cases); 193 RunTest(test_cases);
194 }; 194 };
195 195
196 TEST_F(ProcessOutputWatcherTest, SplitUTF8Character) { 196 // http://crbug.com/396496
197 TEST_F(ProcessOutputWatcherTest, DISABLED_SplitUTF8Character) {
197 std::vector<TestCase> test_cases; 198 std::vector<TestCase> test_cases;
198 test_cases.push_back(TestCase("test1\xc2", false, "test1")); 199 test_cases.push_back(TestCase("test1\xc2", false, "test1"));
199 test_cases.push_back(TestCase("\xb5test1", false, "\xc2\xb5test1")); 200 test_cases.push_back(TestCase("\xb5test1", false, "\xc2\xb5test1"));
200 201
201 RunTest(test_cases); 202 RunTest(test_cases);
202 } 203 }
203 204
204 TEST_F(ProcessOutputWatcherTest, SplitSoleUTF8Character) { 205 // http://crbug.com/396496
206 TEST_F(ProcessOutputWatcherTest, DISABLED_SplitSoleUTF8Character) {
205 std::vector<TestCase> test_cases; 207 std::vector<TestCase> test_cases;
206 test_cases.push_back(TestCase("\xc2", false, "")); 208 test_cases.push_back(TestCase("\xc2", false, ""));
207 test_cases.push_back(TestCase("\xb5", false, "\xc2\xb5")); 209 test_cases.push_back(TestCase("\xb5", false, "\xc2\xb5"));
208 210
209 RunTest(test_cases); 211 RunTest(test_cases);
210 } 212 }
211 213
212 TEST_F(ProcessOutputWatcherTest, SplitUTF8CharacterLength3) { 214 // http://crbug.com/396496
215 TEST_F(ProcessOutputWatcherTest, DISABLED_SplitUTF8CharacterLength3) {
213 std::vector<TestCase> test_cases; 216 std::vector<TestCase> test_cases;
214 test_cases.push_back(TestCase("test3\xe2\x82", false, "test3")); 217 test_cases.push_back(TestCase("test3\xe2\x82", false, "test3"));
215 test_cases.push_back(TestCase("\xac", false, "\xe2\x82\xac")); 218 test_cases.push_back(TestCase("\xac", false, "\xe2\x82\xac"));
216 219
217 RunTest(test_cases); 220 RunTest(test_cases);
218 } 221 }
219 222
220 TEST_F(ProcessOutputWatcherTest, SplitSoleUTF8CharacterThreeWays) { 223 // http://crbug.com/396496
224 TEST_F(ProcessOutputWatcherTest, DISABLED_SplitSoleUTF8CharacterThreeWays) {
221 std::vector<TestCase> test_cases; 225 std::vector<TestCase> test_cases;
222 test_cases.push_back(TestCase("\xe2", false, "")); 226 test_cases.push_back(TestCase("\xe2", false, ""));
223 test_cases.push_back(TestCase("\x82", false, "")); 227 test_cases.push_back(TestCase("\x82", false, ""));
224 test_cases.push_back(TestCase("\xac", false, "\xe2\x82\xac")); 228 test_cases.push_back(TestCase("\xac", false, "\xe2\x82\xac"));
225 229
226 RunTest(test_cases); 230 RunTest(test_cases);
227 } 231 }
228 232
229 TEST_F(ProcessOutputWatcherTest, EndsWithThreeByteUTF8Character) { 233 TEST_F(ProcessOutputWatcherTest, EndsWithThreeByteUTF8Character) {
230 std::vector<TestCase> test_cases; 234 std::vector<TestCase> test_cases;
(...skipping 17 matching lines...) Expand all
248 RunTest(test_cases); 252 RunTest(test_cases);
249 } 253 }
250 254
251 TEST_F(ProcessOutputWatcherTest, MulitByteUTF8CharNullTerminated) { 255 TEST_F(ProcessOutputWatcherTest, MulitByteUTF8CharNullTerminated) {
252 std::vector<TestCase> test_cases; 256 std::vector<TestCase> test_cases;
253 test_cases.push_back(TestCase("test\xe2\x82\xac", true, "test\xe2\x82\xac")); 257 test_cases.push_back(TestCase("test\xe2\x82\xac", true, "test\xe2\x82\xac"));
254 258
255 RunTest(test_cases); 259 RunTest(test_cases);
256 } 260 }
257 261
258 TEST_F(ProcessOutputWatcherTest, MultipleMultiByteUTF8Characters) { 262 // http://crbug.com/396496
263 TEST_F(ProcessOutputWatcherTest, DISABLED_MultipleMultiByteUTF8Characters) {
259 std::vector<TestCase> test_cases; 264 std::vector<TestCase> test_cases;
260 test_cases.push_back( 265 test_cases.push_back(
261 TestCase("test\xe2\x82\xac\xc2", false, "test\xe2\x82\xac")); 266 TestCase("test\xe2\x82\xac\xc2", false, "test\xe2\x82\xac"));
262 test_cases.push_back(TestCase("\xb5", false, "\xc2\xb5")); 267 test_cases.push_back(TestCase("\xb5", false, "\xc2\xb5"));
263 268
264 RunTest(test_cases); 269 RunTest(test_cases);
265 } 270 }
266 271
267 TEST_F(ProcessOutputWatcherTest, ContainsInvalidUTF8) { 272 TEST_F(ProcessOutputWatcherTest, ContainsInvalidUTF8) {
268 std::vector<TestCase> test_cases; 273 std::vector<TestCase> test_cases;
269 test_cases.push_back(TestCase("\xc2_", false, "\xc2_")); 274 test_cases.push_back(TestCase("\xc2_", false, "\xc2_"));
270 275
271 RunTest(test_cases); 276 RunTest(test_cases);
272 } 277 }
273 278
274 TEST_F(ProcessOutputWatcherTest, InvalidUTF8SeriesOfTrailingBytes) { 279 // http://crbug.com/396496
280 TEST_F(ProcessOutputWatcherTest, DISABLED_InvalidUTF8SeriesOfTrailingBytes) {
275 std::vector<TestCase> test_cases; 281 std::vector<TestCase> test_cases;
276 test_cases.push_back(TestCase("\x82\x82\x82", false, "\x82\x82\x82")); 282 test_cases.push_back(TestCase("\x82\x82\x82", false, "\x82\x82\x82"));
277 test_cases.push_back(TestCase("\x82\x82\x82", false, "\x82\x82\x82")); 283 test_cases.push_back(TestCase("\x82\x82\x82", false, "\x82\x82\x82"));
278 284
279 RunTest(test_cases); 285 RunTest(test_cases);
280 } 286 }
281 287
282 TEST_F(ProcessOutputWatcherTest, EndsWithInvalidUTF8) { 288 TEST_F(ProcessOutputWatcherTest, EndsWithInvalidUTF8) {
283 std::vector<TestCase> test_cases; 289 std::vector<TestCase> test_cases;
284 test_cases.push_back(TestCase("\xff", false, "\xff")); 290 test_cases.push_back(TestCase("\xff", false, "\xff"));
285 291
286 RunTest(test_cases); 292 RunTest(test_cases);
287 } 293 }
288 294
289 TEST_F(ProcessOutputWatcherTest, FourByteUTF8) { 295 // http://crbug.com/396496
296 TEST_F(ProcessOutputWatcherTest, DISABLED_FourByteUTF8) {
290 std::vector<TestCase> test_cases; 297 std::vector<TestCase> test_cases;
291 test_cases.push_back(TestCase("\xf0\xa4\xad", false, "")); 298 test_cases.push_back(TestCase("\xf0\xa4\xad", false, ""));
292 test_cases.push_back(TestCase("\xa2", false, "\xf0\xa4\xad\xa2")); 299 test_cases.push_back(TestCase("\xa2", false, "\xf0\xa4\xad\xa2"));
293 300
294 RunTest(test_cases); 301 RunTest(test_cases);
295 }; 302 };
296 303
297 // Verifies that sending '\0' generates PROCESS_OUTPUT_TYPE_OUT event and does 304 // Verifies that sending '\0' generates PROCESS_OUTPUT_TYPE_OUT event and does
298 // not terminate output watcher. 305 // not terminate output watcher.
299 TEST_F(ProcessOutputWatcherTest, SendNull) { 306 // http://crbug.com/396496
307 TEST_F(ProcessOutputWatcherTest, DISABLED_SendNull) {
300 std::vector<TestCase> test_cases; 308 std::vector<TestCase> test_cases;
301 // This will send '\0' to output watcher. 309 // This will send '\0' to output watcher.
302 test_cases.push_back(TestCase("", true)); 310 test_cases.push_back(TestCase("", true));
303 // Let's verify that next input also gets detected (i.e. output watcher does 311 // Let's verify that next input also gets detected (i.e. output watcher does
304 // not exit after seeing '\0' from previous test case). 312 // not exit after seeing '\0' from previous test case).
305 test_cases.push_back(TestCase("a", true)); 313 test_cases.push_back(TestCase("a", true));
306 314
307 RunTest(test_cases); 315 RunTest(test_cases);
308 }; 316 };
309 317
310 } // namespace chromeos 318 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/test/ppapi/ppapi_browsertest.cc ('k') | content/browser/service_worker/service_worker_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698