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

feat: "Add screenshot" button on local cuke reports #261

Merged
merged 2 commits into from
Nov 15, 2024

Conversation

ksentak
Copy link
Contributor

@ksentak ksentak commented Nov 13, 2024

This PR adds custom logic to the cucumber reporting templates to add an "Add Screenshot" button to locally generated cucumber reports.

@CLAassistant
Copy link

CLAassistant commented Nov 13, 2024

CLA assistant check
All committers have signed the CLA.

@ksentak ksentak requested a review from jbigel November 13, 2024 14:07
@rdkcmf-jenkins
Copy link

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 2 files pending identification.

  • Protex Server Path: /home/blackduck/github/firebolt-certification-suite/261/rdkcentral/firebolt-certification-suite

  • Commit: a025108

Report detail: gist'

jbigel
jbigel previously approved these changes Nov 13, 2024
@rdkcmf-jenkins
Copy link

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 2 files pending identification.

  • Protex Server Path: /home/blackduck/github/firebolt-certification-suite/261/rdkcentral/firebolt-certification-suite

  • Commit: a025108

Report detail: gist'

@mhughesacn
Copy link

Hi Keaton: I am getting code matches in generic.js as well. Please will you add a comment under the Comcast header to say that it:

Adapts material from multiple-cucumber-html-reporter which is:
 Copyright (c) 2023 Wasiq Bhamla
 Licensed under the MIT License

so that the credit is clear here too. Thank you!

@rdkcmf-jenkins
Copy link

b'## Copyright scan failure
Commit: c666e2d
Report detail: https://gist.github.com/rdkcmf-jenkins/1886f4e0a54eb64ef31b6004bdda3ffa'

@rdkcmf-jenkins
Copy link

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 2 files pending identification.

  • Protex Server Path: /home/blackduck/github/firebolt-certification-suite/261/rdkcentral/firebolt-certification-suite

  • Commit: c666e2d

Report detail: gist'

@ksentak
Copy link
Contributor Author

ksentak commented Nov 14, 2024

Hi @mhughesacn, I have added the requested comment. Please let me know if there is anything else I need to do in order for scans to pass on this pull request. Thanks in advance!

@rdkcmf-jenkins
Copy link

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 2 files pending identification.

  • Protex Server Path: /home/blackduck/github/firebolt-certification-suite/261/rdkcentral/firebolt-certification-suite

  • Commit: c666e2d

Report detail: gist'

@mhughesacn
Copy link

Thanks Keaton

@ksentak ksentak merged commit bad0505 into dev Nov 15, 2024
6 checks passed
@ksentak ksentak deleted the feat/local-report-screenshots branch November 15, 2024 13:23
jbigel pushed a commit that referenced this pull request Nov 15, 2024
# [1.4.0-dev.11](v1.4.0-dev.10...v1.4.0-dev.11) (2024-11-15)

### Features

* "Add screenshot" button on local cuke reports ([#261](#261)) ([bad0505](bad0505))
@jbigel
Copy link
Contributor

jbigel commented Nov 15, 2024

🎉 This PR is included in version 1.4.0-dev.11 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request Nov 27, 2024
# [1.4.0](v1.3.0...v1.4.0) (2024-11-27)

### Bug Fixes

* added json object missing ([#259](#259)) ([2fc89e5](2fc89e5))
* Fixing clear event listener issue ([#243](#243)) ([b06818e](b06818e))
* null scenario to v2 ([#265](#265)) ([e30bd9c](e30bd9c))
* SDK report field for module runs ([#247](#247)) ([e6ad482](e6ad482))

### Features

* "Add screenshot" button on local cuke reports ([#261](#261)) ([bad0505](bad0505))
* [FCS] - Streamline dependencies on pub sub ([#224](#224)) ([e76863a](e76863a))
* discovery entity changes for 1.4 release of sdk ([#209](#209)) ([0bd0a3d](0bd0a3d))
* Enhancement of current performance scripts/code ([#248](#248)) ([5864c57](5864c57))
* Module certification issues 2 ([#242](#242)) ([5dcdbee](5dcdbee))
* Start/stop performance metrics collection for each test case without glue code ([#236](#236)) ([d909237](d909237))
* updated utils.js for getEnvVariable fix ([#246](#246)) ([6fe233e](6fe233e))
* Updating Scenario names ([#245](#245)) ([e3d2997](e3d2997))
* XSB accessibility cleanup ([#249](#249)) ([55b0547](55b0547))
* XSB Advertising module cleanup ([#251](#251)) ([6e457d0](6e457d0))
* XSB authentication cleanup ([#255](#255)) ([f4dd871](f4dd871))
* XSB discover launch hot cleanup ([#256](#256)) ([68987c7](68987c7))
* XSB discovery cleanup ([#258](#258)) ([55f5f7e](55f5f7e))
* XSB discovery launch cold cleanup ([#260](#260)) ([34f3f62](34f3f62))
* Xsb localization cleanup ([#222](#222)) ([4d3c746](4d3c746))
@jbigel
Copy link
Contributor

jbigel commented Nov 27, 2024

🎉 This PR is included in version 1.4.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

5 participants