Skip to content

Commit

Permalink
only use 3 hooks, don't be insane
Browse files Browse the repository at this point in the history
  • Loading branch information
cwaldren-ld committed Apr 2, 2024
1 parent 9c45311 commit 89b58a3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions sdktests/server_side_hooks.go
Original file line number Diff line number Diff line change
Expand Up @@ -276,7 +276,7 @@ func beforeEvaluationDataPropagatesToAfterMigration(t *ldtest.T) {
// The client MUST handle exceptions which are thrown (or errors returned, if idiomatic for the language)
// during the execution of a stage or handler allowing operations to complete unaffected.
func errorInBeforeStageDoesNotAffectAfterStage(t *ldtest.T) {
const numHooks = 100 // why not?
const numHooks = 3

// We're configuring the beforeEvaluation stage with some data, but we don't expect
// to see it propagated into afterEvaluation since we're also configuring beforeEvaluation
Expand Down Expand Up @@ -312,7 +312,7 @@ func errorInBeforeStageDoesNotAffectAfterStage(t *ldtest.T) {
"returned in this stage")

assert.Equal(t, 0, len(call.EvaluationSeriesData.Value()), "HOOKS:1.3.7.1: Since "+
"beforeEvaluation should have failed, the data passed to afterEvaluation should be an empty string")
"beforeEvaluation should have failed, the data passed to afterEvaluation should be empty")
}
}

Expand Down

0 comments on commit 89b58a3

Please sign in to comment.