Mike Frysinger | d0b4385 | 2021-02-12 19:04:57 -0500 | [diff] [blame] | 1 | # Copyright 2021 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 | """Unittests for wrapper3""" |
| 6 | |
| 7 | import os |
| 8 | from pathlib import Path |
| 9 | import sys |
| 10 | from typing import List, Union |
| 11 | |
| 12 | from chromite.lib import constants |
| 13 | from chromite.lib import cros_build_lib |
| 14 | from chromite.lib import cros_test_lib |
| 15 | from chromite.lib import timeout_util |
Mike Frysinger | d0b4385 | 2021-02-12 19:04:57 -0500 | [diff] [blame] | 16 | |
| 17 | |
Mike Frysinger | d0b4385 | 2021-02-12 19:04:57 -0500 | [diff] [blame] | 18 | WRAPPER = Path(__file__).resolve().parent / 'wrapper3.py' |
| 19 | |
| 20 | |
| 21 | class FindTargetTests(cros_test_lib.TempDirTestCase): |
| 22 | """Tests for FindTarget().""" |
| 23 | |
| 24 | def setUp(self): |
Mike Frysinger | d0b4385 | 2021-02-12 19:04:57 -0500 | [diff] [blame] | 25 | # Create a skeleton chromite layout. |
| 26 | # tmpdir/ |
| 27 | # chromite/ |
| 28 | # bin/<wrapper> |
| 29 | # scripts/ |
| 30 | # api -> <real chromite>/api/ |
| 31 | # lib -> <real chromite>/lib/ |
| 32 | # utils -> <real chromite>/utils/ |
Chris McDonald | 59650c3 | 2021-07-20 15:29:28 -0600 | [diff] [blame] | 33 | # __init__.py -> <real chromite>/__init__.py |
Mike Frysinger | d0b4385 | 2021-02-12 19:04:57 -0500 | [diff] [blame] | 34 | # PRESUBMIT.cfg # Marker file for our wrapper to find chromite. |
| 35 | self.chromite_dir = self.tempdir / 'chromite' |
| 36 | self.bindir = self.chromite_dir / 'bin' |
| 37 | self.bindir.mkdir(parents=True) |
| 38 | self.scripts_dir = self.chromite_dir / 'scripts' |
| 39 | self.scripts_dir.mkdir() |
Chris McDonald | b55b703 | 2021-06-17 16:41:32 -0600 | [diff] [blame] | 40 | for subdir in ('api', 'cbuildbot', 'lib', 'third_party', 'utils'): |
Mike Frysinger | d0b4385 | 2021-02-12 19:04:57 -0500 | [diff] [blame] | 41 | (self.chromite_dir / subdir).symlink_to( |
| 42 | Path(constants.CHROMITE_DIR) / subdir) |
Chris McDonald | 59650c3 | 2021-07-20 15:29:28 -0600 | [diff] [blame] | 43 | for subfile in ('__init__.py',): |
| 44 | (self.chromite_dir / subfile).symlink_to( |
| 45 | Path(constants.CHROMITE_DIR) / subfile) |
| 46 | for subfile in ('PRESUBMIT.cfg',): |
Mike Frysinger | d0b4385 | 2021-02-12 19:04:57 -0500 | [diff] [blame] | 47 | (self.chromite_dir / subfile).touch() |
| 48 | self.wrapper = self.scripts_dir / WRAPPER.name |
| 49 | # Copy over the wrapper. We can't just symlink it because the code also |
| 50 | # walks & resolves symlinks on itself. Try hardlink at first, but if the |
| 51 | # tempdir is on a diff mount, fallback to a copy. |
| 52 | try: |
| 53 | if sys.version_info >= (3, 8): |
| 54 | self.wrapper.link_to(WRAPPER) |
| 55 | else: |
| 56 | os.link(WRAPPER, self.wrapper) |
| 57 | except OSError: |
| 58 | self.wrapper.write_bytes(WRAPPER.read_bytes()) |
| 59 | self.wrapper.chmod(0o755) |
| 60 | |
| 61 | @staticmethod |
| 62 | def insert_path(var: str, value: str): |
| 63 | """Insert |value| into the start of the environment |var|.""" |
| 64 | if var in os.environ: |
| 65 | value += f':{os.environ[var]}' |
| 66 | os.environ[var] = value |
| 67 | |
| 68 | def gen_script(self, path: Path, wrapper: Path = None): |
| 69 | """Create a script at |path|.""" |
| 70 | path.parent.mkdir(parents=True, exist_ok=True) |
| 71 | path = path.with_suffix('.py') |
| 72 | path.write_text('def main(argv):\n print("hi", argv)\n') |
| 73 | if wrapper is None: |
| 74 | wrapper = path.with_suffix('') |
| 75 | wrapper.symlink_to(self.wrapper) |
| 76 | |
| 77 | def run_script(self, argv: List[Union[Path, str]], **kwargs): |
| 78 | """Run |prog| and return the output.""" |
| 79 | # Log the directory layout to help with debugging. |
| 80 | try: |
| 81 | cros_build_lib.run(['tree', '-p', str(self.tempdir)], encoding='utf-8', |
| 82 | print_cmd=False) |
| 83 | except cros_build_lib.RunCommandError: |
| 84 | pass |
| 85 | |
| 86 | # Helper to include a small timeout in case of bugs. |
| 87 | with timeout_util.Timeout(30): |
| 88 | return cros_build_lib.run([str(x) for x in argv], capture_output=True, |
| 89 | encoding='utf-8', **kwargs) |
| 90 | |
| 91 | def _run_tests(self, prog: Path, verify=None, **kwargs): |
| 92 | """Run |prog| in the different fun ways.""" |
| 93 | if verify is None: |
Mike Frysinger | 876a8e5 | 2022-06-23 18:07:30 -0400 | [diff] [blame] | 94 | verify = lambda result: self.assertEqual('hi []\n', result.stdout) |
Mike Frysinger | d0b4385 | 2021-02-12 19:04:57 -0500 | [diff] [blame] | 95 | |
| 96 | # Execute absolute path. |
| 97 | result = self.run_script([prog], **kwargs) |
| 98 | verify(result) |
| 99 | |
| 100 | # Execute ./ relative path. |
| 101 | result = self.run_script([f'./{prog.name}'], cwd=prog.parent, **kwargs) |
| 102 | verify(result) |
| 103 | |
| 104 | # Execute ./path/ relative path. |
| 105 | result = self.run_script([f'./{prog.parent.name}/{prog.name}'], |
| 106 | cwd=prog.parent.parent, **kwargs) |
| 107 | verify(result) |
| 108 | |
| 109 | # Run via $PATH. |
| 110 | self.insert_path('PATH', str(prog.parent)) |
| 111 | result = self.run_script([prog.name], **kwargs) |
| 112 | verify(result) |
| 113 | |
| 114 | def testExternal(self): |
| 115 | """Verify use from outside of chromite/ works with main() scripts.""" |
| 116 | prog = self.tempdir / 'path' / 'prog' |
| 117 | self.gen_script(prog) |
| 118 | self._run_tests(prog) |
| 119 | |
| 120 | def testChromiteBin(self): |
| 121 | """Verify chromite/bin/ works with module in chromite/scripts/.""" |
| 122 | prog = self.bindir / 'prog' |
| 123 | self.gen_script(self.scripts_dir / prog.name, prog) |
| 124 | self._run_tests(prog) |
| 125 | |
| 126 | def testChromiteScripts(self): |
| 127 | """Verify chromite/scripts/ works with main() scripts.""" |
| 128 | prog = self.scripts_dir / 'prog' |
| 129 | self.gen_script(prog) |
| 130 | self._run_tests(prog) |
| 131 | |
| 132 | def testChromiteCustomdir(self): |
| 133 | """Verify chromite/customdir/ works with main() scripts.""" |
| 134 | prog = self.chromite_dir / 'customdir' / 'prog' |
| 135 | self.gen_script(prog) |
| 136 | self._run_tests(prog) |
| 137 | |
| 138 | def testChromiteTopdir(self): |
| 139 | """Verify chromite/ works with main() scripts.""" |
| 140 | prog = self.chromite_dir / 'prog' |
| 141 | self.gen_script(prog) |
| 142 | self._run_tests(prog) |
| 143 | |
| 144 | def testUnittests(self): |
| 145 | """Allow direct execution of unittests.""" |
| 146 | prog = self.chromite_dir / 'subdir' / 'prog_unittest' |
| 147 | prog.parent.mkdir(parents=True, exist_ok=True) |
| 148 | path = prog.with_suffix('.py') |
| 149 | path.write_text('import sys; print("hi", sys.argv[1:])\n') |
| 150 | prog.symlink_to(self.wrapper) |
| 151 | self._run_tests(prog) |
| 152 | |
| 153 | def testTests(self): |
| 154 | """Allow direct execution of tests.""" |
| 155 | prog = self.chromite_dir / 'subdir' / 'prog_unittest' |
| 156 | prog.parent.mkdir(parents=True, exist_ok=True) |
| 157 | prog.symlink_to(self.wrapper) |
| 158 | prog.with_suffix('.py').write_text( |
| 159 | 'import sys; print("hi", sys.argv[1:])\n') |
| 160 | self._run_tests(prog) |
| 161 | |
| 162 | def testWrapper(self): |
| 163 | """Fail quickly when running the wrapper directly.""" |
| 164 | verify = lambda result: self.assertEqual(result.returncode, 100) |
| 165 | self._run_tests(self.wrapper, verify=verify, check=False) |
| 166 | |
| 167 | def testMissingScript(self): |
| 168 | """Fail quickly if wrapped script is missing.""" |
| 169 | verify = lambda result: self.assertNotEqual(result.returncode, 0) |
| 170 | prog = self.bindir / 'prog' |
| 171 | prog.symlink_to(self.wrapper) |
| 172 | self._run_tests(prog, verify=verify, check=False) |
| 173 | |
| 174 | def testBrokenScript(self): |
| 175 | """Fail quickly if wrapped script is corrupt.""" |
| 176 | verify = lambda result: self.assertNotEqual(result.returncode, 0) |
| 177 | prog = self.scripts_dir / 'prog' |
| 178 | prog.symlink_to(self.wrapper) |
| 179 | # Script has syntax errors and cannot be imported. |
| 180 | prog.with_suffix('.py').write_text('}') |
| 181 | self._run_tests(prog, verify=verify, check=False) |
Mike Frysinger | d07b854 | 2021-06-21 11:57:20 -0400 | [diff] [blame] | 182 | |
| 183 | def testDashes(self): |
| 184 | """Check behavior of scripts with dashes in their names.""" |
| 185 | script = self.chromite_dir / 'scripts' / 'p_r_o_g' |
| 186 | self.gen_script(script) |
| 187 | prog = self.chromite_dir / 'bin' / 'p-r-o-g' |
| 188 | prog.symlink_to(self.wrapper) |
| 189 | self._run_tests(prog) |