Browse Source

Fix issue with one-character args in ArgsToStrings.

tags/v2.1.0
Alex Jaspersen 4 years ago
parent
commit
2cb91bab04
2 changed files with 9 additions and 1 deletions
  1. 1
    1
      irc/utils/args.go
  2. 8
    0
      irc/utils/args_test.go

+ 1
- 1
irc/utils/args.go View File

@@ -35,7 +35,7 @@ func ArgsToStrings(maxLength int, arguments []string, delim string) []string {
35 35
 			continue
36 36
 		}
37 37
 
38
-		if len(buffer) > 1 {
38
+		if len(buffer) > 0 {
39 39
 			buffer += delim
40 40
 		}
41 41
 		buffer += arguments[0]

+ 8
- 0
irc/utils/args_test.go View File

@@ -5,6 +5,14 @@ package utils
5 5
 
6 6
 import "testing"
7 7
 
8
+func TestArgsToStrings(t *testing.T) {
9
+	val := ArgsToStrings(512, []string{"a", "b", "c"}, ",")
10
+	assertEqual(val, []string{"a,b,c"}, t)
11
+
12
+	val = ArgsToStrings(10, []string{"abcd", "efgh", "ijkl"}, ",")
13
+	assertEqual(val, []string{"abcd,efgh", "ijkl"}, t)
14
+}
15
+
8 16
 func TestStringToBool(t *testing.T) {
9 17
 	val, err := StringToBool("on")
10 18
 	assertEqual(val, true, t)

Loading…
Cancel
Save