feat: add --output-format=github-actions flag to test command#346
feat: add --output-format=github-actions flag to test command#346A-d-i-t-y wants to merge 1 commit into
Conversation
|
Please pass the DCO. |
dd34a22 to
be0f819
Compare
|
Hi @Vaishnav88sk, thank you for the review! I have signed off all commits to pass the DCO check. Could you please verify it and let me know if there are any further changes required? I am happy to work on any feedback you have. |
|
This look like a total rewrite and could be hard to review. If you are using an AI, it should only make the propose fix and not full change |
|
Hi @Caesarsage, thank you for the feedback! You are right, the changeset is larger than needed. The core change is only:
The rest of microcks_client.go was unintentionally included. I will clean up the PR to show only the minimal necessary diff. Sorry for the noise! |
Add --output-format flag supporting: - plain (default): existing behavior unchanged - github-actions: emits ::error/::notice workflow commands for PR annotations Closes microcks#332 Signed-off-by: Aditya <aaaditya1909@gmail.com>
be0f819 to
872cdd6
Compare
|
Hi @Caesarsage @Harsh4902, I have cleaned up the PR. Now it only contains the minimal necessary changes:
Please let me know if any further changes are needed! |
Summary
Adds
--output-formatflag to thetestcommand to improve CI/CD experience.Problem
Test failures in GitHub Actions are buried in plain text logs. Developers
must open the Microcks UI to see per-operation results.
Solution
--output-format=plain(default) — existing behavior, no breaking change--output-format=github-actions— emits workflow commands as PR annotationsExample output in GitHub Actions
::notice title=Test Passed::Operation GET /beer PASSED
::error title=Test Failed::Operation POST /beer FAILED
Files changed
cmd/test.go— added --output-format flag with validationpkg/connectors/microcks_client.go— added TestResult, TestCaseResult,TestStepResult structs and GetTestResultDetails method
Closes #332