4 commits

Author	SHA1	Message	Date
Michael Vogt	ed95178b80	test: add test that ensures return value of chroot.run() ... Small followup for https://github.com/osbuild/osbuild/pull/1854 which added the return value to `util.Chroot.run`. This commit now adds a (super trivial) test for this.	2024-09-11 20:19:46 +02:00
Achilleas Koutsou	83fcc8a0b1	test: compare full arg_list in chroot context test ... - Add an extra call to `/bin/false` and explicitly set the `check` argument for both `run()` calls. - Compare full call_args_list. This checks that all the options are as expected, that the `check` argument is set properly, and that the full order of all the calls is as expected, including the chroot path. Co-authored-by: Michael Vogt <michael.vogt@gmail.com>	2024-08-28 16:45:48 -07:00
Achilleas Koutsou	1093b5eeb2	util/chroot: use subprocess.run() for all commands ... For consistency, use subprocess.run() with check=True for the calls that were previously using subprocess.check_call(). Update the affected tests to match.	2024-08-28 16:45:48 -07:00
Achilleas Koutsou	73464ff119	test: add test for chroot context ... Add a test for the chroot context that mocks subprocess.run() and subprocess.check_call(). The test verifies that the functions are called the expected number of times with the expected command (first arg).	2024-08-28 16:45:48 -07:00