test(optimize): add metrics coverage for selected optimizations#1945
test(optimize): add metrics coverage for selected optimizations#1945mishraa-G wants to merge 2 commits intoasyncapi:masterfrom
Conversation
|
There was a problem hiding this comment.
Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.
|
/u |
|



What this PR does
This PR adds missing test coverage for the
--output=overwritepath in theoptimizecommand.While running the test suite, I noticed that the overwrite behavior wasn’t covered, even though this path updates the original AsyncAPI file. This test makes sure that behavior is exercised and helps protect against future regressions.
Why this is needed
Overwriting the original spec file is a sensitive operation, and having it untested could allow unintended changes to slip in unnoticed. Adding coverage here felt like a small but useful improvement.
What’s included
--output=overwrite --no-ttyis usedRelated
Related to #1944