Skip to content
This repository was archived by the owner on Nov 14, 2024. It is now read-only.

feat: enable pretty output #16

Merged
merged 8 commits into from
Sep 14, 2021
Merged

feat: enable pretty output #16

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
14baaca
feat: humanwriter: prefix with result.Name if present
johnstcn Sep 13, 2021
614ff14
fix: internal/checks/kube: remove duplicated resource names in output
johnstcn Sep 13, 2021
7df3d0c
feat: enable pretty output
johnstcn Sep 13, 2021
62786bd
fix: humanwriter does not print result.Details (and probably should n…
johnstcn Sep 13, 2021
28d7032
feat: separate color and ascii arguments, attach to rootCmd
johnstcn Sep 14, 2021
f4a51bf
revert: revert adding result.Name prefix to HumanWriter
johnstcn Sep 14, 2021
82cec10
consolidate kube auth info into one simple monthly payment
johnstcn Sep 14, 2021
205a138
fixup! feat: separate color and ascii arguments, attach to rootCmd
johnstcn Sep 14, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
feat: humanwriter: prefix with result.Name if present
  • Loading branch information
@johnstcn
johnstcn committed Sep 13, 2021
commit 14baaca3a38579acadf2ece746072e34651e8b03
7 changes: 6 additions & 1 deletion internal/humanwriter/human.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,11 @@ func (w *HumanResultWriter) WriteResult(result *api.CheckResult) error {
return err
}

_, err = fmt.Fprintln(w.out, prefix, result.Summary)
namePrefix := ""
if result.Name != "" {
namePrefix = fmt.Sprintf(" [%s]", result.Name)
}

_, err = fmt.Fprintf(w.out, "%s%s %s\n", prefix, namePrefix, result.Summary)
return err
}
4 changes: 2 additions & 2 deletions internal/humanwriter/human_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,13 +52,13 @@ func TestHumanWriter(t *testing.T) {
var expected string
switch mode {
case humanwriter.OutputModeEmoji:
expected = "👍 human writer check test\n" +
expected = "👍 [check-test] human writer check test\n" +
"🔔 summary\n" +
"👎 failed message\n" +
"⚠️ \n" +
"⏩ skipped check\n"
case humanwriter.OutputModeText:
expected = "PASS human writer check test\n" +
expected = "PASS [check-test] human writer check test\n" +
"INFO summary\n" +
"FAIL failed message\n" +
"WARN \n" +
Expand Down