Project

General

Profile

Backport(バックポート) #3637

テストケースをSQLite等のDBMS上で動作させることができない

Added by 誠二 天重 about 5 years ago. Updated about 5 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2011-03-29
Due date:
% Done:

100%


Description

現象

php5 test/unit/model/doctrine/MemberTest.php などのようにテストケースを実行すると、下記のようなエラーが出力されて中断する。

PHP Fatal error:  Uncaught exception 'Doctrine_Connection_Sqlite_Exception' with message 'SQLSTATE[23000]: Integrity constraint violation: 19 activity_data.body may not be NULL' in /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php:1082
Stack trace:
#0 /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection/Statement.php(269): Doctrine_Connection->rethrowException(Object(PDOException), Object(Doctrine_Connection_Statement))
#1 /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php(1042): Doctrine_Connection_Statement->execute(Array)
#2 /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php(687): Doctrine_Connection->exec('INSERT INTO act...', Array)
#3 /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctr in /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php on line 1082

Fatal error: Uncaught exception 'Doctrine_Connection_Sqlite_Exception' with message 'SQLSTATE[23000]: Integrity constraint violation: 19 activity_data.body may not be NULL' in /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php:1082
Stack trace:
#0 /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection/Statement.php(269): Doctrine_Connection->rethrowException(Object(PDOException), Object(Doctrine_Connection_Statement))
#1 /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php(1042): Doctrine_Connection_Statement->execute(Array)
#2 /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php(687): Doctrine_Connection->exec('INSERT INTO act...', Array)
#3 /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctr in /home/upsilon/git/openpne3/sqlite/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php on line 1082

原因

test/fixtures/test_data.yml の中にActivityDataのbodyの値が設定されていないレコードが含まれている。

修正内容

dummy_activity7 と dummy_activity8 に不足している値を追加する。

#1527 に対する修正によって再度発生した問題

現象

php5 test/unit/model/doctrine/MemberTest.php などのようにテストケースを実行すると、下記のようなエラーが出力されて中断する。

PHP Fatal error:  Uncaught exception 'Doctrine_Connection_Sqlite_Exception' with message 'SQLSTATE[HY000]: General error: 1 near "SET": syntax error. Failing Query: "SET FOREIGN_KEY_CHECKS = 0"' in /home/upsilon/git/openpne3/houou/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php:1082
Stack trace:
#0 /home/upsilon/git/openpne3/houou/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php(1061): Doctrine_Connection->rethrowException(Object(PDOException), Object(Doctrine_Connection_Sqlite_ExtraFunctions), 'SET FOREIGN_KEY...')
#1 /home/upsilon/git/openpne3/houou/test/bootstrap/database.php(15): Doctrine_Connection->exec('SET FOREIGN_KEY...')
#2 /home/upsilon/git/openpne3/houou/test/unit/model/doctrine/MemberTest.php(4): include_once('/home/upsilon/g...')
#3 {main}
  thrown in /home/upsilon/git/openpne3/houou/lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Connection.php on line 1082

原因

6e7de531 によって test/bootstrap/database.php に追加された、

$conn = opDoctrineQuery::getMasterConnectionDirect();
$conn->exec('SET FOREIGN_KEY_CHECKS = 0');

のコードからエラーが発生している。ここで実行されている SET FOREIGN_KEY_CHECKS = 0 はMySQLに依存するクエリであるため、MySQL以外のDBMS上では実行に失敗しエラーが発生する。

修正内容

上記のコードを、DBMSがMySQLである場合のみ実行するように修正する。


Related issues

Copied from OpenPNE 3 - Bug(バグ) #1984: テストケースをSQLite等のDBMS上で動作させることができない Fixed(完了) 2011-03-29

Associated revisions

Revision 267bc856 (diff)
Added by isao sano about 5 years ago

avoid incompatible query when not using MySQL (fixes #3637, BP from #1984)

Revision 09b59a9a
Added by Shinichi Urabe about 5 years ago

Merge pull request #151 from isaosano/T-3637

avoid incompatible query when not using MySQL (fixes #3637, BP from #1984)

History

#1 Updated by 誠二 天重 about 5 years ago

  • Copied from Bug(バグ) #1984: テストケースをSQLite等のDBMS上で動作させることができない added

#2 Updated by isao sano about 5 years ago

https://github.com/openpne/OpenPNE3/pull/150
にて修正をプルリクしました。

019a6ce780fb636795dab151fc4ef679f4aca244 の修正に関しては、stable-3.8.xブランチ作成時以前の修正なのですでに取り込まれています。

#3 Updated by isao sano about 5 years ago

https://github.com/openpne/OpenPNE3/pull/151
にて修正をプルリクしました。

note1のプルリクエストはリクエスト先をmasterにしてしまったので取り下げました

#4 Updated by isao sano about 5 years ago

  • Status changed from New(新規) to Accepted(着手)

#5 Updated by isao sano about 5 years ago

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

#6 Updated by isao sano about 5 years ago

  • Assignee set to isao sano

#7 Updated by Shinichi Urabe about 5 years ago

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

レビューOKです

#8 Updated by 誠二 天重 about 5 years ago

トピックブランチでの単体試験完了しました。
sqliteおよびmysqlでテストケースを実行、特に問題ありません。
stable-3.8.xでの結合試験完了まではステータスはテスト待ちのままになります。

#10 Updated by isao sano about 5 years ago

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

更新履歴 267bc856188e92f953abd21f61df64331946749b で適用されました。

#11 Updated by Shinichi Urabe about 5 years ago

更新履歴 09b59a9a0ca6525bd78ead23f24acfbf65bc9013 で適用されました。

#12 Updated by Shinichi Urabe about 5 years ago

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

レビュー済み、取り込み済みなので、ステータスを変更します

#13 Updated by isao sano about 5 years ago

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

結合試験完了しました。
問題ありませんのでステータスを「Fixed」いたします。

Also available in: Atom PDF