Fix TestErrors() to not misuse auth bypass

This commit is contained in:
Bernhard Froehlich 2020-05-26 19:54:20 +00:00
parent 44fcf08c5e
commit d4395c8d1b
No known key found for this signature in database
GPG key ID: 4DD88C3F9F3B8333

View file

@ -1188,12 +1188,8 @@ func TestErrors(t *testing.T) {
t.Fatalf("AUTH didn't fail: %v", err) t.Fatalf("AUTH didn't fail: %v", err)
} }
if err := c.Mail("sender@example.org"); err != nil {
t.Fatalf("MAIL failed: %v", err)
}
if err := c.Mail("sender@example.org"); err == nil { if err := c.Mail("sender@example.org"); err == nil {
t.Fatal("Duplicate MAIL didn't fail") t.Fatalf("MAIL didn't fail")
} }
if err := cmd(c.Text, 502, "STARTTLS"); err != nil { if err := cmd(c.Text, 502, "STARTTLS"); err != nil {
@ -1228,6 +1224,14 @@ func TestErrors(t *testing.T) {
t.Fatalf("AUTH didn't work: %v", err) t.Fatalf("AUTH didn't work: %v", err)
} }
if err := c.Mail("sender@example.org"); err != nil {
t.Fatalf("MAIL failed: %v", err)
}
if err := c.Mail("sender@example.org"); err == nil {
t.Fatalf("Duplicate MAIL didn't fail")
}
if err := c.Quit(); err != nil { if err := c.Quit(); err != nil {
t.Fatalf("Quit failed: %v", err) t.Fatalf("Quit failed: %v", err)
} }