vektor-inc / vk-all-in-one-expansion-unit

wordpress plugin of powerful support
https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/
GNU General Public License v2.0
7 stars 1 forks source link

【確認待ち】有効化設定の初期設定がブロックテーマの場合ブロックテーマ用の設定になるように #842

Closed drill-lancer closed 2 years ago

drill-lancer commented 2 years ago

チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)

https://github.com/vektor-inc/vk-all-in-one-expansion-unit/issues/835

どういう変更をしたか?

レビューに回す前に確認する事

プログラムの変更の場合

変更内容について何を確認したか、どういう方法で確認をしたかなど

確認URL

wp-env 環境で npx wp-env destroy 後に npx wp-env start を実行した環境

レビュワーの確認方法・確認する内容など

レビュワーに回す前の確認事項


レビュワー向け

確認して変更が反映されていない場合の確認事項

kurudrive commented 2 years ago

@drill-lancer ありがとうございます。 veu_get_packages() の PHPUnitテストを追加してください。

drill-lancer commented 2 years ago

@kurudrive veu_get_packages() はテスト結果の配列が長いので veu_get_common_options() か veu_get_common_options_default()あたりが妥当なように思えますがいかがでしょうか?

kurudrive commented 2 years ago

@drill-lancer veu_get_common_options() だとブロックテーマかそうじゃない時かの 挙動のテストにならなくないですか(・w・?

kurudrive commented 2 years ago

今回の変更は ブロックテーマの時とそうじゃない場合の初期で有効化される機能の切り替えなので、 それがテスト出来ればOKです。

drill-lancer commented 2 years ago

@kurudrive テスト追加しました

kurudrive commented 2 years ago

@drill-lancer あーりーがーとーうーございましたたたたーん!