Yilin Yang | 8e1b24c | 2020-02-03 15:02:25 +0800 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 2 | # |
| 3 | # Copyright 2015 The Chromium OS Authors. All rights reserved. |
| 4 | # Use of this source code is governed by a BSD-style license that can be |
| 5 | # found in the LICENSE file. |
| 6 | |
| 7 | """Tests for regions.py. |
| 8 | |
| 9 | These tests ensure that all regions in regions.py are valid. The tests use |
| 10 | testdata pulled from the Chromium sources. |
| 11 | """ |
| 12 | |
| 13 | from __future__ import print_function |
| 14 | |
Yilin Yang | 7b242ae | 2020-02-03 15:19:25 +0800 | [diff] [blame] | 15 | import io |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 16 | import os |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 17 | import sys |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 18 | import unittest |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 19 | import yaml # pylint: disable=import-error |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 20 | |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 21 | # Find chromite! |
| 22 | sys.path.insert(0, os.path.join(os.path.dirname(os.path.realpath(__file__)), |
| 23 | '..', '..', '..')) |
| 24 | |
| 25 | # pylint: disable=wrong-import-position |
| 26 | from chromite.lib import cros_logging as logging |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 27 | import regions |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 28 | # pylint: enable=wrong-import-position |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 29 | |
| 30 | |
Hung-Te Lin | 8f8e0c4 | 2015-04-13 11:42:42 +0800 | [diff] [blame] | 31 | _WARN_UNKNOWN_DATA_IN_UNCONFIRMED_REGION = ( |
| 32 | 'Missing %s %r; does this new data need to be added to CrOS, or ' |
| 33 | 'does testdata need to be updated? (just a warning, since region ' |
| 34 | '%r is not a confirmed region)') |
| 35 | |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 36 | CustomLoader = yaml.SafeLoader |
| 37 | CustomLoader.add_constructor('tag:yaml.org,2002:python/tuple', |
| 38 | CustomLoader.construct_yaml_seq) |
| 39 | |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 40 | class RegionTest(unittest.TestCase): |
| 41 | """Tests for the Region class.""" |
| 42 | |
| 43 | @classmethod |
| 44 | def _ReadTestData(cls, name): |
| 45 | """Reads a YAML-formatted test data file. |
| 46 | |
| 47 | Args: |
| 48 | name: Name of file in the testdata directory. |
| 49 | |
| 50 | Returns: |
| 51 | The parsed value. |
| 52 | """ |
| 53 | with open(os.path.join(os.path.dirname(__file__), |
| 54 | 'testdata', name + '.yaml')) as f: |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 55 | return yaml.load(f, Loader=CustomLoader) |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 56 | |
| 57 | @classmethod |
| 58 | def setUpClass(cls): |
| 59 | cls.locales = cls._ReadTestData('locales') |
| 60 | cls.time_zones = cls._ReadTestData('time_zones') |
| 61 | cls.migration_map = cls._ReadTestData('migration_map') |
| 62 | cls.input_methods = cls._ReadTestData('input_methods') |
| 63 | |
| 64 | def _ResolveInputMethod(self, method): |
| 65 | """Resolves an input method using the migration map. |
| 66 | |
| 67 | Args: |
| 68 | method: An input method ID that may contain prefixes from the |
| 69 | migration map. (E.g., "m17n:ar", which contains the "m17n:" prefix.) |
| 70 | |
| 71 | Returns: |
| 72 | The input method ID after mapping any prefixes. (E.g., "m17n:ar" will |
| 73 | be mapped to "vkd_".) |
| 74 | """ |
| 75 | for k, v in self.migration_map: |
| 76 | if method.startswith(k): |
| 77 | method = v + method[len(k):] |
| 78 | return method |
| 79 | |
| 80 | def testZoneInfo(self): |
| 81 | all_regions = regions.BuildRegionsDict(include_all=True) |
| 82 | |
| 83 | # Make sure all time zones are present in /usr/share/zoneinfo. |
| 84 | all_zoneinfos = [os.path.join('/usr/share/zoneinfo', tz) |
| 85 | for r in all_regions.values() for tz in r.time_zones] |
| 86 | missing = [z for z in all_zoneinfos if not os.path.exists(z)] |
| 87 | self.assertFalse(missing, |
| 88 | ('Missing zoneinfo files; are timezones misspelled?: %r' % |
| 89 | missing)) |
| 90 | |
| 91 | def testBadLocales(self): |
Yilin Yang | 4d719b3 | 2020-02-03 15:11:08 +0800 | [diff] [blame] | 92 | self.assertRaisesRegex( |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 93 | AssertionError, "Locale 'en-us' does not match", regions.Region, |
| 94 | 'us', 'xkb:us::eng', 'America/Los_Angeles', 'en-us', 'ANSI') |
| 95 | |
| 96 | def testBadKeyboard(self): |
Yilin Yang | 4d719b3 | 2020-02-03 15:11:08 +0800 | [diff] [blame] | 97 | self.assertRaisesRegex( |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 98 | AssertionError, "Keyboard pattern 'xkb:us::' does not match", |
| 99 | regions.Region, 'us', 'xkb:us::', 'America/Los_Angeles', 'en-US', |
| 100 | 'ANSI') |
| 101 | |
| 102 | def testKeyboardRegexp(self): |
| 103 | self.assertTrue(regions.KEYBOARD_PATTERN.match('xkb:us::eng')) |
| 104 | self.assertTrue(regions.KEYBOARD_PATTERN.match('ime:ko:korean')) |
| 105 | self.assertTrue(regions.KEYBOARD_PATTERN.match('m17n:ar')) |
| 106 | self.assertFalse(regions.KEYBOARD_PATTERN.match('m17n:')) |
| 107 | self.assertFalse(regions.KEYBOARD_PATTERN.match('foo_bar')) |
| 108 | |
| 109 | def testTimeZones(self): |
| 110 | for r in regions.BuildRegionsDict(include_all=True).values(): |
| 111 | for tz in r.time_zones: |
| 112 | if tz not in self.time_zones: |
| 113 | if r.region_code in regions.REGIONS: |
| 114 | self.fail( |
Hung-Te Lin | 8f8e0c4 | 2015-04-13 11:42:42 +0800 | [diff] [blame] | 115 | 'Missing time zones: %r; does a new time zone need to be added ' |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 116 | 'to CrOS, or does testdata need to be updated?' % tz) |
| 117 | else: |
| 118 | # This is an unconfirmed region; just print a warning. |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 119 | logging.warning(_WARN_UNKNOWN_DATA_IN_UNCONFIRMED_REGION, |
| 120 | 'time zone', tz, r.region_code) |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 121 | |
| 122 | def testLocales(self): |
| 123 | missing = [] |
| 124 | for r in regions.BuildRegionsDict(include_all=True).values(): |
| 125 | for l in r.locales: |
| 126 | if l not in self.locales: |
Hung-Te Lin | 8f8e0c4 | 2015-04-13 11:42:42 +0800 | [diff] [blame] | 127 | if r.region_code in regions.REGIONS: |
| 128 | missing.append(l) |
| 129 | else: |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 130 | logging.warning(_WARN_UNKNOWN_DATA_IN_UNCONFIRMED_REGION, 'locale', |
| 131 | l, r.region_code) |
Hung-Te Lin | 8f8e0c4 | 2015-04-13 11:42:42 +0800 | [diff] [blame] | 132 | self.assertFalse(missing, |
| 133 | ('Missing locale; does testdata need to be updated?: %r' % |
| 134 | missing)) |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 135 | |
| 136 | def testInputMethods(self): |
| 137 | # Verify that every region is present in the dict. |
| 138 | for r in regions.BuildRegionsDict(include_all=True).values(): |
| 139 | for k in r.keyboards: |
| 140 | resolved_method = self._ResolveInputMethod(k) |
| 141 | # Make sure the keyboard method is present. |
Hung-Te Lin | 8f8e0c4 | 2015-04-13 11:42:42 +0800 | [diff] [blame] | 142 | if resolved_method not in self.input_methods: |
| 143 | if r.region_code in regions.REGIONS: |
| 144 | self.fail('Missing keyboard layout %r (resolved from %r)' % ( |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 145 | resolved_method, k)) |
Hung-Te Lin | 8f8e0c4 | 2015-04-13 11:42:42 +0800 | [diff] [blame] | 146 | else: |
| 147 | # This is an unconfirmed region; just print a warning. |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 148 | logging.warning(_WARN_UNKNOWN_DATA_IN_UNCONFIRMED_REGION, |
| 149 | 'keyboard', k, r.region_code) |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 150 | |
| 151 | def testFirmwareLocales(self): |
Hung-Te Lin | 98501ae | 2017-06-02 15:53:33 +0800 | [diff] [blame] | 152 | # This file is probably in src/platform2/regions |
| 153 | src_root = os.environ.get('CROS_WORKON_SRCROOT', |
| 154 | os.path.join(os.path.dirname(__file__), '..', |
| 155 | '..', '..')) |
| 156 | bmpblk_dir = os.path.join(src_root, 'src', 'platform', 'bmpblk') |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 157 | if not os.path.exists(bmpblk_dir): |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 158 | logging.warning('Skipping testFirmwareLocales, since %r is missing', |
| 159 | bmpblk_dir) |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 160 | return |
| 161 | |
| 162 | bmp_locale_dir = os.path.join(bmpblk_dir, 'strings', 'locale') |
| 163 | for r in regions.BuildRegionsDict(include_all=True).values(): |
Marco Chen | 95a5a87 | 2019-05-13 23:52:56 +0800 | [diff] [blame] | 164 | checked_paths = [] |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 165 | for l in r.locales: |
| 166 | paths = [os.path.join(bmp_locale_dir, l)] |
| 167 | if '-' in l: |
| 168 | paths.append(os.path.join(bmp_locale_dir, l.partition('-')[0])) |
Marco Chen | 95a5a87 | 2019-05-13 23:52:56 +0800 | [diff] [blame] | 169 | if any([os.path.exists(x) for x in paths]): |
| 170 | break |
| 171 | checked_paths += paths |
| 172 | else: |
| 173 | if r.region_code in regions.REGIONS: |
| 174 | self.fail( |
| 175 | 'For region %r, none of %r exists' % (r.region_code, |
| 176 | checked_paths)) |
| 177 | else: |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 178 | logging.warning('For region %r, none of %r exists; ' |
| 179 | 'just a warning since this region is not confirmed', |
| 180 | r.region_code, checked_paths) |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 181 | |
| 182 | def testYAMLOutput(self): |
Yilin Yang | 7b242ae | 2020-02-03 15:19:25 +0800 | [diff] [blame] | 183 | output = io.StringIO() |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 184 | regions.main(['--format', 'yaml'], output) |
Pin-Yen Lin | 0825b26 | 2021-01-14 15:52:55 +0800 | [diff] [blame^] | 185 | data = yaml.load(output.getvalue(), Loader=CustomLoader) |
Yilin Yang | bb9e1ec | 2020-02-03 15:11:48 +0800 | [diff] [blame] | 186 | self.assertEqual( |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 187 | {'keyboards': ['xkb:us::eng'], |
| 188 | 'keyboard_mechanical_layout': 'ANSI', |
| 189 | 'locales': ['en-US'], |
| 190 | 'region_code': 'us', |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 191 | 'description': 'United States', |
Hung-Te Lin | 2c89ccd | 2015-04-07 15:20:35 +0800 | [diff] [blame] | 192 | 'regulatory_domain': 'US', |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 193 | 'time_zones': ['America/Los_Angeles']}, |
| 194 | data['us']) |
| 195 | |
| 196 | def testFieldsDict(self): |
| 197 | # 'description' and 'notes' should be missing. |
Yilin Yang | bb9e1ec | 2020-02-03 15:11:48 +0800 | [diff] [blame] | 198 | self.assertEqual( |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 199 | {'keyboards': ['xkb:b::b'], |
| 200 | 'keyboard_mechanical_layout': 'e', |
| 201 | 'description': 'description', |
| 202 | 'locales': ['d'], |
Hung-Te Lin | 2c89ccd | 2015-04-07 15:20:35 +0800 | [diff] [blame] | 203 | 'region_code': 'aa', |
| 204 | 'regulatory_domain': 'AA', |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 205 | 'time_zones': ['c']}, |
Hung-Te Lin | 98501ae | 2017-06-02 15:53:33 +0800 | [diff] [blame] | 206 | (regions.Region('aa', 'xkb:b::b', 'c', 'd', 'e', 'description', |
| 207 | 'notes').GetFieldsDict())) |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 208 | |
| 209 | def testConsolidateRegionsDups(self): |
| 210 | """Test duplicate handling. Two identical Regions are OK.""" |
| 211 | # Make two copies of the same region. |
Hung-Te Lin | 2c89ccd | 2015-04-07 15:20:35 +0800 | [diff] [blame] | 212 | region_list = [regions.Region('aa', 'xkb:b::b', 'c', 'd', 'e') |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 213 | for _ in range(2)] |
| 214 | # It's OK. |
Yilin Yang | bb9e1ec | 2020-02-03 15:11:48 +0800 | [diff] [blame] | 215 | self.assertEqual( |
Hung-Te Lin | 2c89ccd | 2015-04-07 15:20:35 +0800 | [diff] [blame] | 216 | {'aa': region_list[0]}, regions.ConsolidateRegions(region_list)) |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 217 | |
| 218 | # Modify the second copy. |
| 219 | region_list[1].keyboards = ['f'] |
| 220 | # Not OK anymore! |
Yilin Yang | 4d719b3 | 2020-02-03 15:11:08 +0800 | [diff] [blame] | 221 | self.assertRaisesRegex( |
Hung-Te Lin | 2c89ccd | 2015-04-07 15:20:35 +0800 | [diff] [blame] | 222 | regions.RegionException, "Conflicting definitions for region 'aa':", |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 223 | regions.ConsolidateRegions, region_list) |
| 224 | |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 225 | if __name__ == '__main__': |
Hung-Te Lin | 98501ae | 2017-06-02 15:53:33 +0800 | [diff] [blame] | 226 | logging.basicConfig(format='%(message)s', level=logging.WARNING) |
Hung-Te Lin | 76c55b2 | 2015-03-31 14:47:14 +0800 | [diff] [blame] | 227 | unittest.main() |