Skip to content

Commit 37fe013

Browse files
Remove unnecessary test in requestcontrol_hooks_test
1 parent ed3b0cd commit 37fe013

File tree

1 file changed

+0
-23
lines changed

1 file changed

+0
-23
lines changed

pkg/epp/scheduling/framework/plugins/multi/slo_aware_router/requestcontrol_hooks_test.go

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -178,29 +178,6 @@ func TestSLOAwareRouter_PreRequest_Success(t *testing.T) {
178178
retrievedCtx.RequestReceivedTimestamp.Equal(afterTime))
179179
}
180180

181-
func TestSLOAwareRouter_PreRequest_GeneratesRequestID(t *testing.T) {
182-
router := createTestRouter()
183-
ctx := context.Background()
184-
pod := createTestPod("test-pod", 1, 1, 1)
185-
request := createTestLLMRequest("", 100, 50, true)
186-
request.Headers[requtil.RequestIdHeaderKey] = "" // Explicitly empty
187-
schedulingResult := createTestSchedulingResult(pod.GetPod(), 1, 1, 1)
188-
189-
// Create and set initial SLO context
190-
sloCtx := NewSLORequestContext(request)
191-
sloCtx.AvgTPOTSLO = 50
192-
193-
// Since request ID is empty initially, we need to handle this
194-
// The PreRequest should generate a new ID, so let's test that
195-
router.PreRequest(ctx, request, schedulingResult)
196-
197-
// Request ID should now be set
198-
assert.NotEmpty(t, request.Headers[requtil.RequestIdHeaderKey])
199-
// Verify it's a valid UUID format
200-
_, err := uuid.Parse(request.Headers[requtil.RequestIdHeaderKey])
201-
assert.NoError(t, err, "Generated request ID should be a valid UUID")
202-
}
203-
204181
func TestSLOAwareRouter_PreRequest_AddsToQueue(t *testing.T) {
205182
router := createTestRouter()
206183
ctx := context.Background()

0 commit comments

Comments
 (0)