clean up field assignments with default values to make test configura…

…tion more concise/readable
labstack · Oct 26, 2024 · 5a0b4dd · 5a0b4dd
1 parent 03c0236
commit 5a0b4dd
Showing 1 changed file with 6 additions and 13 deletions.
diff --git a/middleware/basic_auth_test.go b/middleware/basic_auth_test.go
@@ -25,7 +25,6 @@ func TestBasicAuth(t *testing.T) {
 		return false, nil
 	}
 
-	// Define the test cases
 	tests := []struct {
 		name           string
 		authHeader     string
@@ -36,46 +35,40 @@ func TestBasicAuth(t *testing.T) {
 		expectedErrMsg string
 	}{
 		{
-			name:          "Valid credentials",
-			authHeader:    basic + " " + base64.StdEncoding.EncodeToString([]byte("joe:secret")),
-			expectedCode:  http.StatusOK,
-			skipperResult: false,
+			name:         "Valid credentials",
+			authHeader:   basic + " " + base64.StdEncoding.EncodeToString([]byte("joe:secret")),
+			expectedCode: http.StatusOK,
 		},
 		{
-			name:          "Case-insensitive header scheme",
-			authHeader:    strings.ToUpper(basic) + " " + base64.StdEncoding.EncodeToString([]byte("joe:secret")),
-			expectedCode:  http.StatusOK,
-			skipperResult: false,
+			name:         "Case-insensitive header scheme",
+			authHeader:   strings.ToUpper(basic) + " " + base64.StdEncoding.EncodeToString([]byte("joe:secret")),
+			expectedCode: http.StatusOK,
 		},
 		{
 			name:           "Invalid credentials",
 			authHeader:     basic + " " + base64.StdEncoding.EncodeToString([]byte("joe:invalid-password")),
 			expectedCode:   http.StatusUnauthorized,
 			expectedAuth:   basic + ` realm="someRealm"`,
-			skipperResult:  false,
 			expectedErr:    true,
 			expectedErrMsg: "Unauthorized",
 		},
 		{
 			name:           "Invalid base64 string",
 			authHeader:     basic + " invalidString",
 			expectedCode:   http.StatusBadRequest,
-			skipperResult:  false,
 			expectedErr:    true,
 			expectedErrMsg: "Bad Request",
 		},
 		{
 			name:           "Missing Authorization header",
 			expectedCode:   http.StatusUnauthorized,
-			skipperResult:  false,
 			expectedErr:    true,
 			expectedErrMsg: "Unauthorized",
 		},
 		{
 			name:           "Invalid Authorization header",
 			authHeader:     base64.StdEncoding.EncodeToString([]byte("invalid")),
 			expectedCode:   http.StatusUnauthorized,
-			skipperResult:  false,
 			expectedErr:    true,
 			expectedErrMsg: "Unauthorized",
 		},