Skip to content

Commit 26b74af

Browse files
committed
test(routing): pin LightClassifier off in the live-toggle fixture so the edge fires
1 parent 1c12b27 commit 26b74af

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

routing_classifier_install_test.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -164,13 +164,15 @@ func TestSetReliabilitySettingsTogglesLightClassifierLive(t *testing.T) {
164164
ipPath := testLightIpPath(IpProtocolTcp, "93.184.216.1", 443)
165165

166166
parent, clients := flowCapTestParent(t, 0, 50, 1)
167+
// pin the knob off: LightClassifier is ON by default since 2732116, and this
168+
// test needs a real off->on EDGE for SetReliabilitySettings to install on.
169+
parent.settings.LightClassifier = false
167170
parent.config.Store(&multiClientConfig{
168171
serverNameLookup: stubServerNameLookup{names: []string{"netflix.com"}},
169172
})
170173

171-
// before any toggle: constructed with LightClassifier off (flowCapTestParent
172-
// uses DefaultMultiClientSettings, which is zero-value-off), so no
173-
// classifier and the legacy order stands.
174+
// before any toggle: LightClassifier pinned off just above (it defaults ON
175+
// since 2732116), so no classifier and the legacy order stands.
174176
if parent.flowClassifier.Load() != nil {
175177
t.Fatal("no classifier should be installed before any runtime toggle")
176178
}

0 commit comments

Comments
 (0)