Skip to content

Commit 202c8ba

Browse files
committed
Shorten the sleep in test_reuse_sandboxes().
Today, it seems not to need as long. The unpredictability displeases me.
1 parent 2c0f6bc commit 202c8ba

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

tests/test_game_of_life_example.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,14 +36,15 @@ def test_reuse_sandboxes(self):
3636
"""Make sure attempting to issue multiple requests to a single sandbox doesn't crash.
3737
3838
This does not test whether a single sandbox actually served multiple
39-
requests; 2 sandboxes could have served 1 request each.
39+
requests, though it tries to provoke that. Still, 2 sandboxes could have
40+
served 1 request each.
4041
"""
4142
response = self.get("/board/none")
4243
assert response.status_code == 200
4344
response = self.get("/board/none")
4445
assert response.status_code == 200
4546

46-
# The error about failed sandbox reuse comes *after* the request has
47-
# succeeded. And it seems to take forever to show up.
48-
sleep(4) # 2 is not enough. I am sad.
47+
# Reports about crashers in the post-response code come *after* the
48+
# request has succeeded. And it seems to take awhile to show up.
49+
sleep(0.5) # .3 is not enough.
4950
assert "WebAssembly trapped" not in "\n".join(self.server.output_lines)

0 commit comments

Comments
 (0)