問題点 #22281

BcAppModel::expects()のリネーム

滝下 真玄 が3ヶ月前に追加. 3ヶ月前に更新.

ステータス:終了開始日:
優先度:通常期日:
担当者:滝下 真玄進捗率:

100%

カテゴリ:その他
対象バージョン:4.1.3

説明

expects()というメソッド名があるため、現状モデルのモックを作ることができない。
テストの記述が著しく制限されるため、別の名前にしてほしい。

「expect」は「期待する」で、実際の処理(列挙されていないアソシエーションを削除する)を推測しにくい名前なので「cleanAssociations()」のような名前に変えたほうが良いと思う。

【エラーの内容】

Declaration of Mock_Reminder_0874e992::expects(PHPUnit_Framework_MockObject_Matcher_Invocation $matcher) should be compatible with BcAppModel::expects($arguments, $reset = true)

/var/www/html/lib/Cake/TestSuite/CakeTestCase.php:821
/var/www/html/lib/Cake/TestSuite/CakeTestCase.php:864
/var/www/html/lib/Cake/TestSuite/ControllerTestCase.php:380

【定義されている場所】
https://github.com/baserproject/basercms/blob/dev-4/lib/Baser/Model/BcAppModel.php#L1243

【使用している箇所】
https://github.com/baserproject/basercms/blob/dev-4/lib/Baser/Model/BcAppModel.php#L1303
https://github.com/baserproject/basercms/blob/dev-4/lib/Baser/Plugin/Blog/Model/BlogPost.php#L891
https://github.com/baserproject/basercms/blob/dev-4/lib/Baser/Plugin/Blog/Test/Case/View/Helper/BlogHelperTest.php#L104
https://github.com/baserproject/basercms/blob/dev-4/lib/Baser/Plugin/Mail/View/Helper/MailHelper.php#L52
https://github.com/baserproject/basercms/blob/dev-4/lib/Baser/Test/Case/Model/BcAppTest.php#L751

関係しているリビジョン

リビジョン b312162d
Masaharu Takishita が約1ヶ月前に追加

fix #22281 BcAppModel::expects()がテストモック関連のメソッド名と同名でテスト作成に問題が出るためリネーム (#954)

  • BcAppModel::expects()のメソッド名がスタブ設定のメソッド名と被っていてモック作成ができないためreduceAssociations()にリネーム
  • fix #22284 BcAppModel::expects() にて、再帰的な bind 定義を行うとエラーになる問題を改善

履歴

#1 滝下 真玄 が3ヶ月前に更新

  • ステータス を 担当 から 解決 に変更
  • 担当者 を 滝下 真玄 にセット
  • 進捗率 を 0 から 100 に変更

#2 江頭 竜二 が3ヶ月前に更新

  • 対象バージョン を 4.2.0 から 4.1.3 に変更

#3 江頭 竜二 が3ヶ月前に更新

  • ステータス を 解決 から 終了 に変更

他の形式にエクスポート: Atom PDF