all: fix gometalinter warnings

Fix typos, non-canonical code, remove dead code, etc.

2 files changed, 4 insertions, 1 deletions

diff --git a/pkg/email/parser_test.go b/pkg/email/parser_test.go
index 69e237806..a99ab17f6 100644
--- a/pkg/email/parser_test.go
+++ b/pkg/email/parser_test.go
@@ -91,6 +91,9 @@ func TestAddAddrContextEmptyName(t *testing.T) {
 		t.Fatalf("want: %q, got %q", want, email1)
 	}
 	email2, context1, err := RemoveAddrContext(email1)
+	if err != nil {
+		t.Fatal(err)
+	}
 	if email != email2 {
 		t.Fatalf("want: %q, got %q", email, email2)
 	}
diff --git a/pkg/email/patch_test.go b/pkg/email/patch_test.go
index b10211ff0..c8cae4621 100644
--- a/pkg/email/patch_test.go
+++ b/pkg/email/patch_test.go
@@ -157,7 +157,7 @@ The handling of the might_cancel queueing is not properly protected, so
 parallel operations on the file descriptor can race with each other and
 lead to list corruptions or use after free.
 
-Protect the context for these operations with a seperate lock.
+Protect the context for these operations with a separate lock.
 
 Reported-by: Foo Bar <foo@bar.com>
 Signed-off-by: Foo Bar <foo@bar.com>