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

Side by Side Diff: build/android/pylib/base/base_test_runner.py

Issue 12659003: Cleanup test server instance after running each a test in gtest test runner (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Just cleanup test server instance Created 7 years, 9 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 import contextlib 5 import contextlib
6 import httplib 6 import httplib
7 import logging 7 import logging
8 import os 8 import os
9 import tempfile 9 import tempfile
10 import time 10 import time
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 # proper hand-shake/shutdown. 159 # proper hand-shake/shutdown.
160 Forwarder.KillDevice(self.adb, self.tool) 160 Forwarder.KillDevice(self.adb, self.tool)
161 if self._forwarder: 161 if self._forwarder:
162 self._forwarder.Close() 162 self._forwarder.Close()
163 if self._http_server: 163 if self._http_server:
164 self._http_server.ShutdownHttpServer() 164 self._http_server.ShutdownHttpServer()
165 if self._spawning_server: 165 if self._spawning_server:
166 self._spawning_server.Stop() 166 self._spawning_server.Stop()
167 self.flags.Restore() 167 self.flags.Restore()
168 168
169 def CleanupAfterEachTest(self):
craigdh 2013/03/08 19:50:42 This name is a bit misleading as it's only used by
nilesh 2013/03/08 23:37:52 Since test server is something internal to spawnin
170 if self._spawning_server:
171 self._spawning_server.CleanupTestServer()
172
169 def LaunchChromeTestServerSpawner(self): 173 def LaunchChromeTestServerSpawner(self):
170 """Launches test server spawner.""" 174 """Launches test server spawner."""
171 server_ready = False 175 server_ready = False
172 error_msgs = [] 176 error_msgs = []
173 # Try 3 times to launch test spawner server. 177 # Try 3 times to launch test spawner server.
174 for i in xrange(0, 3): 178 for i in xrange(0, 3):
175 # Do not allocate port for test server here. We will allocate 179 # Do not allocate port for test server here. We will allocate
176 # different port for individual test in TestServerThread. 180 # different port for individual test in TestServerThread.
177 self.test_server_spawner_port = ports.AllocateTestServerPort() 181 self.test_server_spawner_port = ports.AllocateTestServerPort()
178 self._spawning_server = SpawningServer(self.test_server_spawner_port, 182 self._spawning_server = SpawningServer(self.test_server_spawner_port,
(...skipping 12 matching lines...) Expand all
191 # Wait for 2 seconds then restart. 195 # Wait for 2 seconds then restart.
192 time.sleep(2) 196 time.sleep(2)
193 if not server_ready: 197 if not server_ready:
194 logging.error(';'.join(error_msgs)) 198 logging.error(';'.join(error_msgs))
195 raise Exception('Can not start the test spawner server.') 199 raise Exception('Can not start the test spawner server.')
196 self._PushTestServerPortInfoToDevice() 200 self._PushTestServerPortInfoToDevice()
197 self._spawner_forwarder = self._CreateAndRunForwarder( 201 self._spawner_forwarder = self._CreateAndRunForwarder(
198 self.adb, 202 self.adb,
199 [(self.test_server_spawner_port, self.test_server_spawner_port)], 203 [(self.test_server_spawner_port, self.test_server_spawner_port)],
200 self.tool, '127.0.0.1', self.build_type) 204 self.tool, '127.0.0.1', self.build_type)
OLDNEW
« no previous file with comments | « no previous file | build/android/pylib/chrome_test_server_spawner.py » ('j') | build/android/pylib/chrome_test_server_spawner.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698