diff --git a/directory/api/entries.go b/directory/api/entries.go index daae8a7f..bc2e4ce1 100644 --- a/directory/api/entries.go +++ b/directory/api/entries.go @@ -1153,15 +1153,9 @@ func (a ApiImpl) UpdateEntry(ctx context.Context, request UpdateEntryRequestObje } else { lmsConfig := request.Body.LmsConfig.MustGet() originalLMSConfig, queryErr := qtx.GetLMSConfigByEntry(ctx, orig.ID) - if queryErr != nil { - if errors.Is(queryErr, pgx.ErrNoRows) { - if lmsConfig.Address == nil || lmsConfig.FromAgency == nil { - return UpdateEntry400TextResponse("lmsConfig.address and lmsConfig.fromAgency are required when creating lmsConfig"), nil - } - } else { - slog.ErrorContext(ctx, "unable to query original lmsConfig", "error", queryErr) - return UpdateEntry500TextResponse("Internal server error"), nil - } + if queryErr != nil && !errors.Is(queryErr, pgx.ErrNoRows) { + slog.ErrorContext(ctx, "unable to query original lmsConfig", "error", queryErr) + return UpdateEntry500TextResponse("Internal server error"), nil } _, err = qtx.UpsertLMSConfig(ctx, db.UpsertLMSConfigParams{ diff --git a/directory/test/entries_test.go b/directory/test/entries_test.go index fcd2f22f..f6a324d5 100644 --- a/directory/test/entries_test.go +++ b/directory/test/entries_test.go @@ -450,7 +450,8 @@ func TestEntryCases(t *testing.T) { method: http.MethodPatch, endpoint: "/entries/by-id/00000000-0000-0000-0000-000000000003", bodyFile: "entry-new-lmsconfig-incomplete.patch.req.json", - status: http.StatusBadRequest, + status: http.StatusNoContent, + refetchFile: "entry-new-lmsconfig-incomplete.patch.refetch.json", addlHeaders: consortiumPermissionHeaders, }, {