From db8b52608b18301b11b54fdfd0363b5e77fa6567 Mon Sep 17 00:00:00 2001 From: Niall Sheridan Date: Sat, 20 Aug 2016 21:04:05 +0100 Subject: Replace Fatals with Errors --- server/config/config_test.go | 8 ++++---- server/signer/signer_test.go | 10 +++++----- 2 files changed, 9 insertions(+), 9 deletions(-) (limited to 'server') diff --git a/server/config/config_test.go b/server/config/config_test.go index 3a9d4c1..c6bdc3d 100644 --- a/server/config/config_test.go +++ b/server/config/config_test.go @@ -14,7 +14,7 @@ func TestServerConfig(t *testing.T) { a := assert.New(t) c, err := ReadConfig(bytes.NewBuffer(testdata.ServerConfig)) if err != nil { - t.Fatal(err) + t.Error(err) } server := c.Server a.IsType(server, &Server{}) @@ -31,7 +31,7 @@ func TestAuthConfig(t *testing.T) { a := assert.New(t) c, err := ReadConfig(bytes.NewBuffer(testdata.AuthConfig)) if err != nil { - t.Fatal(err) + t.Error(err) } auth := c.Auth a.IsType(auth, &Auth{}) @@ -47,7 +47,7 @@ func TestSSHConfig(t *testing.T) { a := assert.New(t) c, err := ReadConfig(bytes.NewBuffer(testdata.SSHConfig)) if err != nil { - t.Fatal(err) + t.Error(err) } ssh := c.SSH a.IsType(ssh, &SSH{}) @@ -57,7 +57,7 @@ func TestSSHConfig(t *testing.T) { a.Equal(ssh.MaxAge, "720h") d, err := time.ParseDuration(ssh.MaxAge) if err != nil { - t.Fatal(err) + t.Error(err) } a.Equal(d.Hours(), float64(720)) } diff --git a/server/signer/signer_test.go b/server/signer/signer_test.go index 5fbf2c6..805f0fc 100644 --- a/server/signer/signer_test.go +++ b/server/signer/signer_test.go @@ -32,24 +32,24 @@ func TestCert(t *testing.T) { } cert, err := signer.SignUserKey(r) if err != nil { - t.Fatal(err) + t.Error(err) } if !bytes.Equal(cert.SignatureKey.Marshal(), signer.ca.PublicKey().Marshal()) { - t.Fatal("Cert signer and server signer don't match") + t.Error("Cert signer and server signer don't match") } var principals []string principals = append(principals, r.Principal) principals = append(principals, signer.principals...) if !reflect.DeepEqual(cert.ValidPrincipals, principals) { - t.Fatalf("Expected %s, got %s", cert.ValidPrincipals, principals) + t.Errorf("Expected %s, got %s", cert.ValidPrincipals, principals) } k1, _, _, _, err := ssh.ParseAuthorizedKey([]byte(r.Key)) k2 := cert.Key if !bytes.Equal(k1.Marshal(), k2.Marshal()) { - t.Fatal("Cert key doesn't match public key") + t.Error("Cert key doesn't match public key") } if cert.ValidBefore != uint64(r.ValidUntil.Unix()) { - t.Fatalf("Invalid validity, expected %d, got %d", r.ValidUntil, cert.ValidBefore) + t.Errorf("Invalid validity, expected %d, got %d", r.ValidUntil, cert.ValidBefore) } } -- cgit v1.2.3