Mike Frysinger | e58c0e2 | 2017-10-04 15:43:30 -0400 | [diff] [blame] | 1 | # -*- coding: utf-8 -*- |
Steven Bennetts | ddf9bcd | 2017-06-14 14:07:43 -0700 | [diff] [blame] | 2 | # Copyright 2017 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 | |
| 6 | """Unit tests for the chrome_chromeos_lkgm program.""" |
| 7 | |
| 8 | from __future__ import print_function |
| 9 | |
| 10 | import os |
| 11 | import time |
| 12 | |
| 13 | from chromite.lib import builder_status_lib |
| 14 | from chromite.lib import constants |
| 15 | from chromite.lib import cros_build_lib_unittest |
| 16 | from chromite.lib import cros_test_lib |
| 17 | from chromite.lib import osutils |
| 18 | from chromite.lib import partial_mock |
| 19 | from chromite.lib import tree_status |
| 20 | from chromite.scripts import chrome_chromeos_lkgm |
| 21 | |
| 22 | # pylint: disable=protected-access |
| 23 | |
Steven Bennetts | 6a6d692 | 2017-09-08 14:33:38 +0000 | [diff] [blame] | 24 | class BaseChromeLGTMCommitterTest(cros_test_lib.MockTempDirTestCase): |
| 25 | """Base class for tests using cros_chromeos_lkgm.ChromeLGTMCommitter.""" |
Steven Bennetts | ddf9bcd | 2017-06-14 14:07:43 -0700 | [diff] [blame] | 26 | |
| 27 | def setUp(self): |
| 28 | """Common set up method for all tests.""" |
Steven Bennetts | 6a6d692 | 2017-09-08 14:33:38 +0000 | [diff] [blame] | 29 | self.committer = chrome_chromeos_lkgm.ChromeLGTMCommitter( |
Steven Bennetts | ac301f4 | 2017-11-07 11:01:14 -0800 | [diff] [blame] | 30 | self.tempdir, '1001.0.0', False, 'user@test.org') |
Steven Bennetts | ddf9bcd | 2017-06-14 14:07:43 -0700 | [diff] [blame] | 31 | self.lkgm_file = os.path.join(self.tempdir, constants.PATH_TO_CHROME_LKGM) |
| 32 | self.pass_status = builder_status_lib.BuilderStatus( |
| 33 | constants.BUILDER_STATUS_PASSED, None) |
| 34 | self.fail_status = builder_status_lib.BuilderStatus( |
| 35 | constants.BUILDER_STATUS_FAILED, None) |
| 36 | # No need to make tests sleep. |
| 37 | self.PatchObject(time, 'sleep') |
| 38 | |
| 39 | |
Steven Bennetts | 6a6d692 | 2017-09-08 14:33:38 +0000 | [diff] [blame] | 40 | class ChromeLGTMCommitterTester(cros_build_lib_unittest.RunCommandTestCase, |
| 41 | BaseChromeLGTMCommitterTest): |
Steven Bennetts | ddf9bcd | 2017-06-14 14:07:43 -0700 | [diff] [blame] | 42 | """Test cros_chromeos_lkgm.Committer.""" |
| 43 | |
| 44 | def testCheckoutChromeLKGM(self): |
| 45 | "Tests that we can read/obtain the old LKGM from mocked out git." |
| 46 | # Write out an old lkgm file as if we got it from a git fetch. |
| 47 | old_lkgm = '1234.0.0' |
| 48 | osutils.SafeMakedirs(os.path.dirname(self.lkgm_file)) |
| 49 | osutils.WriteFile(self.lkgm_file, old_lkgm) |
| 50 | self.committer.CheckoutChromeLKGM() |
| 51 | self.assertTrue(self.committer._old_lkgm, old_lkgm) |
| 52 | |
| 53 | def testCommitNewLKGM(self): |
| 54 | """Tests that we can commit a new LKGM file.""" |
| 55 | osutils.SafeMakedirs(os.path.dirname(self.lkgm_file)) |
Steven Bennetts | 6a6d692 | 2017-09-08 14:33:38 +0000 | [diff] [blame] | 56 | self.committer = chrome_chromeos_lkgm.ChromeLGTMCommitter( |
Steven Bennetts | ac301f4 | 2017-11-07 11:01:14 -0800 | [diff] [blame] | 57 | self.tempdir, '1002.0.0', False, 'user@test.org') |
Steven Bennetts | ddf9bcd | 2017-06-14 14:07:43 -0700 | [diff] [blame] | 58 | |
| 59 | self.PatchObject(tree_status, 'IsTreeOpen', return_value=True) |
| 60 | self.committer.CommitNewLKGM() |
| 61 | |
| 62 | # Check the file was actually written out correctly. |
| 63 | self.assertEqual(osutils.ReadFile(self.lkgm_file), self.committer._lkgm) |
| 64 | self.assertCommandContains(['git', 'commit']) |
| 65 | |
| 66 | def testLandNewLKGM(self): |
| 67 | """Tests that we try to execute git cl land if the tree is open.""" |
| 68 | self.PatchObject(tree_status, 'IsTreeOpen', return_value=True) |
| 69 | |
| 70 | self.committer.LandNewLKGM() |
| 71 | |
| 72 | self.assertCommandContains(['git', 'cl', 'land']) |
| 73 | |
| 74 | def testLandNewLKGMWithRetry(self): |
| 75 | """Tests that we try to rebase if landing fails.""" |
| 76 | self.PatchObject(tree_status, 'IsTreeOpen', return_value=True) |
| 77 | |
| 78 | self.rc.AddCmdResult(partial_mock.In('land'), returncode=1) |
| 79 | self.assertRaises(chrome_chromeos_lkgm.LKGMNotCommitted, |
| 80 | self.committer.LandNewLKGM) |
| 81 | |
| 82 | self.assertCommandContains(['git', 'cl', 'land']) |
| 83 | self.assertCommandContains(['git', 'fetch', 'origin', 'master']) |
| 84 | self.assertCommandContains(['git', 'rebase']) |
Steven Bennetts | 4bc322a | 2017-08-28 09:37:39 -0700 | [diff] [blame] | 85 | |
| 86 | def testOlderLKGMFails(self): |
| 87 | """Tests that trying to update to an older lkgm version fails.""" |
| 88 | old_lkgm = '1002.0.0' |
| 89 | osutils.SafeMakedirs(os.path.dirname(self.lkgm_file)) |
| 90 | osutils.WriteFile(self.lkgm_file, old_lkgm) |
| 91 | |
Steven Bennetts | 6a6d692 | 2017-09-08 14:33:38 +0000 | [diff] [blame] | 92 | self.committer = chrome_chromeos_lkgm.ChromeLGTMCommitter( |
Steven Bennetts | ac301f4 | 2017-11-07 11:01:14 -0800 | [diff] [blame] | 93 | self.tempdir, '1001.0.0', False, 'user@test.org') |
Steven Bennetts | 4bc322a | 2017-08-28 09:37:39 -0700 | [diff] [blame] | 94 | self.committer.CheckoutChromeLKGM() |
| 95 | self.assertTrue(self.committer._old_lkgm, old_lkgm) |
| 96 | |
| 97 | self.PatchObject(tree_status, 'IsTreeOpen', return_value=True) |
| 98 | self.assertRaises(chrome_chromeos_lkgm.LKGMNotValid, |
| 99 | self.committer.CommitNewLKGM) |
| 100 | |
| 101 | def testVersionWithChromeBranch(self): |
| 102 | """Tests passing a version with a chrome branch strips the branch.""" |
| 103 | old_lkgm = '1002.0.0' |
| 104 | osutils.SafeMakedirs(os.path.dirname(self.lkgm_file)) |
| 105 | osutils.WriteFile(self.lkgm_file, old_lkgm) |
| 106 | self.committer.CheckoutChromeLKGM() |
| 107 | self.assertTrue(self.committer._old_lkgm, old_lkgm) |
| 108 | |
Steven Bennetts | 6a6d692 | 2017-09-08 14:33:38 +0000 | [diff] [blame] | 109 | self.committer = chrome_chromeos_lkgm.ChromeLGTMCommitter( |
Steven Bennetts | ac301f4 | 2017-11-07 11:01:14 -0800 | [diff] [blame] | 110 | self.tempdir, '1003.0.0-rc2', False, 'user@test.org') |
Steven Bennetts | 4bc322a | 2017-08-28 09:37:39 -0700 | [diff] [blame] | 111 | |
| 112 | self.PatchObject(tree_status, 'IsTreeOpen', return_value=True) |
| 113 | self.committer.CommitNewLKGM() |
| 114 | |
| 115 | # Check the file was actually written out correctly. |
| 116 | stripped_lkgm = '1003.0.0' |
| 117 | self.assertEqual(osutils.ReadFile(self.lkgm_file), stripped_lkgm) |