【確認待ち】Playwright のテストが通るように

drill-lancer commented 1 year ago

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

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

どういう変更をしたか？

Playwright のテストを修正

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

[x] 複数の意図の変更（機能の不具合修正 + 別の機能追加など）を含んでいないか？
[x] Files changed (変更ファイル)の内容は目視で確認したか？
[ ] readme.txt に変更内容は書いたか？ ⇒テストを修正しただけなので省略
[x] 本当にちゃんと確認をしたか？

プログラムの変更の場合

[x] 書けそうなテストは書いたか？

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

npx wp-env clean && npx playwright test --trace on --project=chromium

確認URL

ローカル環境

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

npx wp-env clean && npx playwright test --trace on --project=chromium

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

[x] このテンプレートのチェック項目をちゃんと確認してチェックしたか？

レビュワー向け

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

プルしたか？
ビルドしたか？
ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
npm install したか？
composer install したか？

kurudrive commented 1 year ago

@drill-lancer ありがとうございます！あー、ブロックテーマでもデフォルトでCTAを有効にするようになったからか... ってあれ、それ結構前のはずなんだけどな...今までなんで通ってたんだ... https://github.com/vektor-inc/vk-all-in-one-expansion-unit/pull/967