Project

General

Profile

Backport（バックポート） #3464

機能テストに存在する雛形のままのテストコードを削除する

Added by Mutsumi Imamura over 7 years ago. Updated over 7 years ago.

Status:
Fixed（完了）
Priority:
Normal（通常）
Assignee:
Akihiro KOBAYASHI
Target version:
OpenPNE 3.8.9
Start date:
2013-10-08
Due date:
% Done:

100%

Description

Overview (現象)

OpenPNE に同梱されている機能テストの一部は symfony が提供する雛形のコードのまま手が加えられておらず、中には symfony 1.4 と互換性のないメソッドを使用しているテストも含まれている。
これらのテストコードは実装に合わせて修正されない限り自動テストの妨げとなるため削除を行う。

Related issues

Related to OpenPNE 3 - Bug（バグ） #3431: 機能テストに存在する雛形のままのテストコードを削除する Won't fix（対応せず） 2013-10-08

Associated revisions

Revision d829f016 (diff)
Added by Youichi Kimura over 7 years ago

remove skeleton tests for mobile_frontend and mobile_mail_frontend
(fixes #3464, BP from #3431)
(cherry picked from commit 5542d6174689fe6951093e37f493f8aaa5c892f4)

Revision 5b6a1f84 (diff)
Added by Youichi Kimura over 7 years ago

remove skeleton tests for pc_frontend (fixes #3464, BP from #3431)
(cherry picked from commit 52010fe24dd57de0c5265689f96986677452c5e7)

Revision 6ebadaee (diff)
Added by Akihiro KOBAYASHI over 7 years ago

remove skeleton tests for mobile_frontend and mobile_mail_frontend
(fixes #3464, BP from #3431)

Revision f563ccb4
Added by Youichi Kimura over 7 years ago

Merge branch 't-3464' into stable-3.8.x

History

#1 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from New（新規） to Accepted（着手）
  • Assignee set to Akihiro KOBAYASHI

#2 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

#3 Updated by Youichi Kimura over 7 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 over 7 years ago

バックポート元の #3431 に含まれる 2 つのコミットのうち 52010fe24dd57de0c5265689f96986677452c5e7 が Pull Request に含まれていないため、これも追加で cherry-pick を行って下さい。

#5 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from Rejected（差し戻し） to Accepted（着手）
  • % Done changed from 50 to 0

#6 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

#7 Updated by Youichi Kimura over 7 years ago

  • Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）

https://github.com/openpne/OpenPNE3/pull/91#issuecomment-28061144 について再度確認お願いします。

#8 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from Rejected（差し戻し） to Accepted（着手）
  • % Done changed from 50 to 0

#9 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

#10 Updated by Youichi Kimura over 7 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 over 7 years ago

  • Status changed from Rejected（差し戻し） to Accepted（着手）
  • % Done changed from 50 to 0

#12 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

#13 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from Pending Review（レビュー待ち） to Accepted（着手）
  • % Done changed from 50 to 0

#14 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

#15 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from Pending Review（レビュー待ち） to Accepted（着手）
  • % Done changed from 50 to 0

#16 Updated by Akihiro KOBAYASHI over 7 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

#17 Updated by Youichi Kimura over 7 years ago

  • Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
  • % Done changed from 50 to 70

レビューOKです

#18 Updated by Youichi Kimura over 7 years ago

  • Status changed from Pending Testing（テスト待ち） to Pending Review（レビュー待ち）
  • % Done changed from 70 to 50

更新履歴 d829f01661d70598bf32d69aa0458515791fb36a で適用されました。

#19 Updated by Youichi Kimura over 7 years ago

更新履歴 5b6a1f84c1b8c6cdf521ab2aa61a0bc1a1b38570 で適用されました。

#20 Updated by Akihiro KOBAYASHI over 7 years ago

更新履歴 6ebadaeed2850bc4f3e646d7e71be91ecf000ca1 で適用されました。

#21 Updated by Youichi Kimura over 7 years ago

  • Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
  • % Done changed from 50 to 70

#22 Updated by Mutsumi Imamura over 7 years ago

  • Status changed from Pending Testing（テスト待ち） to Fixed（完了）
  • % Done changed from 70 to 100

./symfony test:all を実行し異常終了しないことを確認しました。
ブラウザからの動作テストも問題ありません。

Also available in: Atom PDF