Skip to content

Custom junit properties - #85

Closed
zcsteele wants to merge 7 commits into
devicelab-dev:mainfrom
zcsteele:custom-junit-properties
Closed

Custom junit properties#85
zcsteele wants to merge 7 commits into
devicelab-dev:mainfrom
zcsteele:custom-junit-properties

Conversation

@zcsteele

Copy link
Copy Markdown
Contributor

Summary

Add custom JUnit properties to the report file output (Official Maestro documentation & example at https://docs.maestro.dev/maestro-flows/workspace-management/test-reports-and-artifacts#custom-properties)

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring (no functional changes)

Changes Made

  • Add the 'properties' definition to relevant structs ('Config', 'FlowEntry' and 'FlowDetail')
  • Add a new output generator to pkg/report/junit.go which adds additional properties elements for any custom properties provided in the flow file
  • Additional test case TestBuildJUnitXMLCustomProperties to verify the XML generated when a flow specifies custom properties

Related Issues

Fixes #84

Testing

  • Tests pass locally (make test)
  • Linting passes (make lint)
  • Added tests for new functionality
  • Tested manually with sample flows

Checklist

  • Code follows project style guidelines
  • Self-reviewed the code
  • Added/updated documentation as needed
  • No breaking changes (or documented if breaking)
  • CHANGELOG.md updated (for notable changes)

@omnarayan

Copy link
Copy Markdown
Contributor

Superseded by d6327ab, released in v1.1.25 — sorry this sat open while that landed.

Your branch is what settled the spec, and you're credited in the release notes. Thanks for pushing it, and for keeping it rebased against v1.1.21.

Release: https://github.com/devicelab-dev/maestro-runner/releases/tag/v1.1.25

@omnarayan omnarayan closed this Aug 25, 2026
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.

[FEATURE] Add support for custom properties in the JUnit report files

2 participants