Skip to content

Conversation

@srbarrios
Copy link
Member

@srbarrios srbarrios commented Jul 1, 2025

What does this PR change?

This PR aims to implement a skeleton to use to cover End-To-End tests for Uyuni Tools

Test coverage

  • E2E tests were added

  • DONE

Changelogs

Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository

If you don't need a changelog check, please mark this checkbox:

  • No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)

Before you merge

Check How to branch and merge properly!

@srbarrios srbarrios force-pushed the poc-e2e-tests branch 26 times, most recently from ce62a11 to eab9812 Compare July 4, 2025 08:38
@srbarrios srbarrios self-assigned this Jul 4, 2025
@srbarrios srbarrios force-pushed the poc-e2e-tests branch 3 times, most recently from 0c64a71 to 3c62dc8 Compare July 4, 2025 09:11
@srbarrios srbarrios force-pushed the poc-e2e-tests branch 5 times, most recently from c01efe7 to 79ab6f4 Compare July 4, 2025 10:09
@srbarrios srbarrios requested review from a team, aaannz and cbosdo July 4, 2025 10:19
@srbarrios srbarrios marked this pull request as ready for review July 4, 2025 10:19
@srbarrios srbarrios changed the title PoC: End-To-End tests running on a DevContainer Uyuni-Tools End-To-End test framework Jul 4, 2025
@srbarrios srbarrios changed the title Uyuni-Tools End-To-End test framework Skeleton of a Uyuni-Tools End-To-End test framework Jul 4, 2025
./...
- name: Unit tests with all tags
run: go test -tags nok8s,ptf ./...
Copy link
Member Author

Choose a reason for hiding this comment

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

To be discussed with @cbosdo.
I have no idea why the go test failed with nok8s tag.

Copy link
Contributor

Choose a reason for hiding this comment

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

If you disable them, how are we supposed to debug the issue?

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 8, 2025

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.

2 participants