開発環境を更新

[drill-lancer]

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

• 開発環境が古かったので更新

どういう変更をしたか？

• npm scripts を最新版に更新
• Git Hub Action で Composer の設定が引っかる様になったので修正
• Node,js 管理ツール Volta に対応 ( .node-version は 14 系の最新版に )
• 環境を更新したらテストに不備があり通らなくなったのでそれも更新
• vk-wp-unit-test-tools におけるものはそちらで対応 ( https://github.com/vektor-inc/vk-wp-unit-test-tools/pull/1 )

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

• [x] 複数の意図の変更 （ 機能の不具合修正 + 別の機能追加など ） を含んでいないか？
• [x] Files changed (変更ファイル)の内容は目視で確認したか？
• [ ] readme.txt に変更内容は書いたか？ => 開発環境を更新したのみなので不要かと
• [x] 本当にちゃんと確認をしたか？

プログラムの変更の場合

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

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

• npm install が問題なく動くか
• composer install が問題なく動くか
• npm run build が問題なく動くか
• npx wp-env start が問題なく動くか
• vektor-inc/vk-wp-unit-test-tools の dev-fix/add-user と合わせて PHP Unit テストが通るか

以上を確認しました。

確認URL

ローカル環境にて

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

• npm install が問題なく動くか
• composer install が問題なく動くか
• npm run build が問題なく動くか
• npx wp-env start が問題なく動くか
• vektor-inc/vk-wp-unit-test-tools の dev-fix/add-user と合わせて PHP Unit テストが通るか
• その他必要なコマンドがあればそれについても

以上を確認お願いします。

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

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

---

レビュワー向け

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

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

[kurudrive]

@drill-lancer これ RICK さんの環境だとエラーにならない？

1) Test_PHP_Fatal_Error::test_run_php_fatal_error
Trying to get property 'display_name' of non-object

/var/www/html/wp-content/plugins/vk-all-in-one-expansion-unit/inc/article-structure-data/class-vk-article-structure-data.php:195
/var/www/html/wp-content/plugins/vk-all-in-one-expansion-unit/inc/article-structure-data/class-vk-article-structure-data.php:157
/var/www/html/wp-content/plugins/vk-all-in-one-expansion-unit/inc/article-structure-data/class-vk-article-structure-data.php:112
/var/www/html/wp-includes/class-wp-hook.php:308
/var/www/html/wp-includes/class-wp-hook.php:332
/var/www/html/wp-includes/plugin.php:517
/var/www/html/wp-includes/general-template.php:3043
/wordpress-phpunit/data/themedir1/default/index.php:3
/var/www/html/wp-content/plugins/vk-all-in-one-expansion-unit/vendor/vektor-inc/vk-wp-unit-test-tools/src/tests/test-php-fatal-error.php:210

ERRORS!
Tests: 35, Assertions: 241, Errors: 1.

[drill-lancer]

@kurudrive それは https://github.com/vektor-inc/vk-wp-unit-test-tools/pull/1 で修正しています。

[kurudrive]

@drill-lancer うーむ...いや、

"vektor-inc/vk-wp-unit-test-tools": "dev-fix/add-user",

で composer install して npm run phpunit すればいいんだよね？

[drill-lancer]

@kurudrive そのはずですが・・・ composer update か composer.lock を一旦消すとかそのような操作が必要かもしれません。

[kurudrive]

@drill-lancer うーむ、やっぱり解消しないなぁ... RICKさんの環境ではこれで通ってるんだよね？

[drill-lancer]

@kurudrive 通ったのを確認してからプルリクを出していますがもう一回確認してみます。

[drill-lancer]

@kurudrive composer からはなぜか dev-fix/add-user が古い状態で引っ張られてきますね。

[drill-lancer]

@kurudrive "vektor-inc/vk-wp-unit-test-tools": "dev-fix/add-user", の方調整しておきました。