Description

I encountered this code while trying to write an integration test which simulated server responses.

In this code:

If the backend's assemble endpoint only ever returns a "created" response, and this response lacks an "artifact_url" (which could happen if we ever remove this field from the server's responses in the future for whatever reason), we will get stuck in an infinite loop.

Two thoughts:

1. The final if statement should also check if we return "created", not just "Error" or "OK"
2. There needs to be some sort of timeout and/or maximum iteration count on this loop, so that it is impossible to get stuck in an infinite loop.