Skip to content

Commit d75947a

Browse files
committed
Fix: Update
1 parent 5d8416b commit d75947a

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
@@ -364,22 +364,16 @@ func TestWorkflowQueues(t *testing.T) {
364364
}
365365

366366
// Check the workflow completes
367-
// dlqCompleteEvent.Set()
368367
for _, handle := range handles {
369-
result, resErr := handle.GetResult()
370-
handleStatus, statusErr := handle.GetStatus()
371368

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

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

374+
assert.Equal(t, WorkflowStatusMaxRecoveryAttemptsExceeded, dbosErr.Code, "expected workflow to be in DLQ after max retries exceeded")
382375
}
376+
dlqCompleteEvent.Set()
383377

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

0 commit comments

Comments
 (0)