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

adding tmt #1

Open
wants to merge 42 commits into
base: main
Choose a base branch
from
Open

adding tmt #1

wants to merge 42 commits into from

Conversation

Lorquas
Copy link
Owner

@Lorquas Lorquas commented Sep 13, 2023

No description provided.

@Lorquas
Copy link
Owner Author

Lorquas commented Sep 13, 2023

/packit test

Repository owner deleted a comment from packit-as-a-service bot Sep 21, 2023
@ptoscano ptoscano force-pushed the packit branch 21 times, most recently from 976e01e to eccb06a Compare September 22, 2023 07:48
jstavel and others added 9 commits October 26, 2023 15:02
* Drop Go 1.16 from testing matrix
* Added Go 1.20 to the same matrix
* Note: The RHEL7.9 uses Go 1.19 and the latest Fedora uses 1.20
Bumps [golang.org/x/sys](https://github.com/golang/sys) from 0.0.0-20210615035016-665e8c7367d1 to 0.1.0.
- [Commits](https://github.com/golang/sys/commits/v0.1.0)

---
updated-dependencies:
- dependency-name: golang.org/x/sys
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
* Card ID: RHELPLAN-164718
* Checking of issues using shell script analysis tool
  shellcheck should be without any issue now.
* Note: current version of bash auto-complete script provided by
  urfave package looks little bit different, but it is not
  compatible with rhc for some reason.
This commits fixes the insights-connection status when it is
not connected. Now it shows false when printing in the json
format.

Signed-off-by: Alba Hita Catala <[email protected]>

with subtests.test(msg="RHC disconnect"):
proc = rhc.disconnect()
proc_stdout = proc.stdout.decode()
Copy link
Owner Author

@Lorquas Lorquas Nov 7, 2023

Choose a reason for hiding this comment

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

you assign proc_stdout but never use it?

integration-tests/test_rhc.py Outdated Show resolved Hide resolved
... it will disconnect from all Red Hat services
... it will inform a user how it goes
"""
def test_rhc_connect_disconnect(external_candlepin, test_config, rhc, subtests):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Test for activation-key?

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.

6 participants