Skip to content

Commit d14364c

Browse files
committed
f
1 parent f4a302c commit d14364c

File tree

1 file changed

+12
-9
lines changed

1 file changed

+12
-9
lines changed

api/client/client_test.go

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -881,18 +881,22 @@ func TestLinuxPatchAppConfigValues(t *testing.T) {
881881

882882
// Create a test server
883883
server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
884+
// Check request method and path
884885
assert.Equal(t, "PATCH", r.Method)
885-
assert.Equal(t, "/api/linux/install/app/config/values", r.URL.Path)
886+
assert.Equal(t, "/api/kubernetes/install/app/config/values", r.URL.Path)
886887

888+
// Check headers
887889
assert.Equal(t, "application/json", r.Header.Get("Content-Type"))
888890
assert.Equal(t, "Bearer test-token", r.Header.Get("Authorization"))
889891

890-
// Decode and verify request body
892+
// Decode request body
891893
var req types.PatchAppConfigValuesRequest
892894
err := json.NewDecoder(r.Body).Decode(&req)
893895
require.NoError(t, err, "Failed to decode request body")
896+
897+
// Verify the request contains expected values
894898
assert.Equal(t, "new-value", req.Values["test-item"])
895-
assert.Equal(t, "required-value", request.Values["required-item"])
899+
assert.Equal(t, "required-value", req.Values["required-item"])
896900

897901
// Return successful response
898902
w.WriteHeader(http.StatusOK)
@@ -931,7 +935,7 @@ func TestLinuxPatchAppConfigValues(t *testing.T) {
931935
assert.Equal(t, "Bad Request", apiErr.Message)
932936
}
933937

934-
func TestKubernetesSetAppConfigValues(t *testing.T) {
938+
func TestKubernetesPatchAppConfigValues(t *testing.T) {
935939
// Define expected config once
936940
expectedConfig := types.AppConfig{
937941
Groups: []kotsv1beta1.ConfigGroup{
@@ -956,20 +960,19 @@ func TestKubernetesSetAppConfigValues(t *testing.T) {
956960
// Check request method and path
957961
assert.Equal(t, "PATCH", r.Method)
958962
assert.Equal(t, "/api/kubernetes/install/app/config/values", r.URL.Path)
959-
assert.Equal(t, "Bearer test-token", r.Header.Get("Authorization"))
960963

961964
// Check headers
962965
assert.Equal(t, "application/json", r.Header.Get("Content-Type"))
963966
assert.Equal(t, "Bearer test-token", r.Header.Get("Authorization"))
964967

965968
// Decode request body
966-
var request types.SetAppConfigValuesRequest
967-
err := json.NewDecoder(r.Body).Decode(&request)
969+
var req types.PatchAppConfigValuesRequest
970+
err := json.NewDecoder(r.Body).Decode(&req)
968971
require.NoError(t, err, "Failed to decode request body")
969972

970973
// Verify the request contains expected values
971-
assert.Equal(t, "new-value", request.Values["test-item"])
972-
assert.Equal(t, "required-value", request.Values["required-item"])
974+
assert.Equal(t, "new-value", req.Values["test-item"])
975+
assert.Equal(t, "required-value", req.Values["required-item"])
973976

974977
// Return successful response
975978
w.WriteHeader(http.StatusOK)

0 commit comments

Comments
 (0)