Skip to content

Implemented use of 3 variables instead of a single #18

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Implemented use of 3 variables instead of a single #18

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
7c35cb2
Implemented use of 3 variables instead of a single
jonasbn Mar 9, 2020
c4b7aec
Changed into using constants for repeated string values
jonasbn May 12, 2020
33c93e9
Revert "Implemented use of 3 variables instead of a single"
jonasbn May 13, 2020
7ca2673
Fixed a recently introduced bug
jonasbn May 13, 2020
a407bac
WIP: Made first iteration on changes based on feedback from @subtleps…
jonasbn May 16, 2020
eca7f85
Updated from master, conflicts resolved
jonasbn Jun 8, 2020
5d71bfc
Single color definitions now holds precedence over the palette defini…
jonasbn Jun 8, 2020
8573394
First shot at documentation, In general the documentation needs an ov…
jonasbn Jun 8, 2020
6f170e0
Updated README.md
jonasbn Jun 9, 2020
ee765ea
Removed excess line
jonasbn Jun 9, 2020
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
46 changes: 24 additions & 22 deletions main.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,11 @@ import (
)

var (
success = color.New(color.FgGreen)
skipped = color.New(color.FgYellow)
fail = color.New(color.FgHiRed)
pass = color.New(color.FgGreen)
skip = color.New(color.FgYellow)
fail = color.New(color.FgHiRed)
)

const paletteEnv = "GOTEST_PALETTE"

func main() {
setPalette()
enableOnCI()
Expand Down Expand Up @@ -86,19 +84,19 @@ func parse(line string) {
case strings.HasPrefix(trimmed, "?"):
c = nil

// success
// succeeded
case strings.HasPrefix(trimmed, "--- PASS"):
fallthrough
case strings.HasPrefix(trimmed, "ok"):
fallthrough
case strings.HasPrefix(trimmed, "PASS"):
c = success
c = pass

// skipped
case strings.HasPrefix(trimmed, "--- SKIP"):
c = skipped
c = skip

// failure
// failed
case strings.HasPrefix(trimmed, "--- FAIL"):
fallthrough
case strings.HasPrefix(trimmed, "FAIL"):
Expand Down Expand Up @@ -127,19 +125,23 @@ func enableOnCI() {
}

func setPalette() {
v := os.Getenv(paletteEnv)
if v == "" {
return
}
vals := strings.Split(v, ",")
if len(vals) != 2 {
return
}
if c, ok := colors[vals[0]]; ok {
fail = color.New(c)
}
if c, ok := colors[vals[1]]; ok {
success = color.New(c)
envArray := [3]string{"GOTEST_SKIP_COLOR", "GOTEST_FAIL_COLOR", "GOTEST_PASS_COLOR"}

for _, e := range envArray {
v := os.Getenv(e)
if v == "" {
continue
}
if c, ok := colors[v]; ok {
switch e {
case "GOTEST_FAIL_COLOR":
fail = color.New(c)
case "GOTEST_PASS_COLOR":
pass = color.New(c)
case "GOTEST_SKIP_COLOR":
skip = color.New(c)
}
}
}
}

Expand Down