Skip to content
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.

Sign up for GitHub

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

Jump to bottom

#4975 でのテストコードの不具合の修正 #5238

Merged
merged 1 commit into from
Nov 26, 2021
Merged

#4975 でのテストコードの不具合の修正 #5238

merged 1 commit into from
Nov 26, 2021

Conversation

matsuoshi
Copy link
Contributor

@matsuoshi matsuoshi commented Nov 26, 2021 •
edited by chihiro-adachi
Loading

概要(Overview・Refs Issue)

こちらの修正にて、ユニットテストが落ちてしまっていた件を修正しています
#4975

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

管理画面・タグ検索機能に関するユニットテストを修正しました

相談（Discussion）

マイナーバージョン互換性保持のための制限事項チェックリスト

  • 既存機能の仕様変更はありません
  • フックポイントの呼び出しタイミングの変更はありません
  • フックポイントのパラメータの削除・データ型の変更はありません
  • twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか

@matsuoshi matsuoshi force-pushed the feature/tag-search-admin branch from c9a156f to 3266b4c Compare November 26, 2021 01:38
@chihiro-adachi chihiro-adachi added this to the 4.1.1 milestone Nov 26, 2021
@chihiro-adachi chihiro-adachi merged commit 04627d4 into EC-CUBE:4.1 Nov 26, 2021
@chihiro-adachi
Copy link
Contributor

@matsuoshi
マージしました。

@chihiro-adachi chihiro-adachi mentioned this pull request Nov 26, 2021
Merged
11 tasks
@matsuoshi matsuoshi changed the title テストコードの修正 #4975 でのテストコードの不具合の修正 Dec 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug:Low
Projects
None yet
Milestone
4.1.1
Development

Successfully merging this pull request may close these issues.
2 participants
@matsuoshi @chihiro-adachi