nats-io / nsc / 12676056675

74%

Build Type

push

github

Committed by web-flow

Commit Message

Normalize CLI output / Simplify `ExecuteCmd` output handling in tests. (#696)

* [FIX] normalized use of stdout/stderr. Reports on non-output commands will now go to stdout if there is no error otherwise to stderr. Fixed commands to use fmt.Fprintf/variants to the right stream.
Fixes #580

[CI] Refactored test cases to modernize the use of `ExecuteCmd` to not use Pipes and simply provide io.Writer for test output collection. This solves spurious test failures in Windows

Signed-off-by: Alberto Ricart <alberto@synadia.com>

Run Details

183 of 229 new or added lines in 20 files covered. (79.91%)

13 existing lines in 6 files now uncovered.

13036 of 17625 relevant lines covered (73.96%)

1.75 hits per line