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

Side by Side Diff: unittests/autoroll_test.py

Issue 2829203002: [autoroller] All commits in updates(), only roll interesting ones. (Closed)
Patch Set: windows fix Created 3 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
« no previous file with comments | « recipe_engine/unittests/fetch_test.py ('k') | unittests/repo_test_util.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 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2016 The LUCI Authors. All rights reserved. 2 # Copyright 2016 The LUCI Authors. All rights reserved.
3 # Use of this source code is governed under the Apache License, Version 2.0 3 # Use of this source code is governed under the Apache License, Version 2.0
4 # that can be found in the LICENSE file. 4 # that can be found in the LICENSE file.
5 5
6 import copy 6 import copy
7 import json 7 import json
8 import logging 8 import logging
9 import os 9 import os
10 import subprocess 10 import subprocess
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 a_c1 = self.commit_in_repo(repos['a'], message='c1') 84 a_c1 = self.commit_in_repo(repos['a'], message='c1')
85 85
86 roll_result = self.run_roll(repos['b']) 86 roll_result = self.run_roll(repos['b'])
87 self.assertTrue(roll_result['success']) 87 self.assertTrue(roll_result['success'])
88 self.assertTrue(roll_result['trivial']) 88 self.assertTrue(roll_result['trivial'])
89 89
90 spec = copy.deepcopy(b_package_spec.spec_pb) 90 spec = copy.deepcopy(b_package_spec.spec_pb)
91 spec.deps['a'].revision = a_c1['revision'] 91 spec.deps['a'].revision = a_c1['revision']
92 92
93 expected_picked_roll = { 93 expected_picked_roll = {
94 'commit_infos': { 94 'commit_infos': {
95 'a': [ 95 'a': [
96 { 96 {
97 'author_email': a_c1['author_email'], 97 'author_email': a_c1['author_email'],
98 'message_lines': a_c1['message_lines'], 98 'message_lines': a_c1['message_lines'],
99 'revision': a_c1['revision'], 99 'revision': a_c1['revision'],
100 }, 100 },
101 ], 101 ],
102 }, 102 },
103 'spec': package_io.dump_obj(spec), 103 'spec': package_io.dump_obj(spec),
104 } 104 }
105 105
106 self.assertEqual(expected_picked_roll['commit_infos'], 106 self.assertEqual(expected_picked_roll['commit_infos'],
107 roll_result['picked_roll_details']['commit_infos']) 107 roll_result['picked_roll_details']['commit_infos'])
108 self.assertEqual(expected_picked_roll['spec'], 108 self.assertEqual(expected_picked_roll['spec'],
109 roll_result['picked_roll_details']['spec']) 109 roll_result['picked_roll_details']['spec'])
110 self.assertEqual( 110 self.assertEqual(
111 0, roll_result['picked_roll_details']['recipes_simulation_test']['rc']) 111 0, roll_result['picked_roll_details']['recipes_simulation_test']['rc'])
112 112
113 def test_nontrivial(self): 113 def test_nontrivial(self):
(...skipping 403 matching lines...) Expand 10 before | Expand all | Expand 10 after
517 self.assertEqual([], roll_result['roll_details']) 517 self.assertEqual([], roll_result['roll_details'])
518 self.assertGreater(len(roll_result['rejected_candidate_specs']), 0) 518 self.assertGreater(len(roll_result['rejected_candidate_specs']), 0)
519 519
520 520
521 if __name__ == '__main__': 521 if __name__ == '__main__':
522 if '-v' in sys.argv: 522 if '-v' in sys.argv:
523 logging.basicConfig( 523 logging.basicConfig(
524 level=logging.DEBUG, 524 level=logging.DEBUG,
525 handler=repo_test_util.CapturableHandler()) 525 handler=repo_test_util.CapturableHandler())
526 sys.exit(unittest.main()) 526 sys.exit(unittest.main())
OLDNEW
« no previous file with comments | « recipe_engine/unittests/fetch_test.py ('k') | unittests/repo_test_util.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698