feat(test): add --output (text/json/yaml/github-actions) for machine-readable results - #488
Merged
Merged
Conversation
Contributor
Author
|
/hold Working on a structure logger and helper function from last mentorship call |
Contributor
Author
Harsh4902
previously approved these changes
Jul 30, 2026
Harsh4902
left a comment
Member
There was a problem hiding this comment.
New dependancies are in place. GTG
Harsh4902
self-requested a review
July 30, 2026 16:26
|
I think the review comments are still not resolved. After that 👍🏻 |
Contributor
Author
Yes, once the other PRs are merged, I will rebase and rework attending to your reviews. this is so as to avoid many conflicts. As stated here. |
Caesarsage
force-pushed
the
feat/test-output
branch
from
August 12, 2026 10:40
f5cd644 to
5347cc3
Compare
Vaishnav88sk
previously requested changes
Aug 12, 2026
Contributor
Author
|
Thank you for the review. I have updated it @Vaishnav88sk |
Caesarsage
force-pushed
the
feat/test-output
branch
from
August 12, 2026 12:05
e0d2b57 to
8d4da5a
Compare
Signed-off-by: caesarsage <destinyerhabor6@gmail.com>
Caesarsage
force-pushed
the
feat/test-output
branch
from
August 12, 2026 12:06
8d4da5a to
88ea973
Compare
|
LGTM @Harsh4902 |
|
Thanks for the contribution!! @Caesarsage |
Harsh4902
dismissed
Vaishnav88sk’s stale review
August 12, 2026 15:40
Dismissing this as @Vaishnav88sk' has commented LGTM
Harsh4902
approved these changes
Aug 12, 2026
Harsh4902
left a comment
Member
There was a problem hiding this comment.
Good to go. Thanks @Caesarsage and @Vaishnav88sk
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds
--outputtomicrocks test, with four formats:text(default)jsonTestResultas JSONyamlTestResultas YAMLgithub-actionsWorks for both the normal server path and
--dry-run, since it's wired into theshared test engine (
runTestAndWait).github-actions format
::group::/::endgroup::log-folding per operation::error::per failing step, with the validation message::error file=<spec>,line=<n>::when the local spec is known (--dry-run --artifact),mapping the failing operation to its line so the annotation lands on the diff
::notice::for passing operations, gated byMICROCKS_ACTIONS_VERBOSE$GITHUB_STEP_SUMMARYTests
summary, and the OpenAPI line-mapping (
pkg/output).Notes
--output, extended with github-actions + diffannotations) — that PR can be closed.
Refs Feat: Microcks-CLI v2 - IDE (Vs code) Integration and Local Dev Loop Enhancement #255