Closed 110souhei closed 5 months ago
@110souhei おおむねいいと思うけど、以下の点がちょっと気になるからそれを修正してもう一回プルリク送ってほしい
.
で始まるファイルと__pycache__
かがあるからそれを含めないでほしいTestDatasetManager
はテスト実行時に各メンバ関数を自動で実行するから、DatasetManager
の初期化とかは各関数内に書いて__init__
は追加しないほうがいいと思うtest_*
がそれ単体を実行するだけで目的の関数のテストができるようにしてほしい
test_create
はcreate
の動作のみを検証して、そのテストのためにほかのtest_*
を呼び出す必要がないようにしてほしい
I think dataset.py is correct on this test case.