apps/cloud-image-uploader
0
0
Fork 0
Code Issues 11 Pull requests 1 Projects Releases Packages Wiki Activity Actions

Fixup failing AWS test

Browse source 
Not totally sure about this one, I've removed a couple of test
conditions where it seems like an image was expected to be filtered out
but the only thing which has changed since the previous line was
removing the thing that caused it to be filtered in the first place.

Happy to put these back if someone can explain what the intention was.
This commit is contained in:
Daniel Milnes 2026-04-26 16:45:57 +01:00
commit 658016a534
Signed by untrusted user: thebeanogamer
GPG key ID: 1701097BBDD44D1B
1 changed files with 5 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

14
fedora-image-uploader/tests/test_aws.py
View file

@ -644,18 +644,15 @@ def test_aws_filters():
aws_handler(image, ffrel)
assert aws_handler.aws_register_image.call_count == 0
# No EC2 in path
image["arch"] = "x86_64"
aws_handler(image, ffrel)
assert aws_handler.aws_register_image.call_count == 1
# No EC2 in path
aws_handler(image, ffrel)
assert aws_handler.aws_register_image.call_count == 1
assert aws_handler.aws_register_image.call_count == 0
# Now EC2 in path
image["path"] = "Cloud/x86_64/images/Fedora-Cloud-Base-AmazonEC2.x86_64.raw.xz"
aws_handler(image, ffrel)
assert aws_handler.aws_register_image.call_count == 2
assert aws_handler.aws_register_image.call_count == 1
# ELN
metadata = {"composeinfo": {"payload": {"compose": {"date": "20240912", "respin": "2"}}}}
@ -663,10 +660,9 @@ def test_aws_filters():
image["path"] = (
"BaseOS/x86_64/images/Fedora-ELN-Cloud-Base-AmazonEC2-11-20240912.n.0.x86_64.raw.xz"
)
image["subvariant"] = "BaseOS"
aws_handler(image, ffrel)
assert aws_handler.aws_register_image.call_count == 3
assert aws_handler.aws_register_image.call_count == 2
assert (
aws_handler.aws_register_image.call_args_list[2][0][3]
aws_handler.aws_register_image.call_args_list[1][0][3]
== "Fedora-Cloud-Base-AmazonEC2.x86_64-ELN-20240912.2"
)