Mike Frysinger | e58c0e2 | 2017-10-04 15:43:30 -0400 | [diff] [blame] | 1 | # -*- coding: utf-8 -*- |
Aviv Keshet | b1238c3 | 2013-04-01 11:42:13 -0700 | [diff] [blame] | 2 | # Copyright (c) 2013 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 autotest_quickmerge.""" |
| 7 | |
Mike Frysinger | 383367e | 2014-09-16 15:06:17 -0400 | [diff] [blame] | 8 | from __future__ import print_function |
| 9 | |
Mike Frysinger | 6db648e | 2018-07-24 19:57:58 -0400 | [diff] [blame] | 10 | import mock |
| 11 | |
Aviv Keshet | b1238c3 | 2013-04-01 11:42:13 -0700 | [diff] [blame] | 12 | from chromite.lib import cros_test_lib |
| 13 | from chromite.scripts import autotest_quickmerge |
| 14 | |
Aviv Keshet | 787ffcd | 2013-04-08 15:14:56 -0700 | [diff] [blame] | 15 | |
| 16 | RSYNC_TEST_OUTPUT = """.d..t...... ./ |
| 17 | >f..t...... touched file with spaces |
| 18 | >f..t...... touched_file |
| 19 | >f.st...... modified_contents_file |
| 20 | .f...p..... modified_permissions_file |
| 21 | .f....o.... modified_owner_file |
| 22 | >f+++++++++ new_file |
| 23 | cL+++++++++ new_symlink -> directory_a/new_file_in_directory |
| 24 | .d..t...... directory_a/ |
| 25 | >f+++++++++ directory_a/new_file_in_directory |
| 26 | >f..t...... directory_a/touched_file_in_directory |
| 27 | cd+++++++++ new_empty_directory/ |
| 28 | .d..t...... touched_empty_directory/""" |
| 29 | # The output format of rsync's itemized changes has a few unusual cases |
| 30 | # that are ambiguous. For instance, if the operation involved creating a |
| 31 | # symbolic link named "a -> b" to a file named "c", the rsync output would be: |
| 32 | # cL+++++++++ a -> b -> c |
| 33 | # which is indistinguishable from the output for creating a symbolic link named |
| 34 | # "a" to a file named "b -> c". |
| 35 | # Since there is no easy resolution to this ambiguity, and it seems like a case |
| 36 | # that would rarely or never be encountered in the wild, rsync quickmerge |
| 37 | # will exclude all files which contain the substring " -> " in their name. |
| 38 | |
Aviv Keshet | 75d6596 | 2013-04-17 16:15:23 -0700 | [diff] [blame] | 39 | RSYNC_TEST_OUTPUT_FOR_PACKAGE_UPDATE = \ |
| 40 | """>f..t...... client/ardvark.py |
| 41 | .d..t...... client/site_tests/ |
| 42 | >f+++++++++ client/site_tests/nothing.py |
| 43 | .d..t...... client/site_tests/factory_Leds/ |
| 44 | >f+++++++++ client/site_tests/factory_Leds/factory_Leds2.py |
| 45 | >f..tpog... client/site_tests/login_UserPolicyKeys/control |
| 46 | >f..tpog... client/site_tests/login_UserPolicyKeys/login_UserPolicyKeys.py |
| 47 | >f..t...... client/site_tests/platform_Cryptohome/platform_Cryptohome.py |
| 48 | >f..tpog... server/site_tests/security_DbusFuzzServer/control |
| 49 | >f..t.og... utils/coverage_suite.py |
| 50 | .d..t...... client/site_tests/power_Thermal/ |
| 51 | cd+++++++++ client/site_tests/power_Thermal/a/ |
| 52 | cd+++++++++ client/site_tests/power_Thermal/a/b/ |
| 53 | cd+++++++++ client/site_tests/power_Thermal/a/b/c/ |
| 54 | >f+++++++++ client/site_tests/power_Thermal/a/b/c/d.py""" |
| 55 | |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 56 | RSYNC_TEST_DESTINATION_PATH = '/foo/bar/' |
| 57 | |
| 58 | TEST_PACKAGE_CP = 'a_cute/little_puppy' |
| 59 | TEST_PACKAGE_CPV = 'a_cute/little_puppy-3.14159' |
| 60 | TEST_PACKAGE_C = 'a_cute' |
| 61 | TEST_PACKAGE_PV = 'little_puppy-3.14159' |
| 62 | TEST_PORTAGE_ROOT = '/bib/bob/' |
| 63 | TEST_PACKAGE_OLDCONTENTS = { |
Mike Frysinger | e65f375 | 2014-12-08 00:46:39 -0500 | [diff] [blame] | 64 | u'/by/the/prickling/of/my/thumbs': (u'obj', '1234', '4321'), |
| 65 | u'/something/wicked/this/way/comes': (u'dir',) |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 66 | } |
| 67 | |
Mike Frysinger | e65f375 | 2014-12-08 00:46:39 -0500 | [diff] [blame] | 68 | |
Mike Frysinger | cc851fc | 2014-12-08 11:31:59 -0500 | [diff] [blame] | 69 | class ItemizeChangesFromRsyncOutput(cros_test_lib.TestCase): |
Don Garrett | 25f309a | 2014-03-19 14:02:12 -0700 | [diff] [blame] | 70 | """Test autotest_quickmerge.ItemizeChangesFromRsyncOutput.""" |
Aviv Keshet | 787ffcd | 2013-04-08 15:14:56 -0700 | [diff] [blame] | 71 | |
| 72 | def testItemizeChangesFromRsyncOutput(self): |
| 73 | """Test that rsync output parser returns correct FileMutations.""" |
Aviv Keshet | 787ffcd | 2013-04-08 15:14:56 -0700 | [diff] [blame] | 74 | expected_new = set( |
| 75 | [('>f+++++++++', '/foo/bar/new_file'), |
| 76 | ('>f+++++++++', '/foo/bar/directory_a/new_file_in_directory'), |
| 77 | ('cL+++++++++', '/foo/bar/new_symlink')]) |
| 78 | |
| 79 | expected_mod = set( |
| 80 | [('>f..t......', '/foo/bar/touched file with spaces'), |
| 81 | ('>f..t......', '/foo/bar/touched_file'), |
| 82 | ('>f.st......', '/foo/bar/modified_contents_file'), |
| 83 | ('.f...p.....', '/foo/bar/modified_permissions_file'), |
| 84 | ('.f....o....', '/foo/bar/modified_owner_file'), |
| 85 | ('>f..t......', '/foo/bar/directory_a/touched_file_in_directory')]) |
| 86 | |
| 87 | expected_dir = set([('cd+++++++++', '/foo/bar/new_empty_directory/')]) |
| 88 | |
| 89 | report = autotest_quickmerge.ItemizeChangesFromRsyncOutput( |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 90 | RSYNC_TEST_OUTPUT, RSYNC_TEST_DESTINATION_PATH) |
Aviv Keshet | 787ffcd | 2013-04-08 15:14:56 -0700 | [diff] [blame] | 91 | |
| 92 | self.assertEqual(expected_new, set(report.new_files)) |
| 93 | self.assertEqual(expected_mod, set(report.modified_files)) |
| 94 | self.assertEqual(expected_dir, set(report.new_directories)) |
| 95 | |
| 96 | |
Mike Frysinger | cc851fc | 2014-12-08 11:31:59 -0500 | [diff] [blame] | 97 | class PackageNameParsingTest(cros_test_lib.TestCase): |
Don Garrett | 25f309a | 2014-03-19 14:02:12 -0700 | [diff] [blame] | 98 | """Test autotest_quickmerge.GetStalePackageNames.""" |
Aviv Keshet | 75d6596 | 2013-04-17 16:15:23 -0700 | [diff] [blame] | 99 | |
| 100 | def testGetStalePackageNames(self): |
| 101 | autotest_sysroot = '/an/arbitrary/path/' |
| 102 | change_report = autotest_quickmerge.ItemizeChangesFromRsyncOutput( |
| 103 | RSYNC_TEST_OUTPUT_FOR_PACKAGE_UPDATE, autotest_sysroot) |
| 104 | package_matches = autotest_quickmerge.GetStalePackageNames( |
| 105 | change_report.modified_files + change_report.new_files, |
| 106 | autotest_sysroot) |
| 107 | expected_set = set(['factory_Leds', 'login_UserPolicyKeys', |
| 108 | 'platform_Cryptohome', 'power_Thermal']) |
| 109 | self.assertEqual(set(package_matches), expected_set) |
| 110 | |
| 111 | |
Benjamin Gordon | 121a2aa | 2018-05-04 16:24:45 -0600 | [diff] [blame] | 112 | class RsyncCommandTest(cros_test_lib.RunCommandTestCase): |
Don Garrett | 25f309a | 2014-03-19 14:02:12 -0700 | [diff] [blame] | 113 | """Test autotest_quickmerge.RsyncQuickmerge.""" |
Aviv Keshet | b1238c3 | 2013-04-01 11:42:13 -0700 | [diff] [blame] | 114 | |
| 115 | def testRsyncQuickmergeCommand(self): |
Mike Frysinger | 45602c7 | 2019-09-22 02:15:11 -0400 | [diff] [blame] | 116 | """Test that RsyncQuickMerge makes correct call to sudo_run""" |
Aviv Keshet | b1238c3 | 2013-04-01 11:42:13 -0700 | [diff] [blame] | 117 | include_file_name = 'an_include_file_name' |
| 118 | source_path = 'a_source_path' |
| 119 | sysroot_path = 'a_sysroot_path' |
| 120 | |
| 121 | expected_command = ['rsync', '-a', '-n', '-u', '-i', |
| 122 | '--exclude=**.pyc', '--exclude=**.pyo', |
Aviv Keshet | 787ffcd | 2013-04-08 15:14:56 -0700 | [diff] [blame] | 123 | '--exclude=** -> *', |
Aviv Keshet | b1238c3 | 2013-04-01 11:42:13 -0700 | [diff] [blame] | 124 | '--include-from=%s' % include_file_name, |
| 125 | '--exclude=*', |
| 126 | source_path, |
| 127 | sysroot_path] |
| 128 | |
| 129 | autotest_quickmerge.RsyncQuickmerge(source_path, sysroot_path, |
| 130 | include_file_name, |
| 131 | pretend=True, |
Aviv Keshet | 60968ec | 2013-04-11 18:44:14 -0700 | [diff] [blame] | 132 | overwrite=False) |
Aviv Keshet | b1238c3 | 2013-04-01 11:42:13 -0700 | [diff] [blame] | 133 | |
| 134 | self.assertCommandContains(expected_command) |
| 135 | |
| 136 | |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame] | 137 | class PortageManipulationsTest(cros_test_lib.MockTestCase): |
Don Garrett | 25f309a | 2014-03-19 14:02:12 -0700 | [diff] [blame] | 138 | """Test usage of autotest_quickmerge.portage.""" |
| 139 | |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 140 | def testUpdatePackageContents(self): |
| 141 | """Test that UpdatePackageContents makes the correct calls to portage.""" |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame] | 142 | autotest_quickmerge.portage = mock.MagicMock() |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 143 | portage = autotest_quickmerge.portage |
| 144 | |
| 145 | portage.root = TEST_PORTAGE_ROOT |
| 146 | |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame] | 147 | mock_vartree = mock.MagicMock() |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 148 | mock_vartree.settings = {'an arbitrary' : 'dictionary'} |
| 149 | mock_tree = {TEST_PORTAGE_ROOT : {'vartree' : mock_vartree}} |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame] | 150 | portage.create_trees.return_value = mock_tree |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 151 | |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame] | 152 | mock_vartree.dbapi = mock.MagicMock() |
| 153 | mock_vartree.dbapi.cp_list.return_value = [TEST_PACKAGE_CPV] |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 154 | |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame] | 155 | mock_package = mock.MagicMock() |
| 156 | portage.dblink.return_value = mock_package # pylint: disable=no-member |
| 157 | mock_package.getcontents.return_value = TEST_PACKAGE_OLDCONTENTS |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 158 | |
| 159 | EXPECTED_NEW_ENTRIES = { |
| 160 | '/foo/bar/new_empty_directory': (u'dir',), |
| 161 | '/foo/bar/directory_a/new_file_in_directory': (u'obj', '0', '0'), |
| 162 | '/foo/bar/new_file': (u'obj', '0', '0'), |
| 163 | '/foo/bar/new_symlink': (u'obj', '0', '0') |
| 164 | } |
| 165 | RESULT_DICIONARY = TEST_PACKAGE_OLDCONTENTS.copy() |
| 166 | RESULT_DICIONARY.update(EXPECTED_NEW_ENTRIES) |
| 167 | |
| 168 | mock_vartree.dbapi.writeContentsToContentsFile(mock_package, |
Mike Frysinger | e65f375 | 2014-12-08 00:46:39 -0500 | [diff] [blame] | 169 | RESULT_DICIONARY) |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 170 | |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 171 | change_report = autotest_quickmerge.ItemizeChangesFromRsyncOutput( |
Mike Frysinger | e65f375 | 2014-12-08 00:46:39 -0500 | [diff] [blame] | 172 | RSYNC_TEST_OUTPUT, RSYNC_TEST_DESTINATION_PATH) |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 173 | autotest_quickmerge.UpdatePackageContents(change_report, TEST_PACKAGE_CP, |
Mike Frysinger | e65f375 | 2014-12-08 00:46:39 -0500 | [diff] [blame] | 174 | TEST_PORTAGE_ROOT) |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 175 | |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 176 | |
Mike Frysinger | cc851fc | 2014-12-08 11:31:59 -0500 | [diff] [blame] | 177 | class PortageAPITest(cros_test_lib.TestCase): |
Aviv Keshet | b60fb3a | 2013-10-10 13:46:55 -0700 | [diff] [blame] | 178 | """Ensures that required portage API exists.""" |
Mike Frysinger | e65f375 | 2014-12-08 00:46:39 -0500 | [diff] [blame] | 179 | |
Aviv Keshet | b60fb3a | 2013-10-10 13:46:55 -0700 | [diff] [blame] | 180 | def runTest(self): |
| 181 | try: |
| 182 | import portage |
| 183 | except ImportError: |
| 184 | self.skipTest('Portage not available in test environment. Re-run test ' |
| 185 | 'in chroot.') |
| 186 | try: |
Mike Frysinger | e65f375 | 2014-12-08 00:46:39 -0500 | [diff] [blame] | 187 | # pylint: disable=no-member |
Aviv Keshet | b60fb3a | 2013-10-10 13:46:55 -0700 | [diff] [blame] | 188 | f = portage.vardbapi.writeContentsToContentsFile |
| 189 | except AttributeError: |
| 190 | self.fail('Required writeContentsToContentsFile function does ' |
| 191 | 'not exist.') |
Aviv Keshet | 940c17f | 2013-04-11 18:41:42 -0700 | [diff] [blame] | 192 | |
Mike Frysinger | ca28b0e | 2019-10-13 21:13:48 -0400 | [diff] [blame^] | 193 | self.assertTrue( |
| 194 | callable(f), |
| 195 | msg='Required writeContentsToContentsFile is not a function.') |