From 79e57cdf0be2748451e04794ff5eb1dae5adb85f Mon Sep 17 00:00:00 2001
From: xdrm-brackets <doowap31@gmail.com>
Date: Sun, 27 Jan 2019 12:57:15 +0100
Subject: [PATCH] add minimal tests for 'internal/color'

---
 internal/color/color.go      |   4 +-
 internal/color/color_test.go | 172 +++++++++++++++++++++++++++++++++++
 2 files changed, 175 insertions(+), 1 deletion(-)
 create mode 100644 internal/color/color_test.go

diff --git a/internal/color/color.go b/internal/color/color.go
index 4c10092..8d4ad84 100644
--- a/internal/color/color.go
+++ b/internal/color/color.go
@@ -6,6 +6,8 @@ import (
 	"strconv"
 )
 
+var ErrInvalidHexSize = fmt.Errorf("expect a size of 3 or 6 (without the '#' prefix)")
+
 // T represents a color
 type T uint32
 
@@ -24,7 +26,7 @@ func FromName(t Theme, s string) (T, error) {
 // the format is 'abc' or 'abcdef'
 func FromHex(s string) (T, error) {
 	if len(s) != 3 && len(s) != 6 {
-		return 0, fmt.Errorf("expect a size of 3 or 6 (without the '#' prefix)")
+		return 0, ErrInvalidHexSize
 	}
 
 	// short version
diff --git a/internal/color/color_test.go b/internal/color/color_test.go
new file mode 100644
index 0000000..7938f91
--- /dev/null
+++ b/internal/color/color_test.go
@@ -0,0 +1,172 @@
+package color
+
+import (
+	"strconv"
+	"testing"
+)
+
+func TestFromHexSizeError(t *testing.T) {
+
+	tests := []struct {
+		Hex string
+		err error
+	}{
+
+		{"", ErrInvalidHexSize},
+		{"f", ErrInvalidHexSize},
+		{"f0", ErrInvalidHexSize},
+		{"f00", nil}, // 3 chars
+		{"ff00", ErrInvalidHexSize},
+		{"ff000", ErrInvalidHexSize},
+		{"ff0000", nil}, // 6 chars
+		{"ff00000", ErrInvalidHexSize},
+	}
+
+	for i, test := range tests {
+
+		_, err := FromHex(test.Hex)
+		if err != nil {
+			if test.err == nil {
+				t.Errorf("[%d] unexpected error <%s>", i, err)
+			} else if err != test.err {
+				t.Errorf("[%d] got error <%s> expected <%s>", i, err, test.err)
+			}
+			break
+		}
+
+		if test.err != nil {
+			t.Errorf("[%d] expected error <%s>", i, test.err)
+		}
+
+	}
+
+}
+func TestFromHexCharsetError(t *testing.T) {
+
+	tests := []struct {
+		Hex         string
+		validSyntax bool
+	}{
+
+		{"012345", true},
+		{"789abc", true},
+		{"abcdef", true},
+		{"aBcDeF", true},
+		{"bcdefg", false},
+	}
+
+	for i, test := range tests {
+
+		_, err := FromHex(test.Hex)
+		if err != nil {
+			if test.validSyntax {
+				t.Errorf("[%d] unexpected error <%s>", i, err)
+				break
+			}
+
+			if nerr, ok := err.(*strconv.NumError); ok {
+
+				if nerr.Err != strconv.ErrSyntax {
+					t.Errorf("[%d] got unexpected error <%s> expected <%s>", i, nerr.Err, strconv.ErrSyntax)
+				}
+				break
+
+			}
+
+			t.Errorf("[%d] got an unknown error <%s> expected <%s>", i, err, "strconv.NumError")
+			break
+		}
+
+		if !test.validSyntax {
+			t.Errorf("[%d] expected error", i)
+		}
+
+	}
+
+}
+func TestFromHexRGB(t *testing.T) {
+
+	tests := []struct {
+		Hex string
+		R   uint8
+		G   uint8
+		B   uint8
+	}{
+		{"000", 0, 0, 0},
+		{"000000", 0, 0, 0},
+
+		{"f00", 255, 0, 0},
+		{"ff0000", 255, 0, 0},
+
+		{"0f0", 0, 255, 0},
+		{"00ff00", 0, 255, 0},
+
+		{"00f", 0, 0, 255},
+		{"0000ff", 0, 0, 255},
+
+		{"4cb5ae", 76, 181, 174},
+	}
+
+	for i, test := range tests {
+
+		color, err := FromHex(test.Hex)
+		if err != nil {
+			t.Errorf("[%d] unexpected error <%s>", i, err)
+			break
+		}
+
+		if color.Red() != test.R || color.Green() != test.G || color.Blue() != test.B {
+			t.Errorf("[%d] invalid color rgb(%d,%d,%d) expected rgb(%d,%d,%d)", i, color.Red(), color.Green(), color.Blue(), test.R, test.G, test.B)
+		}
+
+	}
+
+}
+
+func getTheme() Theme {
+	t := make(Theme)
+	t["white"] = 0xffffff
+	t["black"] = 0x0
+	t["red"] = 0xff0000
+	t["green"] = 0x00ff00
+	t["blue"] = 0x0000ff
+	t["CaSeSeNsItIvE"] = 0x4cb5ae
+
+	return t
+}
+
+func TestFromName(t *testing.T) {
+	// fetch custom theme
+	theme := getTheme()
+
+	tests := []struct {
+		Name  string
+		Fail  bool
+		Color T
+	}{
+		{"white", false, 0xffffff},
+		{"black", false, 0x0},
+		{"red", false, 0xff0000},
+		{"green", false, 0x00ff00},
+		{"blue", false, 0x0000ff},
+
+		{"CaSeSeNsItIvE", false, 0x4cb5ae},
+		{"casesensitive", true, 0x4cb5ae},
+	}
+
+	for i, test := range tests {
+
+		color, err := FromName(theme, test.Name)
+		if err != nil {
+			if !test.Fail {
+				t.Errorf("[%d] unexpected error <%s>", i, err)
+			}
+			break
+		}
+
+		if color != test.Color {
+			t.Errorf("[%d] invalid color %x expected %x", i, color, test.Color)
+		}
+
+	}
+}