-
Notifications
You must be signed in to change notification settings - Fork 131
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ユニットテスト BlogPostsTable::beforeSave #3087
ユニットテスト BlogPostsTable::beforeSave #3087
Conversation
@thangnnmd コードを再フォーマットして下さい。 // format lại code đi a |
@HungDV2022 fromat code and add some comment |
@ryuring レピューをお願いします。 |
@thangnnmd @HungDV2022
|
ご返信ありがとうございます、来週修正させていただきます
Vào Th 7, 17 thg 2, 2024 lúc 10:01 ryuring ***@***.***> đã
viết:
… Assigned #3087 <#3087> to
@thangnnmd <https://github.com/thangnnmd>.
—
Reply to this email directly, view it on GitHub
<#3087 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BD7D3GJKHXRSVNOXREIKX4DYUAMP3AVCNFSM6AAAAABC3MSWJSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRHAZTKMZUGI2DMMI>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
@HungDV2022 $this->BlogPostsTable->isExcluded(); |
@ryuring レピューをお願いします。 |
@HungDV2022 @thangnnmd シナリオを使ってもう少しコードを短くできないでしょうか? |
@HungDV2022 add BlogPostsScenario.class, please see it |
@thangnnmd 新しいシナリオを作成ではなく既存シナリオを使った方がいいと思います。//a dùng cái Scenario cũ nhá, thiếu phần nào bổ sung = Factory ý. |
@HungDV2022 Is this solution better? |
#3087 ユニットテスト調整 |
@ryuring コードが大分減らしました。確認お願いします。 |
@thangnnmd @HungDV2022 だいぶスッキリしましたね。ありがとうございます。 |
No description provided.