fix(controller): Cancel tests on fatal errors (#1994)

When requesting a token, any error is fatal to the test.
The previous behavior would return a nil token, which
could cause subsequent uses to panic. This will instead
interrupt any test immediately on error.
This commit is contained in:
Johan Brandhorst-Satzkorn authored and GitHub committed 2022-04-12 16:56:04 -04:00
1 parent 9b48dbc2fd
commit 8d80d10ff7
1 file changed
+3 -3
+3 -3
View File
@@ -152,7 +152,7 @@ func (tc *TestController) Logger() hclog.Logger {
func (tc *TestController) Token() *authtokens.AuthToken {
if tc.opts.DisableAuthMethodCreation {
tc.t.Error("no default auth method ID configured")
tc.t.Fatal("no default auth method ID configured")
return nil
}
result, err := authmethods.NewClient(tc.Client()).Authenticate(
@@ -165,12 +165,12 @@ func (tc *TestController) Token() *authtokens.AuthToken {
},
)
if err != nil {
tc.t.Error(fmt.Errorf("error logging in: %w", err))
tc.t.Fatal(fmt.Errorf("error logging in: %w", err))
return nil
}
token := new(authtokens.AuthToken)
if err := json.Unmarshal(result.GetRawAttributes(), token); err != nil {
tc.t.Error(fmt.Errorf("error unmarshaling token: %w", err))
tc.t.Fatal(fmt.Errorf("error unmarshaling token: %w", err))
return nil
}
return token