search

Azure / azure-functions-durable-powershell

PowerShell SDK for writing Durable Functions apps
MIT License
7 stars 3 forks source link

Add suspend-resume support for durable client #72

Closed nytian closed 6 months ago

nytian commented 7 months ago

As titled.

nytian commented 7 months ago

I didn't add the resume test yet since for the resume e2e test, I think we need 4 responses back: 1.orchestration has been initialized, 2. orchestration is running, 3. orchestration is suspended, 4. orchestration is resumed. And it seems like ValidateDurableWorkflowResults can have 3 responses check in max.

But I think it's better not edit the method ValidateDurableWorkflowResults considering lots of tests using this. My idea now is that starting an orchestrator can't be finished, like listening to an external event which will never be sent, and then calling suspend, waiting for 3 sec to make sure it's suspended, calling resume, check if it's back to running? Appreciate your thoughts about this test. Thank you! @davidmrdavid

davidmrdavid commented 7 months ago

I didn't add the resume test yet since for the resume e2e test, I think we need 4 responses back: 1.orchestration has been initialized, 2. orchestration is running, 3. orchestration is suspended, 4. orchestration is resumed. And it seems like ValidateDurableWorkflowResults can have 3 responses check in max.

Not sure I follow this. Let's sync on this tomorrow