blob: c9bdbbef5d4bf766d3710232cdbbcee9e87b536d [file] [log] [blame]
Steven Bennettsddf9bcd2017-06-14 14:07:43 -07001# Copyright 2017 The Chromium OS Authors. All rights reserved.
2# Use of this source code is governed by a BSD-style license that can be
3# found in the LICENSE file.
4
5"""Unit tests for the chrome_chromeos_lkgm program."""
6
Steven Bennettsddf9bcd2017-06-14 14:07:43 -07007import os
Ben Pastene6a8361a2021-08-19 19:52:05 -07008from unittest import mock
Steven Bennettsddf9bcd2017-06-14 14:07:43 -07009
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070010from chromite.lib import constants
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070011from chromite.lib import cros_test_lib
12from chromite.lib import osutils
13from chromite.lib import partial_mock
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070014from chromite.scripts import chrome_chromeos_lkgm
15
Achuith Bhandarkarec8d7a72018-03-01 16:56:22 -080016
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070017# pylint: disable=protected-access
Benjamin Gordon121a2aa2018-05-04 16:24:45 -060018class ChromeLKGMCommitterTester(cros_test_lib.RunCommandTestCase,
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070019 cros_test_lib.MockTempDirTestCase):
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070020 """Test cros_chromeos_lkgm.Committer."""
Achuith Bhandarkarec8d7a72018-03-01 16:56:22 -080021
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070022 def setUp(self):
23 """Common set up method for all tests."""
24 self.committer = chrome_chromeos_lkgm.ChromeLKGMCommitter(
Achuith Bhandarkarb6f40252019-12-09 16:31:34 -080025 'user@test.org', self.tempdir, '1001.0.0')
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070026 self.lkgm_file = os.path.join(self.tempdir, constants.PATH_TO_CHROME_LKGM)
27 self.old_lkgm = None
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070028
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070029 def _createOldLkgm(self, *args, **kwargs): # pylint: disable=unused-argument
Steven Bennettsd36fc782017-11-08 14:45:43 -080030 # Write out an old lkgm file as if we got it from a git fetch.
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070031 osutils.SafeMakedirs(os.path.join(self.tempdir, '.git', 'info'))
Steven Bennettsd36fc782017-11-08 14:45:43 -080032 osutils.SafeMakedirs(os.path.dirname(self.lkgm_file))
33 osutils.WriteFile(self.lkgm_file, self.old_lkgm)
34
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070035 def testCheckoutChromeLKGM(self):
Mike Frysinger3ef6d972019-08-24 20:07:42 -040036 """Tests that we can read/obtain the old LKGM from mocked out git."""
Steven Bennettsd36fc782017-11-08 14:45:43 -080037 self.old_lkgm = '1234.0.0'
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070038 self.rc.AddCmdResult(partial_mock.In('remote'), returncode=0,
Steven Bennettsd36fc782017-11-08 14:45:43 -080039 side_effect=self._createOldLkgm)
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070040 self.committer.CheckoutChrome()
Achuith Bhandarkarec8d7a72018-03-01 16:56:22 -080041
42 self.assertEqual(self.committer.lkgm_file, self.lkgm_file)
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070043 self.assertEqual(osutils.ReadFile(self.lkgm_file), self.old_lkgm)
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070044
45 def testCommitNewLKGM(self):
46 """Tests that we can commit a new LKGM file."""
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070047 self.old_lkgm = '999.0.0'
48 self.rc.AddCmdResult(partial_mock.In('remote'), returncode=0,
49 side_effect=self._createOldLkgm)
50 self.committer.CheckoutChrome()
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070051
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070052 self.assertEqual(self.committer.lkgm_file, self.lkgm_file)
53
54 self.committer.UpdateLKGM()
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070055 self.committer.CommitNewLKGM()
56
57 # Check the file was actually written out correctly.
58 self.assertEqual(osutils.ReadFile(self.lkgm_file), self.committer._lkgm)
59 self.assertCommandContains(['git', 'commit'])
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070060 self.assertEqual(self.committer._old_lkgm, self.old_lkgm)
Steven Bennettsddf9bcd2017-06-14 14:07:43 -070061
Steven Bennetts4bc322a2017-08-28 09:37:39 -070062 def testOlderLKGMFails(self):
63 """Tests that trying to update to an older lkgm version fails."""
Steven Bennettsd36fc782017-11-08 14:45:43 -080064 self.old_lkgm = '1002.0.0'
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070065 self.rc.AddCmdResult(partial_mock.In('remote'), returncode=0,
Steven Bennettsd36fc782017-11-08 14:45:43 -080066 side_effect=self._createOldLkgm)
Steven Bennetts4bc322a2017-08-28 09:37:39 -070067
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070068 self.committer.CheckoutChrome()
Steven Bennetts4bc322a2017-08-28 09:37:39 -070069
Steven Bennetts4bc322a2017-08-28 09:37:39 -070070 self.assertRaises(chrome_chromeos_lkgm.LKGMNotValid,
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070071 self.committer.UpdateLKGM)
72 self.assertEqual(self.committer._old_lkgm, self.old_lkgm)
73 self.assertEqual(self.committer._lkgm, '1001.0.0')
74 self.assertEqual(osutils.ReadFile(self.lkgm_file), '1002.0.0')
Steven Bennetts4bc322a2017-08-28 09:37:39 -070075
76 def testVersionWithChromeBranch(self):
77 """Tests passing a version with a chrome branch strips the branch."""
Achuith Bhandarkar1b9180f2018-02-22 19:12:09 -080078 self.committer = chrome_chromeos_lkgm.ChromeLKGMCommitter(
Achuith Bhandarkarb6f40252019-12-09 16:31:34 -080079 'user@test.org', self.tempdir, '1003.0.0-rc2')
Steven Bennetts4bc322a2017-08-28 09:37:39 -070080
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070081 self.old_lkgm = '1002.0.0'
82 self.rc.AddCmdResult(partial_mock.In('remote'), returncode=0,
83 side_effect=self._createOldLkgm)
84
85 self.committer.CheckoutChrome()
86 self.committer.UpdateLKGM()
Steven Bennetts4bc322a2017-08-28 09:37:39 -070087 self.committer.CommitNewLKGM()
88
89 # Check the file was actually written out correctly.
90 stripped_lkgm = '1003.0.0'
91 self.assertEqual(osutils.ReadFile(self.lkgm_file), stripped_lkgm)
Achuith Bhandarkardc3a3512018-03-16 15:10:57 -070092 self.assertEqual(self.committer._old_lkgm, self.old_lkgm)
Ben Pastened3b93d42019-10-10 09:56:29 -070093
94 def testCommitMsg(self):
95 """Tests format of the commit message."""
Ben Pastened3b93d42019-10-10 09:56:29 -070096 self.committer._PRESUBMIT_BOTS = ['bot1', 'bot2']
Ben Pastenef0fb0622021-09-24 23:38:13 +000097 self.committer._buildbucket_id = 'some-build-id'
Ben Pastened3b93d42019-10-10 09:56:29 -070098 commit_msg_lines = self.committer.ComposeCommitMsg().splitlines()
Ben Pastene6a8361a2021-08-19 19:52:05 -070099 self.assertIn('LKGM 1001.0.0 for chromeos.', commit_msg_lines)
Ben Pastenef0fb0622021-09-24 23:38:13 +0000100 self.assertIn(
101 'Uploaded by https://ci.chromium.org/b/some-build-id', commit_msg_lines)
Ben Pastened3b93d42019-10-10 09:56:29 -0700102 self.assertIn('CQ_INCLUDE_TRYBOTS=luci.chrome.try:bot1', commit_msg_lines)
103 self.assertIn('CQ_INCLUDE_TRYBOTS=luci.chrome.try:bot2', commit_msg_lines)
Ben Pastene6a8361a2021-08-19 19:52:05 -0700104
105 def testFindAlreadyOpenLKGMRoll(self):
106 already_open_issues = [123456]
107 with mock.patch.object(
108 self.committer._gerrit_helper, 'Query',
109 return_value=already_open_issues):
110 self.assertEqual(
111 self.committer.FindAlreadyOpenLKGMRoll(),
112 already_open_issues[0])
113 already_open_issues = [123456, 654321]
114 with mock.patch.object(
115 self.committer._gerrit_helper, 'Query',
116 return_value=already_open_issues):
117 self.assertRaises(
118 chrome_chromeos_lkgm.LKGMNotValid,
119 self.committer.FindAlreadyOpenLKGMRoll)
120
121 def testSubmitToCQ(self):
122 already_open_issue = 123456
123 with mock.patch.object(self.committer._gerrit_helper, 'SetReview'):
124 self.committer.SubmitToCQ(already_open_issue)