Skip to content

test: component versions report in Cypress HTML report#3274

Draft
kithwa wants to merge 2 commits intomainfrom
component-versions-report
Draft

test: component versions report in Cypress HTML report#3274
kithwa wants to merge 2 commits intomainfrom
component-versions-report

Conversation

@kithwa
Copy link
Copy Markdown
Contributor

@kithwa kithwa commented Apr 21, 2026

PR Checklist

  • Unit Tests have been added for new changes
  • API tests have been updated if applicable
  • All commented code has been removed
  • If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

  • add versions of DMT components in the custom report

Anything the reviewer should know when reviewing this PR?

If the there are associated PRs in other repositories, please link them here (i.e. device-management-toolkit/repo#365 )

Comment thread cypress/support/version-fetcher.js Dismissed
Comment thread cypress/support/version-fetcher.js Dismissed
@kithwa kithwa force-pushed the component-versions-report branch from 0b49063 to 8e2e48b Compare April 21, 2026 05:56
@madhavilosetty-intel
Copy link
Copy Markdown
Contributor

@kithwa quick question on setupNodeEvents, the function updates config.env on lines 61 and 71, but doesn't return config at the end. From what I've read, Cypress only picks up changes to config when the updated object is returned; otherwise the changes get dropped for anything outside this function.

Have you seen the merged keys actually reach the test specs in CI? Just want to confirm whether this is working as intended, or if return config is missing.

Also, would it be possible to include sample-web-ui and rpc-go versions in the report as well? For sample-web-ui, I think we could read the version from package.json. What do you think?

@kithwa kithwa marked this pull request as draft May 4, 2026 22:25
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.

4 participants