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

Side by Side Diff: client/bin/factory.py

Issue 3197015: Fix test dargs, and statusmap creation. (Closed) Base URL: http://src.chromium.org/git/autotest.git
Patch Set: tested, this should work Created 10 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
« no previous file with comments | « no previous file | client/site_tests/factory_Review/factory_Review.py » ('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 (c) 2010 The Chromium OS Authors. All rights reserved. 1 # Copyright (c) 2010 The Chromium OS 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 5
6 # DESCRIPTION : 6 # DESCRIPTION :
7 # 7 #
8 # This library provides common types and routines for the factory ui 8 # This library provides common types and routines for the factory ui
9 # infrastructure. This library explicitly does not import gtk, to 9 # infrastructure. This library explicitly does not import gtk, to
10 # allow its use by the autotest control process. 10 # allow its use by the autotest control process.
(...skipping 327 matching lines...) Expand 10 before | Expand all | Expand 10 after
338 self._status_map = status_map 338 self._status_map = status_map
339 self._log_data = LogData() 339 self._log_data = LogData()
340 self._std_dargs = { 340 self._std_dargs = {
341 'test_widget_size': ui.test_widget_size, 341 'test_widget_size': ui.test_widget_size,
342 'trigger_set': status_map.test_db.kbd_shortcut_set, 342 'trigger_set': status_map.test_db.kbd_shortcut_set,
343 'status_file_path' : status_file_path, 343 'status_file_path' : status_file_path,
344 'test_list': test_list} 344 'test_list': test_list}
345 345
346 def run_test(self, test): 346 def run_test(self, test):
347 self._status_map.incr_count(test) 347 self._status_map.incr_count(test)
348 dargs = test.dargs 348 dargs = {}
349 dargs.update(test.dargs)
349 dargs.update(self._std_dargs) 350 dargs.update(self._std_dargs)
350 test_tag = self._status_map.lookup_tag(test) 351 test_tag = self._status_map.lookup_tag(test)
351 dargs.update({'tag': test_tag, 352 dargs.update({'tag': test_tag,
352 'subtest_tag': test_tag, 353 'subtest_tag': test_tag,
353 'shared_dict': self._log_data.shared_dict}) 354 'shared_dict': self._log_data.shared_dict})
354 if test.drop_caches: 355 if test.drop_caches:
355 self._job.drop_caches_between_iterations = True 356 self._job.drop_caches_between_iterations = True
356 self._job.run_test(test.autotest_name, **dargs) 357 self._job.run_test(test.autotest_name, **dargs)
357 self._job.drop_caches_between_iterations = False 358 self._job.drop_caches_between_iterations = False
358 self._log_data.read_new_data() 359 self._log_data.read_new_data()
359 activated_ks = self._log_data.shared_dict.pop( 360 activated_ks = self._log_data.shared_dict.pop(
360 'activated_kbd_shortcut', None) 361 'activated_kbd_shortcut', None)
361 lookup = self._status_map.test_db.get_test_by_kbd_shortcut 362 lookup = self._status_map.test_db.get_test_by_kbd_shortcut
362 self.activated_kbd_shortcut_test = ( 363 self.activated_kbd_shortcut_test = (
363 activated_ks and lookup(activated_ks) or None) 364 activated_ks and lookup(activated_ks) or None)
OLDNEW
« no previous file with comments | « no previous file | client/site_tests/factory_Review/factory_Review.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698