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

Add new tnf command to check the test results #1864

Merged

Conversation

jmontesi
Copy link
Contributor

@jmontesi jmontesi commented Feb 8, 2024

The command compares the test results of a CNFCERT execution using the log file with those provided in a YAML template to check that they match. If not, a table showing the expected vs actual results will be printed on stdout.

To generate the test results YAML template from a log file:

  • tnf check results --generate-template

To check the test results against a reference template:

  • tnf check results --template <expected_results.yaml> --log-file <cnf-certsuite.log>

The command compares the test results of a CNFCERT execution using the
log file with those provided in a YAML template to check that they match.
If not, a table showing the expected vs actual results will be printed
on stdout.

To generate the test results YAML template from a log file:

* tnf check results --generate-template

To check the test results against a reference template:

* tnf check results --template <expected_results.yaml> --log-file
  <cnf-certsuite.log>
@dcibot
Copy link
Collaborator

dcibot commented Feb 8, 2024

@dcibot
Copy link
Collaborator

dcibot commented Feb 12, 2024

@jmontesi jmontesi requested a review from greyerof February 13, 2024 08:45
cmd/tnf/check/results/results.go Show resolved Hide resolved
@dcibot
Copy link
Collaborator

dcibot commented Feb 13, 2024

@jmontesi jmontesi merged commit 9fb8550 into redhat-best-practices-for-k8s:main Feb 13, 2024
19 checks passed
@dcibot
Copy link
Collaborator

dcibot commented Feb 13, 2024

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.

5 participants