fix(GODT-2201): Flag list can be empty

imap/command/flags.go

@@ -24,27 +24,29 @@ func ParseFlagList(p *rfcparser.Parser) ([]string, error) {
 		return nil, err
 	}
 
-	{
-		firstFlag, err := ParseFlag(p)
-		if err != nil {
-			return nil, err
+	if !p.Check(rfcparser.TokenTypeRParen) {
+		{
+			firstFlag, err := ParseFlag(p)
+			if err != nil {
+				return nil, err
+			}
+			flags = append(flags, firstFlag)
 		}
-		flags = append(flags, firstFlag)
-	}
 
-	for {
-		if ok, err := p.Matches(rfcparser.TokenTypeSP); err != nil {
-			return nil, err
-		} else if !ok {
-			break
-		}
+		for {
+			if ok, err := p.Matches(rfcparser.TokenTypeSP); err != nil {
+				return nil, err
+			} else if !ok {
+				break
+			}
 
-		flag, err := ParseFlag(p)
-		if err != nil {
-			return nil, err
-		}
+			flag, err := ParseFlag(p)
+			if err != nil {
+				return nil, err
+			}
 
-		flags = append(flags, flag)
+			flags = append(flags, flag)
+		}
 	}
 
 	if err := p.Consume(rfcparser.TokenTypeRParen, "Expected ')' at end of flag list"); err != nil {

imap/command/flags_test.go

@@ -11,6 +11,7 @@ func TestParser_ParseFlagList(t *testing.T) {
 	values := map[string][]string{
 		`(\Answered)`:                {`\Answered`},
 		`(\Answered Foo \Something)`: {`\Answered`, `Foo`, `\Something`},
+		`()`:                         nil,
 	}
 
 	for input, expected := range values {
@@ -24,7 +25,6 @@ func TestParser_ParseFlagList(t *testing.T) {
 
 func TestParser_ParseFlagListInvalid(t *testing.T) {
 	inputs := [][]byte{
-		[]byte(`()`),
 		[]byte(`(\Foo\Bar)`),
 		[]byte(`"(\Recent)`),
 		[]byte(`(\Foo )`),