search

soma0822 / webserv

2 stars 1 forks source link

POSTのテストを追加する #398

Closed tkuramot closed 7 months ago

tkuramot commented 7 months ago

1. issue number

373

2. やったこと

POSTのテストを追加しました これが完了したらtestブランチはマージしちゃってよさそう

3. やらないこと

4. 動作確認

5. 懸念点

6. 最近楽しかったこと

7. その他

リクエストのディレクトリ名をbad_request_400にすると、 テスト結果の順序がぐちゃぐちゃでわかりづらいなと今になって思いました 400_bad_requestみたいにステータスコードを先に書くと結果がステータスコード順になってみやすそう 今更すぎるけどやりますか？

soma0822 commented 7 months ago

1. issue number

373

2. やったこと

POSTのテストを追加しました これが完了したらtestブランチはマージしちゃってよさそう

3. やらないこと

4. 動作確認

5. 懸念点

6. 最近楽しかったこと

7. その他

リクエストのディレクトリ名をbad_request_400にすると、 テスト結果の順序がぐちゃぐちゃでわかりづらいなと今になって思いました 400_bad_requestみたいにステータスコードを先に書くと結果がステータスコード順になってみやすそう 今更すぎるけどやりますか？

やった方が良さそう？ ここでやりますか？ 別ブランチ立てますか？

tkuramot commented 7 months ago

やった方が良さそう？ ここでやりますか？ 別ブランチ立てますか？

やっぱいいかと思い始めました とりあえずmergeしちゃいます