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

integration tests for DBus Unregister call #3484

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jstavel
Copy link
Contributor

@jstavel jstavel commented Dec 18, 2024

Card-ID: CCT-738

integration tests for DBus Unregister call

there are two tests in the PR

  • when a system is registered
  • when a system is not registered

Copy link

github-actions bot commented Dec 18, 2024

Coverage

Coverage (computed on Fedora latest) •
FileStmtsMissCoverMissing
TOTAL17432446274% 
report-only-changed-files is enabled. No files were changed during this commit :)

Tests Skipped Failures Errors Time
2416 14 💤 0 ❌ 0 🔥 30.994s ⏱️

Copy link
Contributor

@jirihnidek jirihnidek left a comment

Choose a reason for hiding this comment

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

Please fix failed test. Suggestion is in the comment.

integration-tests/test_unregister.py Outdated Show resolved Hide resolved
Copy link
Contributor

@jirihnidek jirihnidek left a comment

Choose a reason for hiding this comment

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

Please, squash commits 1, 3, 4 and add appropriate commit messages. Thanks

systemtest/tests/integration/test.sh Show resolved Hide resolved
@jstavel jstavel force-pushed the jstavel/integration-tests-for-dbus-unregister-call branch from 323a6e6 to d477a54 Compare December 20, 2024 00:16
@jstavel
Copy link
Contributor Author

jstavel commented Dec 20, 2024

After I re-run a check 'tmt @ fedora41' the check finished with PASSED. It must be a temporary problem of tmt stack rather than an error in testing stack. I will keep observing this check to get deeper into this problem.

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