build/android/pylib/gtest/test_runner.py - Issue 21405003: [android] Clear content_browsertests profile after each run.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: build/android/pylib/gtest/test_runner.py

Issue 21405003: [android] Clear content_browsertests profile after each run. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Created 7 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

OLD	NEW
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 import logging	5 import logging

6 import os	6 import os

7 import re	7 import re

8	8

9 from pylib import android_commands	9 from pylib import android_commands

10 from pylib import constants	10 from pylib import constants

(...skipping 165 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
176 retry = ':'.join([t.GetName() for t in test_results.GetNotPass()])	176 retry = ':'.join([t.GetName() for t in test_results.GetNotPass()])

177 return test_results, retry	177 return test_results, retry

178	178

179 #override	179 #override

180 def SetUp(self):	180 def SetUp(self):

181 """Sets up necessary test enviroment for the test suite."""	181 """Sets up necessary test enviroment for the test suite."""

182 super(TestRunner, self).SetUp()	182 super(TestRunner, self).SetUp()

183 if _TestSuiteRequiresMockTestServer(self.test_package.suite_name):	183 if _TestSuiteRequiresMockTestServer(self.test_package.suite_name):

184 self.LaunchChromeTestServerSpawner()	184 self.LaunchChromeTestServerSpawner()

185 self.tool.SetupEnvironment()	185 self.tool.SetupEnvironment()

	186 self.adb.RunShellCommand('rm -r /sdcard/content_shell', timeout_time=60 * 2)
	frankf 2013/07/31 22:12:30 As a follow up, can you address: crbug.com/251023 As a follow up, can you address: crbug.com/251023 I don't like these arbitrary hardcoded timeouts. frankf 2013/07/31 22:12:30 Use adb.GetExternalStorage Use adb.GetExternalStorage frankf 2013/07/31 22:12:30 Makes more sense to be in TearDown Makes more sense to be in TearDown craigdh 2013/07/31 22:49:04 Done. Show quoted text On 2013/07/31 22:12:30, frankf wrote: > Use adb.GetExternalStorage Done. craigdh 2013/07/31 22:49:04 Done. Show quoted text On 2013/07/31 22:12:30, frankf wrote: > Makes more sense to be in TearDown Done.
186	187

187 #override	188 #override

188 def TearDown(self):	189 def TearDown(self):

189 """Cleans up the test enviroment for the test suite."""	190 """Cleans up the test enviroment for the test suite."""

190 self.tool.CleanUpEnvironment()	191 self.tool.CleanUpEnvironment()

191 super(TestRunner, self).TearDown()	192 super(TestRunner, self).TearDown()

OLD	NEW

« no previous file with comments | « no previous file | no next file » | no next file with comments »