Skip to content
This repository was archived by the owner on Aug 24, 2021. It is now read-only.

Conversation

@royopa
Copy link
Contributor

@royopa royopa commented Nov 13, 2017

This test was not passing because the VM called by travis CI is slow, see below the failure:
http://gcov.php.net/viewer.php?version=PHP_HEAD&func=tests&file=ext%2Fzlib%2Ftests%2Finflate_add_basic.phpt
So I included the INI section with max_execution_time=10 and the test passed:

Running selected tests.
PASS Test incremental inflate_add() functionality [/usr/src/phpt/inflate_add_basic.phpt]

Number of tests : 1 1
Tests skipped : 0 ( 0.0%) --------
Tests warned : 0 ( 0.0%) ( 0.0%)
Tests failed : 0 ( 0.0%) ( 0.0%)
Expected fail : 0 ( 0.0%) ( 0.0%)
Tests passed : 1 (100.0%) (100.0%)

User Group: PHPSP #phptestfestbrasil
http://phpsp.org.br/

This test was not passing because the VM called by travis CI is slow, see below the failure:
http://gcov.php.net/viewer.php?version=PHP_HEAD&func=tests&file=ext%2Fzlib%2Ftests%2Finflate_add_basic.phpt
So I included the INI section with max_execution_time=10 and the test passed:
=====================================================================
Running selected tests.
PASS Test incremental inflate_add() functionality [/usr/src/phpt/inflate_add_basic.phpt] 
=====================================================================
Number of tests :    1                 1
Tests skipped   :    0 (  0.0%) --------
Tests warned    :    0 (  0.0%) (  0.0%)
Tests failed    :    0 (  0.0%) (  0.0%)
Expected fail   :    0 (  0.0%) (  0.0%)
Tests passed    :    1 (100.0%) (100.0%)

User Group: PHPSP #phptestfestbrasil
http://phpsp.org.br/
@krakjoe
Copy link
Contributor

krakjoe commented Nov 16, 2017

In CLI, max_execution_time is set to 0 by default ... do we know if travis is changing the default ?

@rogeriopradoj
Copy link

#107 (comment)

In CLI, max_execution_time is set to 0 by default ... do we know if travis is changing the default ?

I started digging into Travis ecosystem trying to find some explanation, I guess these sources could be a start: one, two

@petk petk closed this May 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants