Handle STORE FLAGS A B syntax

See emersion#243 (comment).

Author: foxcpp

commands/store.go

@@ -41,7 +41,10 @@ func (cmd *Store) Parse(fields []interface{}) error {
 		cmd.Item = imap.StoreItem(strings.ToUpper(item))
 	}
 
-	// TODO: could be fields[2:] according to RFC 3501 page 91 "store-att-flags"
-	cmd.Value = fields[2]
+	if len(fields[2:]) == 1 {
+		cmd.Value = fields[2]
+	} else {
+		cmd.Value = fields[2:]
+	}
 	return nil
 }

server/cmd_selected_test.go

@@ -369,6 +369,24 @@ func TestStore_InvalidFlags(t *testing.T) {
 	}
 }
 
+func TestStore_NonList(t *testing.T) {
+	s, c, scanner := testServerSelected(t, false)
+	defer c.Close()
+	defer s.Close()
+
+	io.WriteString(c, "a001 STORE 1 FLAGS somestring someanotherstring\r\n")
+
+	scanner.Scan()
+	if scanner.Text() != "* 1 FETCH (FLAGS (somestring someanotherstring))" {
+		t.Fatal("Invalid FETCH response:", scanner.Text())
+	}
+
+	scanner.Scan()
+	if !strings.HasPrefix(scanner.Text(), "a001 OK ") {
+		t.Fatal("Invalid status response:", scanner.Text())
+	}
+}
+
 func TestStore_Uid(t *testing.T) {
 	s, c, scanner := testServerSelected(t, false)
 	defer c.Close()