Wait for end of test

[skymountain]

Fix #241.

[westpaddy]

待つならタイムアウトは指定してほしいです

[skymountain]

どうして必要です？

[skymountain]

ちなみに #241 に関連したより深刻な問題として，detail は「前のテスト結果が表示される」ことが挙げられます．現在の実装だと「何故かテスト通過率は表示されないけど，detail を見るとテスト結果は前と変わらなかったらしい」と誤解する恐れがあります．

さらにちなみにテスト通過率が表示されないのは課題の提出状況がl building に移行するためでした．

[westpaddy]

無限ループするとautomataを立ててる側のサーバーのリソースを握ったままになるので、変だなと思って何回もpostされるとまずい気がします。

[westpaddy]

241 の問題で表示は直さないといけないのは同意してます

[skymountain]

タイムアウトを設けるのは賛成です． 確認ですけど，それは今回のコミットとは直接関係はないですよね？現在の master の実装でもテストが無限ループする(curl の結果が返ってこない)場合はバックグラウンドでゾンビができてしまうことになると思いますが．

[westpaddy]

たしかに。そうなると、消費するリソース量の差くらいの問題でしかないかもしれません。

[westpaddy]

241 の問題ですが、テストがすぐ終わる場合はこのPRで解決できそうですが、すぐ終わらない場合や、無限ループする場合は解決しないと思います。

つまり、このPRを入れた状態で「テスト再実行」すると長々とロード中になると思うのですが、終わらないなと思ってリロードとかすると #241 の状態にたぶんなりますよね？

[westpaddy]

もう一つ別の問題として、学生がpostしたときもテスト終わるまでrecordに遷移しなくなると思うのですが、この間が開く（か、最悪遷移しない）のはよろしくない気がします。

[skymountain]

はい，根本的解決には現在のビルド&テスト状況を返せるようにしないといけないでしょうね． (というかそういう風に書き直すべきでしょう．)

[skymountain]

というわけでこれは一旦閉じます．