Test::FailWarnings で警告が出て失敗するようになったテストを修正する

Description

Test::FailWarnings を導入した結果、一部のテストがこけるようになった。コードフリーズ間際だったため、現在は抜本的な対策を取らず、いくつかのテストで一部のモジュールから出る警告を無視したり、テストそのものを飛ばすような回避策を入れている。が、特に Cloud.pack 関係のテストについては本来は警告が出る理由を調べて適切な箇所を直した方がよさそう。現在一時的な対策を入れているテストは下記の通り。

  • addons/Cloud.pack/t/cms/urls.t

  • addons/Cloud.pack/t/cms/cms-website.t

  • addons/Enterprise.pack/t/01-wizard.t

  • t/cms_permission/110-cms-permission-website.t (Cloud.pack 有効時のみ)

  • t/cms_permission/missing_permission.t (Cloud.pack 有効時のみ)

Checklists

None

Activity

Show:

Kenichi IshigakiSeptember 6, 2024 at 2:31 AM

基本的にテストのみの修正で対応済み。LDAP まわりに残っていた警告については https://movabletype.atlassian.net/browse/MTC-12127 の修正で直っている。完了したものとしてクローズする

Kenichi IshigakiAugust 15, 2024 at 6:57 AM

  • addons/Cloud.pack/t/cms/cms-website.t はどうも一部の(ignore_wwwがらみの)テストが腐っていそう。いきなり post しているのでどのページから飛ぶことを想定していたのかがはっきりしないが、全般設定(cfg_prefs)からであればすでに ignore_www_site_url は存在しないし、 ウェブサーバー設定のドメイン設定経由であれば site_url や enable_archive_paths が存在しない。もともとのテストは https://github.com/movabletype/movabletype-addons/pull/62/files で追加されたもの。課題がリンクされていないので本来どうしたかったのかは不明だが、 ignore_www_archive_url というパラメータは MT6.5 の時代までのもので、MT7 以降テンプレートには存在しなくなっているので整理してデッドコードは消すようにした方がよさそう。

  • addons/Enterprise.pack/t/01-wizard.t もそもそもテストがまともに動作していない(古い _run_app を使っているので気づかれずに来ていたが、後半のステップではエラーメッセージが出ている)。適切な修正を入れるにはまず https://movabletype.atlassian.net/browse/MTC-29685 で LDAP 環境を自前で起動できるようにする必要がありそう。

  • それ以外のテストについてはテストコードと一部のモジュールの修正で直りそう。

Implemented

Details

Assignee

Reporter

Priority

Smart Checklist

Created August 13, 2024 at 4:42 AM
Updated September 6, 2024 at 2:31 AM
Resolved September 6, 2024 at 2:31 AM