Mirrors/go-inbucket
1
0
Fork 0
mirror of https://github.com/jhillyerd/inbucket.git synced 2026-02-10 04:16:00 +00:00
Code Issues Releases Wiki Activity

Replace panic() with t.Fatalf where possible

Browse Source
This commit is contained in:
James Hillyerd
2013-10-14 15:50:07 -07:00
parent 1e85699ccf
commit 6679ce9250
1 changed files with 15 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
smtpd/filestore_test.go
View File

@@ -120,11 +120,11 @@ func TestFSDeliverMany(t *testing.T) {
// Check number of messages // Check number of messages
mb, err := ds.MailboxFor(mbName) mb, err := ds.MailboxFor(mbName)
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to MailboxFor(%q): %v", err)
} }
msgs, err := mb.GetMessages() msgs, err := mb.GetMessages()
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to GetMessages for %q: %v", mbName, err)
} }
assert.Equal(t, i, len(msgs), "Expected %v message(s), but got %v", i, len(msgs)) assert.Equal(t, i, len(msgs), "Expected %v message(s), but got %v", i, len(msgs))
@@ -134,11 +134,11 @@ func TestFSDeliverMany(t *testing.T) {
mb, err := ds.MailboxFor(mbName) mb, err := ds.MailboxFor(mbName)
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to MailboxFor(%q): %v", err)
} }
msgs, err := mb.GetMessages() msgs, err := mb.GetMessages()
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to GetMessages for %q: %v", mbName, err)
} }
assert.Equal(t, len(subjects), len(msgs), "Expected %v message(s), but got %v", assert.Equal(t, len(subjects), len(msgs), "Expected %v message(s), but got %v",
len(subjects), len(msgs)) len(subjects), len(msgs))
@@ -165,11 +165,11 @@ func TestFSDelete(t *testing.T) {
mb, err := ds.MailboxFor(mbName) mb, err := ds.MailboxFor(mbName)
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to MailboxFor(%q): %v", err)
} }
msgs, err := mb.GetMessages() msgs, err := mb.GetMessages()
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to GetMessages for %q: %v", mbName, err)
} }
assert.Equal(t, len(subjects), len(msgs), "Expected %v message(s), but got %v", assert.Equal(t, len(subjects), len(msgs), "Expected %v message(s), but got %v",
len(subjects), len(msgs)) len(subjects), len(msgs))
@@ -181,11 +181,11 @@ func TestFSDelete(t *testing.T) {
// Confirm deletion // Confirm deletion
mb, err = ds.MailboxFor(mbName) mb, err = ds.MailboxFor(mbName)
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to MailboxFor(%q): %v", err)
} }
msgs, err = mb.GetMessages() msgs, err = mb.GetMessages()
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to GetMessages for %q: %v", mbName, err)
} }
subjects = []string{"alpha", "charlie", "echo"} subjects = []string{"alpha", "charlie", "echo"}
@@ -201,11 +201,11 @@ func TestFSDelete(t *testing.T) {
mb, err = ds.MailboxFor(mbName) mb, err = ds.MailboxFor(mbName)
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to MailboxFor(%q): %v", err)
} }
msgs, err = mb.GetMessages() msgs, err = mb.GetMessages()
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to GetMessages for %q: %v", mbName, err)
} }
subjects = []string{"alpha", "charlie", "echo", "foxtrot"} subjects = []string{"alpha", "charlie", "echo", "foxtrot"}
@@ -233,11 +233,11 @@ func TestFSPurge(t *testing.T) {
mb, err := ds.MailboxFor(mbName) mb, err := ds.MailboxFor(mbName)
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to MailboxFor(%q): %v", err)
} }
msgs, err := mb.GetMessages() msgs, err := mb.GetMessages()
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to GetMessages for %q: %v", mbName, err)
} }
assert.Equal(t, len(subjects), len(msgs), "Expected %v message(s), but got %v", assert.Equal(t, len(subjects), len(msgs), "Expected %v message(s), but got %v",
len(subjects), len(msgs)) len(subjects), len(msgs))
@@ -249,11 +249,11 @@ func TestFSPurge(t *testing.T) {
// Confirm deletion // Confirm deletion
mb, err = ds.MailboxFor(mbName) mb, err = ds.MailboxFor(mbName)
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to MailboxFor(%q): %v", err)
} }
msgs, err = mb.GetMessages() msgs, err = mb.GetMessages()
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to GetMessages for %q: %v", mbName, err)
} }
assert.Equal(t, len(msgs), 0, "Expected mailbox to have zero messages, got %v", len(msgs)) assert.Equal(t, len(msgs), 0, "Expected mailbox to have zero messages, got %v", len(msgs))
@@ -278,7 +278,7 @@ func TestFSSize(t *testing.T) {
mb, err := ds.MailboxFor(mbName) mb, err := ds.MailboxFor(mbName)
if err != nil { if err != nil {
panic(err) t.Fatalf("Failed to MailboxFor(%q): %v", err)
} }
for i, id := range sentIds { for i, id := range sentIds {
msg, err := mb.GetMessage(id) msg, err := mb.GetMessage(id)