問題点 #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