Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RPM PoC: Integration tests #532

Merged

Conversation

brunoapimentel
Copy link
Contributor

@brunoapimentel brunoapimentel commented May 2, 2024

This PR implements the final scenarios:

  • multiple packages
  • multiple archs
  • check the contents of the .repo file

Maintainers will complete the following section

  • Commit messages are descriptive enough
  • Code coverage from testing does not decrease and new code is covered
  • [n/a] Docs updated (if applicable)
  • [n/a] Docs links in the code are still valid (if docs were updated)

@brunoapimentel brunoapimentel marked this pull request as ready for review May 8, 2024 17:14
@brunoapimentel brunoapimentel force-pushed the rpm-integration-tests branch 2 times, most recently from a5d8fc9 to c7a7987 Compare May 8, 2024 18:49
Copy link
Member

@ben-alkov ben-alkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but IANAE

@ejegrova
Copy link
Contributor

lgtm

Copy link
Collaborator

@eskultety eskultety left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brunoapimentel looks good, nothing substantial, but see my comments whether you think it's worth pursuing.

tests/integration/test_rpm.py Outdated Show resolved Hide resolved
Signed-off-by: Bruno Pimentel <bpimente@redhat.com>
Signed-off-by: Bruno Pimentel <bpimente@redhat.com>
This test prefetches RPM content and proceeds to generate the .repo file
by using the `inject-files` command.

Since the steps for this test differ from the usual, it was necessary to
customize the test boilerplate to call `inject-files` without actually
fully building the image, since only the contents of the generated .repo
file need to be checked.

Signed-off-by: Bruno Pimentel <bpimente@redhat.com>
@brunoapimentel brunoapimentel added this pull request to the merge queue May 14, 2024
Merged via the queue into containerbuildsystem:main with commit 73c5fa4 May 14, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants