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

show e2e test results #288

Merged
merged 1 commit into from
May 15, 2023

Conversation

chandrareddyp
Copy link
Contributor

@chandrareddyp chandrareddyp commented May 14, 2023

What this PR does / why we need it

This pull request is intended to present the CLI Core E2E test results on the Tanzu CLI Core E2E Tests workflow page.

This PR utilizes the existing script to display the test results. The changes made in this PR involve updating the existing script to accept a file directory as input, containing files with JSON-formatted ginkgo test suite outputs. The script reads each ginkgo output file and presents the test results.

Which issue(s) this PR fixes

Fixes #

Describe testing done for PR

  1. Tested locally
  2. Github workflows are executed and below is the output:
    image
    image

Release note


Additional information

Special notes for your reviewer

@chandrareddyp chandrareddyp requested a review from a team as a code owner May 14, 2023 15:45
@chandrareddyp chandrareddyp changed the title use ginkgo to run e2e tests show e2e test results May 15, 2023
Copy link
Contributor

@mpanchajanya mpanchajanya left a comment

Choose a reason for hiding this comment

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

lgtm! Thanks for extending the script

@chandrareddyp chandrareddyp merged commit 7475ef9 into vmware-tanzu:main May 15, 2023
vuil pushed a commit to vuil/tanzu-cli that referenced this pull request May 16, 2023
Co-authored-by: Chandra Pamuluri <cpamuluri@cpamuluri-a01.vmware.com>
@marckhouzam marckhouzam added this to the v0.90.0 milestone Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants