Backport(バックポート) #3464
機能テストに存在する雛形のままのテストコードを削除する
100%
Description
Overview (現象)¶
OpenPNE に同梱されている機能テストの一部は symfony が提供する雛形のコードのまま手が加えられておらず、中には symfony 1.4 と互換性のないメソッドを使用しているテストも含まれている。
これらのテストコードは実装に合わせて修正されない限り自動テストの妨げとなるため削除を行う。
Related issues
Associated revisions
Merge branch 't-3464' into stable-3.8.x
History
#1
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from New(新規) to Accepted(着手)
- Assignee set to Akihiro KOBAYASHI
#2
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
pull request: https://github.com/openpne/OpenPNE3/pull/91
#3
Updated by Youichi Kimura about 9 years ago
- Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
stable-3.8.x については source:test/functional/pc_frontend/communityActionsTest.php@3e28e407 にも雛形テストコードが存在するため、このファイルも追加で削除して下さい。
#4
Updated by Youichi Kimura about 9 years ago
バックポート元の #3431 に含まれる 2 つのコミットのうち 52010fe24dd57de0c5265689f96986677452c5e7 が Pull Request に含まれていないため、これも追加で cherry-pick を行って下さい。
#5
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Rejected(差し戻し) to Accepted(着手)
- % Done changed from 50 to 0
#6
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
#7
Updated by Youichi Kimura about 9 years ago
- Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
#8
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Rejected(差し戻し) to Accepted(着手)
- % Done changed from 50 to 0
#9
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
#10
Updated by Youichi Kimura about 9 years ago
- Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
note-7 が修正されていることを確認しました。
https://github.com/openpne/OpenPNE3/pull/91#issuecomment-28264180 についてコミットメッセージの不備があるため修正お願いします。
#11
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Rejected(差し戻し) to Accepted(着手)
- % Done changed from 50 to 0
#12
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
#13
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Pending Review(レビュー待ち) to Accepted(着手)
- % Done changed from 50 to 0
#14
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
#15
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Pending Review(レビュー待ち) to Accepted(着手)
- % Done changed from 50 to 0
#16
Updated by Akihiro KOBAYASHI about 9 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
#17
Updated by Youichi Kimura about 9 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
レビューOKです
#18
Updated by Youichi Kimura about 9 years ago
- Status changed from Pending Testing(テスト待ち) to Pending Review(レビュー待ち)
- % Done changed from 70 to 50
更新履歴 d829f01661d70598bf32d69aa0458515791fb36a で適用されました。
#19
Updated by Youichi Kimura about 9 years ago
更新履歴 5b6a1f84c1b8c6cdf521ab2aa61a0bc1a1b38570 で適用されました。
#20
Updated by Akihiro KOBAYASHI about 9 years ago
更新履歴 6ebadaeed2850bc4f3e646d7e71be91ecf000ca1 で適用されました。
#21
Updated by Youichi Kimura about 9 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
#22
Updated by Mutsumi Imamura about 9 years ago
- Status changed from Pending Testing(テスト待ち) to Fixed(完了)
- % Done changed from 70 to 100
./symfony test:all を実行し異常終了しないことを確認しました。
ブラウザからの動作テストも問題ありません。