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.

Sign up for GitHub

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

Jump to bottom

Add Cypress Report Generator #37

Merged
merged 3 commits into from Mar 11, 2021
Merged

Add Cypress Report Generator #37

merged 3 commits into from Mar 11, 2021

Conversation

SagarGaniga
Copy link
Contributor

This PR adds a post-build action to generate the report for Cypress builds using the JSON created during the sync run of browserstack-cypress-cli.

karanshah-browserstack
karanshah-browserstack approved these changes Mar 1, 2021
View reviewed changes
src/main/java/com/browserstack/automate/ci/common/constants/Constants.java Outdated
public static final String BROWSERSTACK_REPORT_PIPELINE_FUNCTION = "browserStackReportPublisher";
public static final String BROWSERSTACK_CYPRESS_REPORT_PIPELINE_FUNCTION = "browserStackReportPublisher";
Copy link

Choose a reason for hiding this comment

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

Unused variable

@rchougule
Copy link
Contributor

Also, please add screenshots if possible of the pages/sections being added

@SagarGaniga
Copy link
Contributor Author

SagarGaniga commented Mar 8, 2021 •
edited

Few Screenshots:

  1. Results page
    image

  2. Summary Page
    image

  3. Error Page
    image

  4. Post Build Action Selection
    image

rchougule
rchougule approved these changes Mar 8, 2021
View reviewed changes
Copy link
Contributor

@rchougule rchougule left a comment

Choose a reason for hiding this comment

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

LGTM

@francisf francisf merged commit 8b97c2a into jenkinsci:master Mar 11, 2021
@SagarGaniga SagarGaniga mentioned this pull request Mar 12, 2021
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rchougule rchougule rchougule approved these changes

@karanshah-browserstack karanshah-browserstack karanshah-browserstack approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@SagarGaniga @rchougule @karanshah-browserstack @francisf