-
Notifications
You must be signed in to change notification settings - Fork 255
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
add new test report format: HTML #1750
Conversation
@depapp Nice idea and congrats for the work! I was working on that too the last week 🚀 |
hi @axelniklasson thanks for your feedback, using Allure is a great idea. but I think we could implement it as an additional format option in a separate Pull Request,
this would be similar to other automation tools. |
hi @ludovicobesana 👋🏻 it's great to hear that you were working on this as well. |
maestro-cli/src/main/java/maestro/cli/report/HtmlTestSuiteReporter.kt
Outdated
Show resolved
Hide resolved
maestro-cli/src/main/java/maestro/cli/report/HtmlTestSuiteReporter.kt
Outdated
Show resolved
Hide resolved
maestro-cli/src/main/java/maestro/cli/report/HtmlTestSuiteReporter.kt
Outdated
Show resolved
Hide resolved
maestro-cli/src/main/java/maestro/cli/report/ReporterFactory.kt
Outdated
Show resolved
Hide resolved
Co-authored-by: Axel Niklasson <axel.r.niklasson@gmail.com>
This reverts commit c3d1feb.
hi @axelniklasson |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of nits regarding copy but apart from that it's looking great! 🙌
maestro-cli/src/main/java/maestro/cli/report/HtmlTestSuiteReporter.kt
Outdated
Show resolved
Hide resolved
maestro-cli/src/main/java/maestro/cli/report/HtmlTestSuiteReporter.kt
Outdated
Show resolved
Hide resolved
maestro-cli/src/main/java/maestro/cli/report/HtmlTestSuiteReporter.kt
Outdated
Show resolved
Hide resolved
maestro-cli/src/main/java/maestro/cli/report/HtmlTestSuiteReporter.kt
Outdated
Show resolved
Hide resolved
maestro-cli/src/main/java/maestro/cli/report/HtmlTestSuiteReporter.kt
Outdated
Show resolved
Hide resolved
…rter.kt Co-authored-by: Axel Niklasson <axel.r.niklasson@gmail.com>
Co-authored-by: Axel Niklasson <axel.r.niklasson@gmail.com>
thanks @axelniklasson 🙇🏻 |
Thanks for this contribution @depapp 🙏 Please tag me in the docs PR when you've raised it. |
Proposed Changes
My team and I - @fitriahfebriani @dhanifatima @akbarb83 - are working on creating a new test report format using HTML.
This feature was also requested in the GitHub issue and Slack channel:
Testing
To use the HTML test report:
maestro test --format html flow.yaml
Test Report
/Users/user/.maestro/tests/*/report.html
directory