refactor: change max size of element name to 128 (#297)

a-h · Nov 13, 2023 · f18e82f · f18e82f
1 parent b511005
commit f18e82f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/parser/v2/elementparser.go b/parser/v2/elementparser.go
@@ -313,9 +313,9 @@ var (
 			in.Seek(start)
 			return
 		}
-		if len(suffix)+1 > 32 {
+		if len(suffix)+1 > 128 {
 			ok = false
-			err = parse.Error("element names must be < 32 characters long", in.Position())
+			err = parse.Error("element names must be < 128 characters long", in.Position())
 			return
 		}
 		return prefix + suffix, true, nil

diff --git a/parser/v2/elementparser_test.go b/parser/v2/elementparser_test.go
@@ -956,13 +956,13 @@ func TestElementParserErrors(t *testing.T) {
 				}),
 		},
 		{
-			name:  "element: names cannot be greater than 32 characters",
-			input: `<aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa></aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa>`,
-			expected: parse.Error("element names must be < 32 characters long",
+			name:  "element: names cannot be greater than 128 characters",
+			input: `<aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa></aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa>`,
+			expected: parse.Error("element names must be < 128 characters long",
 				parse.Position{
-					Index: 34,
+					Index: 130,
 					Line:  0,
-					Col:   34,
+					Col:   130,
 				}),
 		},
 	}