Project

General

Profile

Backport(バックポート) #3471

zlib 拡張の入っていない環境で opDoctrineRecordTest.php のテストが実行できない

Added by Mutsumi Imamura almost 6 years ago. Updated almost 6 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2013-10-09
Due date:
% Done:

100%


Description

Overview (現象)

#3134 のテストのために追加された opDoctrineRecordTest.php には gzip タイプのカラムの値を破壊しないか確認するためのテストが含まれているが、このテストは zlib 拡張が有効な環境でないと実行できない。
source:doc/ja/OpenPNE3_Setup_Guide.txt@b9a08942 には OpenPNE の動作に最低限必須とされている拡張に zlib 拡張が含まれていないため、この拡張が有効でない環境でも gzip 関連のテストをスキップして動作するようにすべきである。

Way to fix (修正内容)

opDoctrineRecordTest 実行時に gzcompress の使用可否をチェックし、それに応じて gzip タイプのカラムに対するテストを行うか否かを分岐するように修正する。


Related issues

Related to OpenPNE 3 - Bug(バグ) #3447: zlib 拡張の入っていない環境で opDoctrineRecordTest.php のテストが実行できない Won't fix(対応せず) 2013-10-09

Associated revisions

Revision 9e083bd2 (diff)
Added by Youichi Kimura almost 6 years ago

skip the test for gzip type column if zlib extensions is missing (fixes
(cherry picked from commit 3743686e4c0cae0afa4d9d130fa3639f9aaf9488)

Revision b50aab8b
Added by Youichi Kimura almost 6 years ago

Merge branches 't-3418', 't-3420', 't-3422', 't-3459', 't-3460', 't-3461', 't-3462', 't-3463', 't-3465' and 't-3471' into stable-3.8.x

History

#1 Updated by Akihiro KOBAYASHI almost 6 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Akihiro KOBAYASHI

#2 Updated by Akihiro KOBAYASHI almost 6 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

pull request:https://github.com/openpne/OpenPNE3/pull/95

#3 Updated by Youichi Kimura almost 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#4 Updated by Mutsumi Imamura almost 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

./symfony test:all を実行し異常終了しないことを確認しました。
ブラウザからの動作テストも問題ありません。

Also available in: Atom PDF