Skip to content

Commit 1d30808

Browse files
vr-varadmaxdml
authored andcommitted
Fix: Update
1 parent c0be5c6 commit 1d30808

File tree

1 file changed

+5
-11
lines changed

1 file changed

+5
-11
lines changed

dbos/queues_test.go

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -365,22 +365,16 @@ func TestWorkflowQueues(t *testing.T) {
365365
}
366366

367367
// Check the workflow completes
368-
// dlqCompleteEvent.Set()
369368
for _, handle := range handles {
370-
result, resErr := handle.GetResult()
371-
handleStatus, statusErr := handle.GetStatus()
372369

373-
fmt.Println("DLQ Test Handle Result:", result, "Error:", resErr, "Status:", handleStatus, "StatusErr:", statusErr)
370+
_, resultErr := handle.GetResult()
374371

375-
if result == nil {
376-
assert.Equal(t, WorkflowStatusMaxRecoveryAttemptsExceeded, handleStatus.Status, "expected workflow to be in DLQ after max retries exceeded")
377-
// resErr is not nil
378-
} else {
379-
// resErr is nil
380-
assert.Equal(t, WorkflowStatusSuccess, handleStatus.Status, "expected workflow status to be SUCCESS when result is not nil")
381-
}
372+
var dbosErr *DBOSError
373+
require.ErrorAs(t, resultErr, &dbosErr, "expected error to be of type *DBOSError, got %T", resultErr)
382374

375+
assert.Equal(t, WorkflowStatusMaxRecoveryAttemptsExceeded, dbosErr.Code, "expected workflow to be in DLQ after max retries exceeded")
383376
}
377+
dlqCompleteEvent.Set()
384378

385379
require.True(t, queueEntriesAreCleanedUp(dbosCtx), "expected queue entries to be cleaned up after successive enqueues test")
386380
})

0 commit comments

Comments
 (0)