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