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

Side by Side Diff: gm/rebaseline_server/compare_configs_test.py

Issue 215503002: rebaseline_server: add --compare-configs option (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 8 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
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 2
3 """ 3 """
4 Copyright 2013 Google Inc. 4 Copyright 2014 Google Inc.
epoger 2014/03/27 21:31:07 rietveld is showing diffs against compare_to_expec
rmistry 2014/03/28 18:51:39 Acknowledged.
5 5
6 Use of this source code is governed by a BSD-style license that can be 6 Use of this source code is governed by a BSD-style license that can be
7 found in the LICENSE file. 7 found in the LICENSE file.
8 8
9 Test compare_to_expectations.py 9 Test compare_configs.py
10 10
11 TODO(epoger): Create a command to update the expected results (in 11 TODO(epoger): Create a command to update the expected results (in
12 self._output_dir_expected) when appropriate. For now, you should: 12 self._output_dir_expected) when appropriate. For now, you should:
13 1. examine the results in self._output_dir_actual and make sure they are ok 13 1. examine the results in self._output_dir_actual and make sure they are ok
14 2. rm -rf self._output_dir_expected 14 2. rm -rf self._output_dir_expected
15 3. mv self._output_dir_actual self._output_dir_expected 15 3. mv self._output_dir_actual self._output_dir_expected
16 Although, if you're using an SVN checkout, this will blow away .svn directories 16 Although, if you're using an SVN checkout, this will blow away .svn directories
17 within self._output_dir_expected, which wouldn't be good... 17 within self._output_dir_expected, which wouldn't be good...
18 18
19 """ 19 """
20 20
21 import os 21 import os
22 import sys 22 import sys
23 23
24 # Imports from within Skia 24 # Imports from within Skia
25 import base_unittest 25 import base_unittest
26 import compare_to_expectations 26 import compare_configs
27 import results 27 import results
28 import gm_json # must import results first, so that gm_json will be in sys.path 28 import gm_json # must import results first, so that gm_json will be in sys.path
29 29
30 30
31 class CompareToExpectationsTest(base_unittest.TestCase): 31 class CompareConfigsTest(base_unittest.TestCase):
32 32
33 def test_gm(self): 33 def test_gm(self):
34 """Process results of a GM run with the Results object.""" 34 """Process results of a GM run with the ConfigComparisons object."""
35 results_obj = compare_to_expectations.Results( 35 results_obj = compare_configs.ConfigComparisons(
36 configs=('8888', 'gpu'),
36 actuals_root=os.path.join(self._input_dir, 'gm-actuals'), 37 actuals_root=os.path.join(self._input_dir, 'gm-actuals'),
37 expected_root=os.path.join(self._input_dir, 'gm-expectations'),
38 generated_images_root=self._temp_dir, 38 generated_images_root=self._temp_dir,
39 diff_base_url='/static/generated-images') 39 diff_base_url='/static/generated-images')
40 results_obj.get_timestamp = mock_get_timestamp 40 results_obj.get_timestamp = mock_get_timestamp
41 gm_json.WriteToFile( 41 gm_json.WriteToFile(
42 results_obj.get_packaged_results_of_type( 42 results_obj.get_packaged_results_of_type(
43 results.KEY__HEADER__RESULTS_ALL), 43 results.KEY__HEADER__RESULTS_ALL),
44 os.path.join(self._output_dir_actual, 'gm.json')) 44 os.path.join(self._output_dir_actual, 'gm.json'))
45 45
46 46
47 def mock_get_timestamp(): 47 def mock_get_timestamp():
48 """Mock version of Results.get_timestamp() for testing.""" 48 """Mock version of BaseComparisons.get_timestamp() for testing."""
49 return 12345678 49 return 12345678
50 50
51 51
52 def main(): 52 def main():
53 base_unittest.main(CompareToExpectationsTest) 53 base_unittest.main(CompareConfigsTest)
54 54
55 55
56 if __name__ == '__main__': 56 if __name__ == '__main__':
57 main() 57 main()
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698