From cb1025796de99be3be9ca34feb4ac6d13273bdb8 Mon Sep 17 00:00:00 2001
From: Sirisha Vadrevu <sirideals@gmail.com>
Date: Sat, 13 Jul 2019 06:44:49 +1000
Subject: [PATCH] Updated PR review fixes

---
 04_numeronym/sirigithub/main.go      | 7 +++----
 04_numeronym/sirigithub/main_test.go | 5 +++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/04_numeronym/sirigithub/main.go b/04_numeronym/sirigithub/main.go
index 47d1a3ce5..86e535aa5 100644
--- a/04_numeronym/sirigithub/main.go
+++ b/04_numeronym/sirigithub/main.go
@@ -13,11 +13,10 @@ func numeronyms(vals ...string) []string {
 	for i, val := range vals {
 		r := []rune(val)
 		runeLength := len(r)
-		if runeLength < 4 {
-			result[i] = val
-		} else {
-			result[i] = fmt.Sprintf("%c%d%c", r[0], runeLength-2, r[runeLength-1])
+		if runeLength > 3 {
+			val = fmt.Sprintf("%c%d%c", r[0], runeLength-2, r[runeLength-1])
 		}
+		result[i] = val
 	}
 	return result
 }
diff --git a/04_numeronym/sirigithub/main_test.go b/04_numeronym/sirigithub/main_test.go
index 5c774e0cd..679796548 100644
--- a/04_numeronym/sirigithub/main_test.go
+++ b/04_numeronym/sirigithub/main_test.go
@@ -36,9 +36,10 @@ func TestNumeronyms(t *testing.T) {
 			expected: []string{"a6$", "*3]"}},
 	}
 	for _, test := range tests {
-		actual := numeronyms(test.input...)
-		expected := test.expected
+		test := test
 		t.Run(test.description, func(t *testing.T) {
+			actual := numeronyms(test.input...)
+			expected := test.expected
 			assert.Equal(t, actual, expected, "actual %v but expected %v", actual, expected)
 		})
 	}