robojerk/tumbi-assembler
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

extra_isos: Provide arch to extra files getter

Browse source 
The getter is already running once per architecture, it just doesn't
make the information available to the scm wrapper.

Signed-off-by: Lubomír Sedlář <lsedlar@redhat.com>
This commit is contained in:
Lubomír Sedlář 2025-01-17 11:17:45 +01:00 • committed by lsedlar
parent e4d1bd4783
commit 46d6c48e0a
2 changed files with 19 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
pungi/phases/extra_isos.py
View file

@ -349,7 +349,7 @@ def get_extra_files(compose, variant, arch, extra_files):
target_path = os.path.join(extra_files_dir, target).rstrip("/")
filelist.extend(
os.path.join(target, f)
for f in getter(scm_dict, target_path, compose=compose)
for f in getter(scm_dict, target_path, compose=compose, arch=arch)
)
if filelist:

21
tests/test_extra_isos_phase.py
View file

@ -536,7 +536,14 @@ class GetExtraFilesTest(helpers.PungiTestCase):
self.assertEqual(get_dir.call_args_list, [])
self.assertEqual(
get_file.call_args_list,
[mock.call(cfg, os.path.join(self.dir, "legalese"), compose=self.compose)],
[
mock.call(
cfg,
os.path.join(self.dir, "legalese"),
compose=self.compose,
arch=self.arch,
)
],
)
self.assertEqual(
populate_md.call_args_list,
@ -565,7 +572,14 @@ class GetExtraFilesTest(helpers.PungiTestCase):
self.assertEqual(get_file.call_args_list, [])
self.assertEqual(
get_dir.call_args_list,
[mock.call(cfg, os.path.join(self.dir, "foo"), compose=self.compose)],
[
mock.call(
cfg,
os.path.join(self.dir, "foo"),
compose=self.compose,
arch=self.arch,
)
],
)
self.assertEqual(
populate_md.call_args_list,
@ -600,8 +614,9 @@ class GetExtraFilesTest(helpers.PungiTestCase):
cfg1,
os.path.join(self.dir, "legalese"),
compose=self.compose,
arch=self.arch,
),
mock.call(cfg2, self.dir, compose=self.compose),
mock.call(cfg2, self.dir, compose=self.compose, arch=self.arch),
],
)
self.assertEqual(