blob: fc7aa1d9c784c956e64db1f6cc607b22069c75f4 [file] [log] [blame]
Mike Frysingere58c0e22017-10-04 15:43:30 -04001# -*- coding: utf-8 -*-
Don Garrettc4114cc2016-11-01 20:04:06 -07002# Copyright 2016 The Chromium OS Authors. All rights reserved.
3# Use of this source code is governed by a BSD-style license that can be
4# found in the LICENSE file.
5
Prathmesh Prabhuc41a0f52018-04-03 13:26:52 -07006"""Unit tests for chromite.scripts.cbuildbot_launch."""
Don Garrettc4114cc2016-11-01 20:04:06 -07007
8from __future__ import print_function
9
Don Garrett86881cb2017-02-15 15:41:55 -080010import mock
Don Garrett7ade05a2017-02-17 13:31:47 -080011import os
Prathmesh Prabhuc41a0f52018-04-03 13:26:52 -070012import time
Don Garrett86881cb2017-02-15 15:41:55 -080013
14from chromite.cbuildbot import repository
Benjamin Gordon90b2dd92018-04-12 14:04:21 -060015from chromite.lib import build_summary
Don Garrett861e9182017-05-15 15:30:23 -070016from chromite.lib import constants
Don Garrett597ddff2017-02-17 18:29:37 -080017from chromite.lib import cros_build_lib
Benjamin Gordon74645232018-05-04 17:40:42 -060018from chromite.lib import cros_sdk_lib
Don Garrett7ade05a2017-02-17 13:31:47 -080019from chromite.lib import cros_test_lib
Don Garrett86881cb2017-02-15 15:41:55 -080020from chromite.lib import osutils
Don Garrett0c54ed72017-03-03 11:18:57 -080021from chromite.scripts import cbuildbot_launch
Don Garrett86881cb2017-02-15 15:41:55 -080022
Don Garrett86881cb2017-02-15 15:41:55 -080023EXPECTED_MANIFEST_URL = 'https://chrome-internal-review.googlesource.com/chromeos/manifest-internal' # pylint: disable=line-too-long
Don Garrettc4114cc2016-11-01 20:04:06 -070024
25
Don Garrettacbb2392017-05-11 18:27:41 -070026# It's reasonable for unittests to look at internals.
27# pylint: disable=protected-access
28
29
Don Garrett8d314792017-05-18 13:11:42 -070030class FakeException(Exception):
31 """Test exception to raise during tests."""
32
33
Don Garrett0c54ed72017-03-03 11:18:57 -080034class CbuildbotLaunchTest(cros_test_lib.MockTestCase):
35 """Tests for cbuildbot_launch script."""
Don Garrettc4114cc2016-11-01 20:04:06 -070036
Don Garrett86881cb2017-02-15 15:41:55 -080037 def testPreParseArguments(self):
Don Garrettc4114cc2016-11-01 20:04:06 -070038 """Test that we can correctly extract branch values from cbuildbot args."""
Don Garrett597ddff2017-02-17 18:29:37 -080039 CASES = (
40 (['--buildroot', '/buildroot', 'daisy-incremental'],
41 (None, '/buildroot', None)),
42
43 (['--buildbot', '--buildroot', '/buildroot',
44 '--git-cache-dir', '/git-cache',
45 '-b', 'release-R57-9202.B',
46 'daisy-incremental'],
47 ('release-R57-9202.B', '/buildroot', '/git-cache')),
48
49 (['--debug', '--buildbot', '--notests',
50 '--buildroot', '/buildroot',
51 '--git-cache-dir', '/git-cache',
52 '--branch', 'release-R57-9202.B',
53 'daisy-incremental'],
54 ('release-R57-9202.B', '/buildroot', '/git-cache')),
Don Garrettc4114cc2016-11-01 20:04:06 -070055 )
56
Don Garrett597ddff2017-02-17 18:29:37 -080057 for cmd_args, expected in CASES:
58 expected_branch, expected_buildroot, expected_cache_dir = expected
59
Don Garrett0c54ed72017-03-03 11:18:57 -080060 options = cbuildbot_launch.PreParseArguments(cmd_args)
Don Garrett597ddff2017-02-17 18:29:37 -080061
62 self.assertEqual(options.branch, expected_branch)
63 self.assertEqual(options.buildroot, expected_buildroot)
64 self.assertEqual(options.git_cache_dir, expected_cache_dir)
Don Garrett86881cb2017-02-15 15:41:55 -080065
Don Garrettf324bc32017-05-23 14:00:53 -070066 def testInitialCheckout(self):
Don Garrett86881cb2017-02-15 15:41:55 -080067 """Test InitialCheckout with minimum settings."""
Don Garrettf324bc32017-05-23 14:00:53 -070068 mock_repo = mock.MagicMock()
69 mock_repo.branch = 'branch'
Don Garrett86881cb2017-02-15 15:41:55 -080070
Don Garrettf324bc32017-05-23 14:00:53 -070071 cbuildbot_launch.InitialCheckout(mock_repo)
Don Garrett86881cb2017-02-15 15:41:55 -080072
73 self.assertEqual(mock_repo.mock_calls, [
Don Garrett5516acb2018-11-15 16:02:59 -080074 mock.call.PreLoad('/preload/chromeos'),
Don Garrettf324bc32017-05-23 14:00:53 -070075 mock.call.Sync(detach=True),
Don Garrett8d314792017-05-18 13:11:42 -070076 ])
77
Don Garrettf15d65b2017-04-12 12:39:55 -070078 def testConfigureGlobalEnvironment(self):
Don Garrett60967922017-04-12 18:51:44 -070079 """Ensure that we can setup our global runtime environment correctly."""
Don Garrett86fec482017-05-17 18:13:33 -070080
81 os.environ.pop('LANG', None)
82 os.environ['LC_MONETARY'] = 'bad'
83
Don Garrettf15d65b2017-04-12 12:39:55 -070084 cbuildbot_launch.ConfigureGlobalEnvironment()
85
Don Garrett86fec482017-05-17 18:13:33 -070086 # Verify umask is updated.
Don Garrettf15d65b2017-04-12 12:39:55 -070087 self.assertEqual(os.umask(0), 0o22)
88
Don Garrett86fec482017-05-17 18:13:33 -070089 # Verify ENVs are cleaned up.
90 self.assertEqual(os.environ['LANG'], 'en_US.UTF-8')
91 self.assertNotIn('LC_MONETARY', os.environ)
92
Don Garrettf15d65b2017-04-12 12:39:55 -070093
Benjamin Gordon121a2aa2018-05-04 16:24:45 -060094class RunTests(cros_test_lib.RunCommandTestCase):
Don Garrett0c54ed72017-03-03 11:18:57 -080095 """Tests for cbuildbot_launch script."""
Don Garrett597ddff2017-02-17 18:29:37 -080096
97 ARGS_BASE = ['--buildroot', '/buildroot']
Don Garrett5cd946b2017-07-20 13:42:20 -070098 EXPECTED_ARGS_BASE = ['--buildroot', '/cbuildbot_buildroot']
Don Garrett597ddff2017-02-17 18:29:37 -080099 ARGS_GIT_CACHE = ['--git-cache-dir', '/git-cache']
100 ARGS_CONFIG = ['config']
Don Garrettbf90cdf2017-05-19 15:54:02 -0700101 CMD = ['/cbuildbot_buildroot/chromite/bin/cbuildbot']
Don Garrett597ddff2017-02-17 18:29:37 -0800102
Don Garrett6e5c6b92018-04-06 17:58:49 -0700103 def verifyCbuildbot(self, args, expected_cmd, version):
Don Garrett86881cb2017-02-15 15:41:55 -0800104 """Ensure we invoke cbuildbot correctly."""
Don Garrett597ddff2017-02-17 18:29:37 -0800105 self.PatchObject(
106 cros_build_lib, 'GetTargetChromiteApiVersion', autospec=True,
107 return_value=version)
108
Don Garrett6e5c6b92018-04-06 17:58:49 -0700109 cbuildbot_launch.Cbuildbot('/cbuildbot_buildroot', '/depot_tools', args)
Don Garrett597ddff2017-02-17 18:29:37 -0800110
111 self.assertCommandCalled(
Don Garretta50bf492017-09-28 18:33:02 -0700112 expected_cmd, extra_env={'PATH': mock.ANY},
113 cwd='/cbuildbot_buildroot', error_code_ok=True)
Don Garrett597ddff2017-02-17 18:29:37 -0800114
Don Garrett6e5c6b92018-04-06 17:58:49 -0700115 def testCbuildbotSimple(self):
Don Garrett597ddff2017-02-17 18:29:37 -0800116 """Ensure we invoke cbuildbot correctly."""
Don Garrett6e5c6b92018-04-06 17:58:49 -0700117 self.verifyCbuildbot(
Don Garrett597ddff2017-02-17 18:29:37 -0800118 self.ARGS_BASE + self.ARGS_CONFIG,
Don Garrettbf90cdf2017-05-19 15:54:02 -0700119 self.CMD + self.ARGS_CONFIG + self.EXPECTED_ARGS_BASE,
Don Garrett597ddff2017-02-17 18:29:37 -0800120 (0, 4))
121
Don Garrett6e5c6b92018-04-06 17:58:49 -0700122 def testCbuildbotNotFiltered(self):
Don Garrett597ddff2017-02-17 18:29:37 -0800123 """Ensure we invoke cbuildbot correctly."""
Don Garrett6e5c6b92018-04-06 17:58:49 -0700124 self.verifyCbuildbot(
Don Garrett597ddff2017-02-17 18:29:37 -0800125 self.ARGS_BASE + self.ARGS_CONFIG + self.ARGS_GIT_CACHE,
Don Garrettbf90cdf2017-05-19 15:54:02 -0700126 (self.CMD + self.ARGS_CONFIG + self.EXPECTED_ARGS_BASE +
127 self.ARGS_GIT_CACHE),
Don Garrett597ddff2017-02-17 18:29:37 -0800128 (0, 4))
129
Don Garrett6e5c6b92018-04-06 17:58:49 -0700130 def testCbuildbotFiltered(self):
Don Garrett597ddff2017-02-17 18:29:37 -0800131 """Ensure we invoke cbuildbot correctly."""
Don Garrett6e5c6b92018-04-06 17:58:49 -0700132 self.verifyCbuildbot(
Don Garrett597ddff2017-02-17 18:29:37 -0800133 self.ARGS_BASE + self.ARGS_CONFIG + self.ARGS_GIT_CACHE,
Don Garrettbf90cdf2017-05-19 15:54:02 -0700134 self.CMD + self.ARGS_CONFIG + self.EXPECTED_ARGS_BASE,
Don Garrett597ddff2017-02-17 18:29:37 -0800135 (0, 2))
Don Garrettc4114cc2016-11-01 20:04:06 -0700136
Don Garrett86881cb2017-02-15 15:41:55 -0800137 def testMainMin(self):
138 """Test a minimal set of command line options."""
Don Garrett597ddff2017-02-17 18:29:37 -0800139 self.PatchObject(osutils, 'SafeMakedirs', autospec=True)
140 self.PatchObject(cros_build_lib, 'GetTargetChromiteApiVersion',
Don Garrett861e9182017-05-15 15:30:23 -0700141 autospec=True, return_value=(constants.REEXEC_API_MAJOR,
142 constants.REEXEC_API_MINOR))
Don Garrettf324bc32017-05-23 14:00:53 -0700143 mock_repo = mock.MagicMock()
144 mock_repo.branch = 'master'
Don Garrettbf90cdf2017-05-19 15:54:02 -0700145 mock_repo.directory = '/root/repository'
146
Don Garrettf324bc32017-05-23 14:00:53 -0700147 mock_repo_create = self.PatchObject(repository, 'RepoRepository',
148 autospec=True, return_value=mock_repo)
Don Garrettbf90cdf2017-05-19 15:54:02 -0700149 mock_clean = self.PatchObject(cbuildbot_launch, 'CleanBuildRoot',
Don Garrett0c54ed72017-03-03 11:18:57 -0800150 autospec=True)
151 mock_checkout = self.PatchObject(cbuildbot_launch, 'InitialCheckout',
Don Garrett86881cb2017-02-15 15:41:55 -0800152 autospec=True)
Benjamin Gordonaee36b82018-02-05 14:25:26 -0700153 mock_cleanup_chroot = self.PatchObject(cbuildbot_launch, 'CleanupChroot',
154 autospec=True)
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600155 mock_set_last_build_state = self.PatchObject(
156 cbuildbot_launch, 'SetLastBuildState', autospec=True)
157
158 expected_build_state = build_summary.BuildSummary(
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600159 build_number=0, master_build_id=0, status=mock.ANY,
160 buildroot_layout=2, branch='master')
Don Garrett7ade05a2017-02-17 13:31:47 -0800161
Dhanya Ganesh95c5c152018-10-08 16:48:29 -0600162 argv = ['-r', '/root', 'config']
163 options = cbuildbot_launch.PreParseArguments(argv)
164 cbuildbot_launch._main(options, argv)
Don Garrettc4114cc2016-11-01 20:04:06 -0700165
Don Garrettf324bc32017-05-23 14:00:53 -0700166 # Did we create the repo instance correctly?
167 self.assertEqual(mock_repo_create.mock_calls,
Don Garrettbf90cdf2017-05-19 15:54:02 -0700168 [mock.call(EXPECTED_MANIFEST_URL, '/root/repository',
Don Garrett33872502018-08-03 22:30:40 +0000169 git_cache_dir=None, branch='master')])
Don Garrettf324bc32017-05-23 14:00:53 -0700170
Don Garrett7ade05a2017-02-17 13:31:47 -0800171 # Ensure we clean, as expected.
Don Garrettbf90cdf2017-05-19 15:54:02 -0700172 self.assertEqual(mock_clean.mock_calls, [
Don Garrett61ce1ee2019-02-26 16:20:25 -0800173 mock.call('/root', mock_repo, '/root/repository/.cache',
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600174 expected_build_state)])
Don Garrett7ade05a2017-02-17 13:31:47 -0800175
Don Garrett86881cb2017-02-15 15:41:55 -0800176 # Ensure we checkout, as expected.
177 self.assertEqual(mock_checkout.mock_calls,
Don Garrettf324bc32017-05-23 14:00:53 -0700178 [mock.call(mock_repo)])
Don Garrettc4114cc2016-11-01 20:04:06 -0700179
Don Garrett86881cb2017-02-15 15:41:55 -0800180 # Ensure we invoke cbuildbot, as expected.
181 self.assertCommandCalled(
Don Garrett5cd946b2017-07-20 13:42:20 -0700182 [
183 '/root/repository/chromite/bin/cbuildbot',
184 'config',
185 '-r', '/root/repository',
Don Garrettb497f552018-07-09 16:01:13 -0700186 '--workspace', '/root/workspace',
Don Garrett61ce1ee2019-02-26 16:20:25 -0800187 '--cache-dir', '/root/repository/.cache',
188 # The duplication is a bug, but not harmful.
189 '--cache-dir', '/root/repository/.cache',
Don Garrett5cd946b2017-07-20 13:42:20 -0700190 '--ts-mon-task-num', '1',
191 ],
Don Garretta50bf492017-09-28 18:33:02 -0700192 extra_env={'PATH': mock.ANY},
Don Garrettbf90cdf2017-05-19 15:54:02 -0700193 cwd='/root/repository',
Don Garrettacbb2392017-05-11 18:27:41 -0700194 error_code_ok=True)
Don Garrettc4114cc2016-11-01 20:04:06 -0700195
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600196 # Ensure we saved the final state, as expected.
197 self.assertEqual(expected_build_state.status,
198 constants.BUILDER_STATUS_PASSED)
199 self.assertEqual(mock_set_last_build_state.mock_calls, [
200 mock.call('/root', expected_build_state)])
201
Benjamin Gordonaee36b82018-02-05 14:25:26 -0700202 # Ensure we clean the chroot, as expected.
203 self.assertEqual(mock_cleanup_chroot.mock_calls, [
204 mock.call('/root/repository')])
205
Don Garrett86881cb2017-02-15 15:41:55 -0800206 def testMainMax(self):
Don Garrett597ddff2017-02-17 18:29:37 -0800207 """Test a larger set of command line options."""
208 self.PatchObject(osutils, 'SafeMakedirs', autospec=True)
209 self.PatchObject(cros_build_lib, 'GetTargetChromiteApiVersion',
Don Garrett861e9182017-05-15 15:30:23 -0700210 autospec=True, return_value=(constants.REEXEC_API_MAJOR,
211 constants.REEXEC_API_MINOR))
Don Garrettf324bc32017-05-23 14:00:53 -0700212 mock_repo = mock.MagicMock()
213 mock_repo.branch = 'branch'
Don Garrettbf90cdf2017-05-19 15:54:02 -0700214 mock_repo.directory = '/root/repository'
215
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600216 mock_summary = build_summary.BuildSummary(
217 build_number=313,
218 master_build_id=123123123,
219 status=constants.BUILDER_STATUS_FAILED,
220 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
221 branch='branch')
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600222
223 mock_get_last_build_state = self.PatchObject(
224 cbuildbot_launch, 'GetLastBuildState', autospec=True,
225 return_value=mock_summary)
Don Garrettf324bc32017-05-23 14:00:53 -0700226 mock_repo_create = self.PatchObject(repository, 'RepoRepository',
227 autospec=True, return_value=mock_repo)
Don Garrettbf90cdf2017-05-19 15:54:02 -0700228 mock_clean = self.PatchObject(cbuildbot_launch, 'CleanBuildRoot',
Don Garrett0c54ed72017-03-03 11:18:57 -0800229 autospec=True)
230 mock_checkout = self.PatchObject(cbuildbot_launch, 'InitialCheckout',
Don Garrett86881cb2017-02-15 15:41:55 -0800231 autospec=True)
Benjamin Gordonaee36b82018-02-05 14:25:26 -0700232 mock_cleanup_chroot = self.PatchObject(cbuildbot_launch, 'CleanupChroot',
233 autospec=True)
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600234 mock_set_last_build_state = self.PatchObject(
235 cbuildbot_launch, 'SetLastBuildState', autospec=True)
Dhanya Ganesh95c5c152018-10-08 16:48:29 -0600236 argv = ['--buildroot', '/root',
237 '--branch', 'branch',
238 '--git-cache-dir', '/git-cache',
Don Garrett61ce1ee2019-02-26 16:20:25 -0800239 '--cache-dir', '/cache',
Dhanya Ganesh95c5c152018-10-08 16:48:29 -0600240 '--remote-trybot',
241 '--master-build-id', '123456789',
242 '--buildnumber', '314',
243 'config']
244 options = cbuildbot_launch.PreParseArguments(argv)
245 cbuildbot_launch._main(options, argv)
Don Garrettc4114cc2016-11-01 20:04:06 -0700246
Don Garrettf324bc32017-05-23 14:00:53 -0700247 # Did we create the repo instance correctly?
248 self.assertEqual(mock_repo_create.mock_calls,
Don Garrettbf90cdf2017-05-19 15:54:02 -0700249 [mock.call(EXPECTED_MANIFEST_URL, '/root/repository',
Don Garrett33872502018-08-03 22:30:40 +0000250 git_cache_dir='/git-cache', branch='branch')])
Don Garrettf324bc32017-05-23 14:00:53 -0700251
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600252 # Ensure we look up the previous status.
253 self.assertEqual(mock_get_last_build_state.mock_calls, [
254 mock.call('/root')])
255
Don Garrett7ade05a2017-02-17 13:31:47 -0800256 # Ensure we clean, as expected.
Don Garrettbf90cdf2017-05-19 15:54:02 -0700257 self.assertEqual(mock_clean.mock_calls, [
Don Garrettd1d90dd2017-06-13 17:35:52 -0700258 mock.call('/root',
259 mock_repo,
Don Garrett61ce1ee2019-02-26 16:20:25 -0800260 '/cache',
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600261 build_summary.BuildSummary(
262 build_number=314,
263 master_build_id=123456789,
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600264 status=mock.ANY,
265 branch='branch',
266 buildroot_layout=2
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600267 ))])
Don Garrett7ade05a2017-02-17 13:31:47 -0800268
Don Garrett86881cb2017-02-15 15:41:55 -0800269 # Ensure we checkout, as expected.
270 self.assertEqual(mock_checkout.mock_calls,
Don Garrettf324bc32017-05-23 14:00:53 -0700271 [mock.call(mock_repo)])
Don Garrett86881cb2017-02-15 15:41:55 -0800272
273 # Ensure we invoke cbuildbot, as expected.
274 self.assertCommandCalled(
Don Garrett5cd946b2017-07-20 13:42:20 -0700275 [
276 '/root/repository/chromite/bin/cbuildbot',
277 'config',
278 '--buildroot', '/root/repository',
279 '--branch', 'branch',
280 '--git-cache-dir', '/git-cache',
Don Garrett61ce1ee2019-02-26 16:20:25 -0800281 '--cache-dir', '/cache',
Don Garrett5cd946b2017-07-20 13:42:20 -0700282 '--remote-trybot',
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600283 '--master-build-id', '123456789',
284 '--buildnumber', '314',
285 '--previous-build-state',
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600286 'eyJzdGF0dXMiOiAiZmFpbCIsICJtYXN0ZXJfYnVpbGRfaWQiOiAxMjMxMjMxMj'
287 'MsICJidWlsZF9udW1iZXIiOiAzMTMsICJidWlsZHJvb3RfbGF5b3V0IjogMiwg'
288 'ImJyYW5jaCI6ICJicmFuY2gifQ==',
Don Garrettb497f552018-07-09 16:01:13 -0700289 '--workspace', '/root/workspace',
Don Garrett61ce1ee2019-02-26 16:20:25 -0800290 '--cache-dir', '/cache',
Don Garrett5cd946b2017-07-20 13:42:20 -0700291 '--ts-mon-task-num', '1',
292 ],
Don Garretta50bf492017-09-28 18:33:02 -0700293 extra_env={'PATH': mock.ANY},
Don Garrettbf90cdf2017-05-19 15:54:02 -0700294 cwd='/root/repository',
Don Garrettacbb2392017-05-11 18:27:41 -0700295 error_code_ok=True)
Don Garrett7ade05a2017-02-17 13:31:47 -0800296
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600297 # Ensure we write the final build state, as expected.
298 final_state = build_summary.BuildSummary(
299 build_number=314,
300 master_build_id=123456789,
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600301 status=constants.BUILDER_STATUS_PASSED,
302 buildroot_layout=2,
303 branch='branch')
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600304 self.assertEqual(mock_set_last_build_state.mock_calls, [
305 mock.call('/root', final_state)])
306
Benjamin Gordonaee36b82018-02-05 14:25:26 -0700307 # Ensure we clean the chroot, as expected.
308 self.assertEqual(mock_cleanup_chroot.mock_calls, [
309 mock.call('/root/repository')])
310
Don Garrett7ade05a2017-02-17 13:31:47 -0800311
Don Garrettbf90cdf2017-05-19 15:54:02 -0700312class CleanBuildRootTest(cros_test_lib.MockTempDirTestCase):
313 """Tests for CleanBuildRoot method."""
Don Garrett7ade05a2017-02-17 13:31:47 -0800314
315 def setUp(self):
316 """Create standard buildroot contents for cleanup."""
Don Garrettbf90cdf2017-05-19 15:54:02 -0700317 self.root = os.path.join(self.tempdir)
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600318 self.previous_build_state = os.path.join(
319 self.root, '.cbuildbot_build_state.json')
Don Garrettbf90cdf2017-05-19 15:54:02 -0700320 self.buildroot = os.path.join(self.root, 'buildroot')
321 self.repo = os.path.join(self.buildroot, '.repo/repo')
Don Garrett36650112018-06-28 15:54:34 -0700322 self.chroot = os.path.join(self.buildroot, 'chroot')
Don Garrettbf90cdf2017-05-19 15:54:02 -0700323 self.general = os.path.join(self.buildroot, 'general/general')
Prathmesh Prabhuc41a0f52018-04-03 13:26:52 -0700324 self.cache = os.path.join(self.buildroot, '.cache')
325 self.distfiles = os.path.join(self.cache, 'distfiles')
Don Garrett7ade05a2017-02-17 13:31:47 -0800326
Don Garrett4166d182018-12-17 12:52:02 -0800327 self.mock_repo = mock.Mock(repository.RepoRepository)
Don Garrettbf90cdf2017-05-19 15:54:02 -0700328 self.mock_repo.directory = self.buildroot
Don Garrettf324bc32017-05-23 14:00:53 -0700329
Benjamin Gordon8642bcc2018-05-01 13:49:56 -0600330 def populateBuildroot(self, previous_build_state=None):
Don Garrett7ade05a2017-02-17 13:31:47 -0800331 """Create standard buildroot contents for cleanup."""
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600332 if previous_build_state:
333 osutils.SafeMakedirs(self.root)
334 osutils.WriteFile(self.previous_build_state, previous_build_state)
335
Don Garrett7ade05a2017-02-17 13:31:47 -0800336 # Create files.
Prathmesh Prabhuc41a0f52018-04-03 13:26:52 -0700337 for f in (self.repo, self.chroot, self.general, self.distfiles):
Don Garrette17e1d92017-04-12 15:28:19 -0700338 osutils.Touch(f, makedirs=True)
Don Garrett7ade05a2017-02-17 13:31:47 -0800339
Don Garrette17e1d92017-04-12 15:28:19 -0700340 def testNoBuildroot(self):
Don Garrettbf90cdf2017-05-19 15:54:02 -0700341 """Test CleanBuildRoot with no history."""
Don Garrettf324bc32017-05-23 14:00:53 -0700342 self.mock_repo.branch = 'master'
343
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600344 build_state = build_summary.BuildSummary(
345 status=constants.BUILDER_STATUS_INFLIGHT,
346 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
347 branch='master')
Don Garrettbf90cdf2017-05-19 15:54:02 -0700348 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800349 self.root, self.mock_repo, self.cache, build_state)
Don Garrett7ade05a2017-02-17 13:31:47 -0800350
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600351 new_summary = cbuildbot_launch.GetLastBuildState(self.root)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600352 self.assertEqual(new_summary.buildroot_layout, 2)
353 self.assertEqual(new_summary.branch, 'master')
354 self.assertIsNotNone(new_summary.distfiles_ts)
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600355 self.assertEqual(new_summary, build_state)
Don Garrett7ade05a2017-02-17 13:31:47 -0800356
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600357 self.assertExists(self.previous_build_state)
358
Don Garrett7ade05a2017-02-17 13:31:47 -0800359 def testBuildrootNoState(self):
Don Garrettbf90cdf2017-05-19 15:54:02 -0700360 """Test CleanBuildRoot with no state information."""
Don Garrett7ade05a2017-02-17 13:31:47 -0800361 self.populateBuildroot()
Don Garrettf324bc32017-05-23 14:00:53 -0700362 self.mock_repo.branch = 'master'
Don Garrett7ade05a2017-02-17 13:31:47 -0800363
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600364 build_state = build_summary.BuildSummary(
365 status=constants.BUILDER_STATUS_INFLIGHT,
366 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
367 branch='master')
Don Garrettbf90cdf2017-05-19 15:54:02 -0700368 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800369 self.root, self.mock_repo, self.cache, build_state)
Don Garrett7ade05a2017-02-17 13:31:47 -0800370
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600371 new_summary = cbuildbot_launch.GetLastBuildState(self.root)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600372 self.assertEqual(new_summary.buildroot_layout, 2)
373 self.assertEqual(new_summary.branch, 'master')
374 self.assertIsNotNone(new_summary.distfiles_ts)
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600375 self.assertEqual(new_summary, build_state)
Prathmesh Prabhuc41a0f52018-04-03 13:26:52 -0700376
Don Garrett60967922017-04-12 18:51:44 -0700377 self.assertNotExists(self.repo)
Don Garrett7ade05a2017-02-17 13:31:47 -0800378 self.assertNotExists(self.chroot)
Don Garrett60967922017-04-12 18:51:44 -0700379 self.assertNotExists(self.general)
Prathmesh Prabhuc41a0f52018-04-03 13:26:52 -0700380 self.assertNotExists(self.distfiles)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600381 self.assertExists(self.previous_build_state)
Don Garrett7ade05a2017-02-17 13:31:47 -0800382
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600383 def testBuildrootFormatMismatch(self):
384 """Test CleanBuildRoot with buildroot layout mismatch."""
385 old_build_state = build_summary.BuildSummary(
386 status=constants.BUILDER_STATUS_PASSED,
387 buildroot_layout=1,
388 branch='master')
389 self.populateBuildroot(previous_build_state=old_build_state.to_json())
390 self.mock_repo.branch = 'master'
391
392 build_state = build_summary.BuildSummary(
393 status=constants.BUILDER_STATUS_INFLIGHT,
394 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
395 branch='master')
396 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800397 self.root, self.mock_repo, self.cache, build_state)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600398
399 new_summary = cbuildbot_launch.GetLastBuildState(self.root)
400 self.assertEqual(new_summary.buildroot_layout, 2)
401 self.assertEqual(new_summary.branch, 'master')
402 self.assertIsNotNone(new_summary.distfiles_ts)
403 self.assertEqual(new_summary, build_state)
404
405 self.assertNotExists(self.repo)
406 self.assertNotExists(self.chroot)
407 self.assertNotExists(self.general)
408 self.assertNotExists(self.distfiles)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600409 self.assertExists(self.previous_build_state)
410
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600411 def testBuildrootBranchChange(self):
412 """Test CleanBuildRoot with a change in branches."""
413 old_build_state = build_summary.BuildSummary(
414 status=constants.BUILDER_STATUS_PASSED,
415 buildroot_layout=2,
416 branch='branchA')
417 self.populateBuildroot(previous_build_state=old_build_state.to_json())
418 self.mock_repo.branch = 'branchB'
Benjamin Gordon74645232018-05-04 17:40:42 -0600419 m = self.PatchObject(cros_sdk_lib, 'CleanupChrootMount')
Don Garrett7ade05a2017-02-17 13:31:47 -0800420
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600421 build_state = build_summary.BuildSummary(
422 status=constants.BUILDER_STATUS_INFLIGHT,
423 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
424 branch='branchB')
Don Garrettbf90cdf2017-05-19 15:54:02 -0700425 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800426 self.root, self.mock_repo, self.cache, build_state)
Don Garrett7ade05a2017-02-17 13:31:47 -0800427
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600428 new_summary = cbuildbot_launch.GetLastBuildState(self.root)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600429 self.assertEqual(new_summary.buildroot_layout, 2)
430 self.assertEqual(new_summary.branch, 'branchB')
431 self.assertIsNotNone(new_summary.distfiles_ts)
432 self.assertEqual(new_summary, build_state)
433
Don Garrett4166d182018-12-17 12:52:02 -0800434 # self.assertExists(self.repo)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600435 self.assertExists(self.general)
436 self.assertNotExists(self.distfiles)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600437 self.assertExists(self.previous_build_state)
Don Garrett36650112018-06-28 15:54:34 -0700438 m.assert_called_with(self.chroot, delete=True)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600439
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600440 def testBuildrootBranchMatch(self):
441 """Test CleanBuildRoot with no change in branch."""
442 old_build_state = build_summary.BuildSummary(
443 status=constants.BUILDER_STATUS_PASSED,
444 buildroot_layout=2,
445 branch='branchA')
446 self.populateBuildroot(previous_build_state=old_build_state.to_json())
447 self.mock_repo.branch = 'branchA'
448
449 build_state = build_summary.BuildSummary(
450 status=constants.BUILDER_STATUS_INFLIGHT,
451 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
452 branch='branchA')
453 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800454 self.root, self.mock_repo, self.cache, build_state)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600455
456 new_summary = cbuildbot_launch.GetLastBuildState(self.root)
457 self.assertEqual(new_summary.buildroot_layout, 2)
458 self.assertEqual(new_summary.branch, 'branchA')
459 self.assertIsNotNone(new_summary.distfiles_ts)
460 self.assertEqual(new_summary, build_state)
461
462 self.assertExists(self.repo)
463 self.assertExists(self.chroot)
464 self.assertExists(self.general)
465 self.assertExists(self.distfiles)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600466 self.assertExists(self.previous_build_state)
Prathmesh Prabhuc41a0f52018-04-03 13:26:52 -0700467
Don Garrett4166d182018-12-17 12:52:02 -0800468 def testBuildrootGitLocksPrevPass(self):
469 """Verify not CleanStaleLocks, if previous build was in passed."""
470 old_build_state = build_summary.BuildSummary(
471 status=constants.BUILDER_STATUS_PASSED,
472 buildroot_layout=2,
473 branch='branchA')
474 self.populateBuildroot(previous_build_state=old_build_state.to_json())
475 self.mock_repo.branch = 'branchA'
476
477 build_state = build_summary.BuildSummary(
478 status=constants.BUILDER_STATUS_INFLIGHT,
479 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
480 branch='branchA')
481 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800482 self.root, self.mock_repo, self.cache, build_state)
Don Garrett4166d182018-12-17 12:52:02 -0800483
484 self.assertEqual(
485 self.mock_repo.mock_calls, [
486 mock.call.PreLoad(),
487 mock.call.BuildRootGitCleanup(prune_all=True),
488 ])
489
490 def testBuildrootGitLocksPrevFail(self):
491 """Verify not CleanStaleLocks, if previous build was in failed."""
492 old_build_state = build_summary.BuildSummary(
493 status=constants.BUILDER_STATUS_FAILED,
494 buildroot_layout=2,
495 branch='branchA')
496 self.populateBuildroot(previous_build_state=old_build_state.to_json())
497 self.mock_repo.branch = 'branchA'
498
499 build_state = build_summary.BuildSummary(
500 status=constants.BUILDER_STATUS_INFLIGHT,
501 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
502 branch='branchA')
503 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800504 self.root, self.mock_repo, self.cache, build_state)
Don Garrett4166d182018-12-17 12:52:02 -0800505
506 self.assertEqual(
507 self.mock_repo.mock_calls, [
508 mock.call.PreLoad(),
509 mock.call.BuildRootGitCleanup(prune_all=True),
510 ])
511
512 def testBuildrootGitLocksPrevInFlight(self):
513 """Verify CleanStaleLocks, if previous build was in flight."""
514 old_build_state = build_summary.BuildSummary(
515 status=constants.BUILDER_STATUS_INFLIGHT,
516 buildroot_layout=2,
517 branch='branchA')
518 self.populateBuildroot(previous_build_state=old_build_state.to_json())
519 self.mock_repo.branch = 'branchA'
520
521 build_state = build_summary.BuildSummary(
522 status=constants.BUILDER_STATUS_INFLIGHT,
523 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
524 branch='branchA')
525 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800526 self.root, self.mock_repo, self.cache, build_state)
Don Garrett4166d182018-12-17 12:52:02 -0800527
528
529 self.assertEqual(
530 self.mock_repo.method_calls, [
531 mock.call.PreLoad(),
532 mock.call.CleanStaleLocks(),
533 mock.call.BuildRootGitCleanup(prune_all=True),
534 ])
535
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600536 def testBuildrootDistfilesRecentCache(self):
537 """Test CleanBuildRoot does not delete distfiles when cache is recent."""
538 seed_distfiles_ts = time.time() - 60
539 old_build_state = build_summary.BuildSummary(
540 status=constants.BUILDER_STATUS_PASSED,
541 buildroot_layout=2,
542 branch='branchA',
543 distfiles_ts=seed_distfiles_ts)
544 self.populateBuildroot(previous_build_state=old_build_state.to_json())
545 self.mock_repo.branch = 'branchA'
546
547 build_state = build_summary.BuildSummary(
548 status=constants.BUILDER_STATUS_INFLIGHT,
549 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
550 branch='branchA')
551 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800552 self.root, self.mock_repo, self.cache, build_state)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600553
554 new_summary = cbuildbot_launch.GetLastBuildState(self.root)
555 self.assertEqual(new_summary.buildroot_layout, 2)
556 self.assertEqual(new_summary.branch, 'branchA')
557 # Same cache creation timestamp is rewritten to state.
558 self.assertEqual(new_summary.distfiles_ts, seed_distfiles_ts)
559 self.assertEqual(new_summary, build_state)
560
561 self.assertExists(self.repo)
562 self.assertExists(self.chroot)
563 self.assertExists(self.general)
564 self.assertExists(self.distfiles)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600565 self.assertExists(self.previous_build_state)
Don Garrette17e1d92017-04-12 15:28:19 -0700566
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600567 def testBuildrootDistfilesCacheExpired(self):
568 """Test CleanBuildRoot when the distfiles cache is too old."""
569 old_build_state = build_summary.BuildSummary(
570 status=constants.BUILDER_STATUS_PASSED,
571 buildroot_layout=2,
572 branch='branchA',
573 distfiles_ts=100.0)
574 self.populateBuildroot(previous_build_state=old_build_state.to_json())
575 self.mock_repo.branch = 'branchA'
576
577 build_state = build_summary.BuildSummary(
578 status=constants.BUILDER_STATUS_INFLIGHT,
579 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
580 branch='branchA')
581 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800582 self.root, self.mock_repo, self.cache, build_state)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600583
584 new_summary = cbuildbot_launch.GetLastBuildState(self.root)
585 self.assertEqual(new_summary.buildroot_layout, 2)
586 self.assertEqual(new_summary.branch, 'branchA')
587 self.assertIsNotNone(new_summary.distfiles_ts)
588 self.assertEqual(new_summary, build_state)
589
590 self.assertExists(self.repo)
591 self.assertExists(self.chroot)
592 self.assertExists(self.general)
593 self.assertNotExists(self.distfiles)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600594 self.assertExists(self.previous_build_state)
595
Don Garrett61ce1ee2019-02-26 16:20:25 -0800596 def testRootOwnedCache(self):
597 """Test CleanBuildRoot with no history."""
598 seed_distfiles_ts = time.time() - 60
599 old_build_state = build_summary.BuildSummary(
600 status=constants.BUILDER_STATUS_PASSED,
601 buildroot_layout=2,
602 branch='branchA',
603 distfiles_ts=seed_distfiles_ts)
604 self.populateBuildroot(previous_build_state=old_build_state.to_json())
605 self.mock_repo.branch = 'branchA'
606
607 osutils.Chown(self.cache, 'root', 'root')
608
609 build_state = build_summary.BuildSummary(
610 status=constants.BUILDER_STATUS_INFLIGHT,
611 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
612 branch='branchA')
613 cbuildbot_launch.CleanBuildRoot(
614 self.root, self.mock_repo, self.cache, build_state)
615
616 new_summary = cbuildbot_launch.GetLastBuildState(self.root)
617 self.assertEqual(new_summary.buildroot_layout, 2)
618 self.assertEqual(new_summary.branch, 'branchA')
619 # Same cache creation timestamp is rewritten to state.
620 self.assertEqual(new_summary.distfiles_ts, seed_distfiles_ts)
621 self.assertEqual(new_summary, build_state)
622
623 self.assertExists(self.repo)
624 self.assertExists(self.chroot)
625 self.assertExists(self.general)
626 self.assertNotExists(self.distfiles)
627 self.assertExists(self.previous_build_state)
628
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600629 def testBuildrootRepoCleanFailure(self):
630 """Test CleanBuildRoot with repo checkout failure."""
631 old_build_state = build_summary.BuildSummary(
632 status=constants.BUILDER_STATUS_PASSED,
633 buildroot_layout=1,
634 branch='branchA')
635 self.populateBuildroot(previous_build_state=old_build_state.to_json())
636 self.mock_repo.branch = 'branchA'
637 self.mock_repo.BuildRootGitCleanup.side_effect = Exception
638
639 build_state = build_summary.BuildSummary(
640 status=constants.BUILDER_STATUS_INFLIGHT,
641 buildroot_layout=cbuildbot_launch.BUILDROOT_BUILDROOT_LAYOUT,
642 branch='branchA')
643 cbuildbot_launch.CleanBuildRoot(
Don Garrett61ce1ee2019-02-26 16:20:25 -0800644 self.root, self.mock_repo, self.cache, build_state)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600645
646 new_summary = cbuildbot_launch.GetLastBuildState(self.root)
647 self.assertEqual(new_summary.buildroot_layout, 2)
648 self.assertEqual(new_summary.branch, 'branchA')
649 self.assertIsNotNone(new_summary.distfiles_ts)
650 self.assertEqual(new_summary, build_state)
651
652 self.assertNotExists(self.repo)
653 self.assertNotExists(self.chroot)
654 self.assertNotExists(self.general)
655 self.assertNotExists(self.distfiles)
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600656 self.assertExists(self.previous_build_state)
Don Garrettf324bc32017-05-23 14:00:53 -0700657
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600658 def testGetCurrentBuildStateNoArgs(self):
659 """Tests GetCurrentBuildState without arguments."""
660 options = cbuildbot_launch.PreParseArguments([
661 '--buildroot', self.root, 'config'
662 ])
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600663 state = cbuildbot_launch.GetCurrentBuildState(options, 'master')
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600664
665 expected_state = build_summary.BuildSummary(
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600666 status=constants.BUILDER_STATUS_INFLIGHT,
667 buildroot_layout=2,
668 branch='master')
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600669 self.assertEqual(state, expected_state)
670
671 def testGetCurrentBuildStateHasArgs(self):
672 """Tests GetCurrentBuildState with arguments."""
673 options = cbuildbot_launch.PreParseArguments([
674 '--buildroot', self.root,
675 '--buildnumber', '20',
676 '--master-build-id', '50',
677 'config'
678 ])
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600679 state = cbuildbot_launch.GetCurrentBuildState(options, 'branchA')
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600680
681 expected_state = build_summary.BuildSummary(
682 build_number=20,
683 master_build_id=50,
Benjamin Gordon8b6d4122018-04-26 13:38:39 -0600684 status=constants.BUILDER_STATUS_INFLIGHT,
685 buildroot_layout=2,
686 branch='branchA')
687 self.assertEqual(state, expected_state)
688
689 def testGetCurrentBuildStateLayout(self):
690 """Test that GetCurrentBuildState uses the current buildroot layout."""
691 # Change to a future version.
692 self.PatchObject(cbuildbot_launch, 'BUILDROOT_BUILDROOT_LAYOUT', 22)
693
694 options = cbuildbot_launch.PreParseArguments([
695 '--buildroot', self.root, 'config'
696 ])
697 state = cbuildbot_launch.GetCurrentBuildState(options, 'branchA')
698
699 expected_state = build_summary.BuildSummary(
700 status=constants.BUILDER_STATUS_INFLIGHT,
701 buildroot_layout=22,
702 branch='branchA')
Benjamin Gordon90b2dd92018-04-12 14:04:21 -0600703 self.assertEqual(state, expected_state)
704
705 def testGetLastBuildStateNoFile(self):
706 """Tests GetLastBuildState if the file is missing."""
707 osutils.SafeMakedirs(self.root)
708 state = cbuildbot_launch.GetLastBuildState(self.root)
709 self.assertEqual(state, build_summary.BuildSummary())
710
711 def testGetLastBuildStateBadFile(self):
712 """Tests GetLastBuildState if the file contains invalid JSON."""
713 osutils.SafeMakedirs(self.root)
714 osutils.WriteFile(self.previous_build_state, '}}')
715 state = cbuildbot_launch.GetLastBuildState(self.root)
716 self.assertEqual(state, build_summary.BuildSummary())
717
718 def testGetLastBuildStateMissingBuildStatus(self):
719 """Tests GetLastBuildState if the file doesn't have a valid status."""
720 osutils.SafeMakedirs(self.root)
721 osutils.WriteFile(self.previous_build_state, '{"build_number": "3"}')
722 state = cbuildbot_launch.GetLastBuildState(self.root)
723 self.assertEqual(state, build_summary.BuildSummary())
724
725 def testGetLastBuildStateGoodFile(self):
726 """Tests GetLastBuildState on a good file."""
727 osutils.SafeMakedirs(self.root)
728 osutils.WriteFile(
729 self.previous_build_state,
730 '{"build_number": 1, "master_build_id": 3, "status": "pass"}')
731 state = cbuildbot_launch.GetLastBuildState(self.root)
732 self.assertEqual(
733 state,
734 build_summary.BuildSummary(
735 build_number=1, master_build_id=3, status='pass'))
736
737 def testSetLastBuildState(self):
738 """Verifies that SetLastBuildState writes to the expected file."""
739 osutils.SafeMakedirs(self.root)
740 old_state = build_summary.BuildSummary(
741 build_number=314,
742 master_build_id=2178,
743 status=constants.BUILDER_STATUS_PASSED)
744 cbuildbot_launch.SetLastBuildState(self.root, old_state)
745
746 saved_state = osutils.ReadFile(self.previous_build_state)
747 new_state = build_summary.BuildSummary()
748 new_state.from_json(saved_state)
749
750 self.assertEqual(old_state, new_state)