Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP unit tests fail using PHP >=7.2 #779

Closed
paddatrapper opened this issue Mar 28, 2019 · 3 comments
Closed

PHP unit tests fail using PHP >=7.2 #779

paddatrapper opened this issue Mar 28, 2019 · 3 comments
Labels
ci This is affecting the ci is: bug php status: pinned This issue or pull request won't stale

Comments

@paddatrapper
Copy link
Contributor

Describe the bug
When running the PHP unit tests using PHP 7.2 or PHP 7.3, they fail due to the following error:

PHP Fatal error:  Uncaught Zend_Session_Exception: Session must be started before any output has been sent to the browser; output started in /home/travis/build/LibreTime/libretime/vendor/phpunit/phpunit/src/Util/Getopt.php/38 in /home/travis/build/LibreTime/libretime/vendor/zf1s/zend-session/library/Zend/Session.php:456
Stack trace:
#0 /home/travis/build/LibreTime/libretime/airtime_mvc/tests/application/bootstrap.php(82): Zend_Session::start()
#1 /home/travis/build/LibreTime/libretime/vendor/phpunit/phpunit/src/Util/Fileloader.php(56): include_once('/home/travis/bu...')
#2 /home/travis/build/LibreTime/libretime/vendor/phpunit/phpunit/src/Util/Fileloader.php(38): PHPUnit_Util_Fileloader::load('/home/travis/bu...')
#3 /home/travis/build/LibreTime/libretime/vendor/phpunit/phpunit/src/TextUI/Command.php(779): PHPUnit_Util_Fileloader::checkAndLoad('/home/travis/bu...')
#4 /home/travis/build/LibreTime/libretime/vendor/phpunit/phpunit/src/TextUI/Command.php(598): PHPUnit_TextUI_Command->handleBootstrap('/home/travis/bu...')
#5 /home/ in /home/travis/build/LibreTime/libretime/vendor/zf1s/zend-session/library/Zend/Session.php on line 456

For full logs, look at the latest build for master on Travis CI.

Expected behavior
The tests pass

LibreTime version:
master

Installation method:

  • Method: Travis CI
@paddatrapper paddatrapper added is: bug php ci This is affecting the ci labels Mar 28, 2019
@Robbt Robbt added this to High priority in Bug Triage Board Apr 23, 2019
@Robbt Robbt moved this from High priority to Needs triage in Bug Triage Board Apr 23, 2019
@Robbt
Copy link
Member

Robbt commented Apr 24, 2019

I found someone who had the same problem and put together a fix - https://discourse.zendframework.com/t/zend-session-error-with-phpunit-and-php-7-2-but-not-7-1/712/2

@Robbt Robbt moved this from Needs triage to Low priority in Bug Triage Board Apr 24, 2019
@stale
Copy link

stale bot commented Oct 20, 2019

This issue has been automatically marked as stale because it has not had activity in the last 5 months. It will be closed if no activity occurs in the next month.
Please chat to us on discourse or ask for help on our chat if you have any questions or need further support with getting this issue resolved.
You may also label an issue as pinned if you would like to make sure that it does not get closed by this bot.

@stale stale bot added the status: stalled This issue or pull request is stalled label Oct 20, 2019
@stale stale bot removed the status: stalled This issue or pull request is stalled label Oct 21, 2019
@jooola jooola added status: pinned This issue or pull request won't stale and removed pinned labels Sep 18, 2021
@jooola
Copy link
Contributor

jooola commented Oct 5, 2021

This can be closed right ?

Bug Triage Board automation moved this from Low priority to Closed Oct 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci This is affecting the ci is: bug php status: pinned This issue or pull request won't stale
Projects
Bug Triage Board
  
Closed
Development

No branches or pull requests

3 participants