Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Cypress/E2E: Fix reporting and save environment details to dashboard #7888

Merged
merged 1 commit into from
Apr 14, 2021
Merged

Cypress/E2E: Fix reporting and save environment details to dashboard #7888

merged 1 commit into from
Apr 14, 2021

Conversation

saturninoabril
Copy link
Member

Summary

  • Fix reporting and save environment details to dashboard (see latest test run in automation channel)
  • Save environment details to dashboard (see attached screenshot)

Others:

  • replace dashboard logic from old dashboard (using reportportal) to the new one.

Ticket Link

none, getting undefined in daily Cypress run instead of the test environment data

Screenshots

Screen Shot 2021-04-12 at 10 38 35 PM

@saturninoabril saturninoabril added the 3: QA Review Requires review by a QA tester label Apr 12, 2021
josephbaylon
josephbaylon approved these changes Apr 12, 2021
View reviewed changes
Copy link
Contributor

@josephbaylon josephbaylon left a comment

Choose a reason for hiding this comment

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

LGTM. Report now showing details

furqanmlk
furqanmlk approved these changes Apr 13, 2021
View reviewed changes
Copy link
Contributor

@furqanmlk furqanmlk left a comment

Choose a reason for hiding this comment

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

LGTM

@josephbaylon josephbaylon added 4: Reviews Complete All reviewers have approved the pull request and removed 3: QA Review Requires review by a QA tester labels Apr 14, 2021
@josephbaylon josephbaylon merged commit f8cb782 into mattermost:master Apr 14, 2021
@saturninoabril saturninoabril deleted the fix-cypress-report branch April 14, 2021 00:01
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Apr 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@josephbaylon josephbaylon josephbaylon approved these changes

@furqanmlk furqanmlk furqanmlk approved these changes

@prapti prapti Awaiting requested review from prapti

Assignees
No one assigned
Labels
4: Reviews Complete All reviewers have approved the pull request Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation
Projects
None yet
Milestone
No milestone
4 participants
@saturninoabril @josephbaylon @furqanmlk @amyblais