[DependencyInjection] properly fix tests on PHP 5

[xabbuh]

Q	A
Branch?	3.4
Bug fix?	yes
New feature?	no
BC breaks?	no
Deprecations?	no
Tests pass?	yes
Fixed tickets
License	MIT
Doc PR

[chalasr]

Thank you @xabbuh.

[przemyslaw-bogusz]

While running Travis CI tests with PHP 5.5 (build 67951.2) for PR #29935 I got:

PHP Parse error: syntax error, unexpected ':', expecting ';' or '{' in /home/travis/build/symfony/symfony/src/Symfony/Component/DependencyInjection/Tests/Fixtures/FactoryDummy.php on line 16
Parse error: syntax error, unexpected ':', expecting ';' or '{' in /home/travis/build/symfony/symfony/src/Symfony/Component/DependencyInjection/Tests/Fixtures/FactoryDummy.php on line 16

I think FactoryReturnTypePassTest on branch 3.4 should check PHP version and skip the test if necessary, same as it does with hhvm. If that's true, I can make the relevant PR.

[xabbuh]

@przemyslaw-bogusz Can you link to part of the job you refer to (you can click on a line number on a left in the Travis CI output)? I do not find anything like that.

[przemyslaw-bogusz]

It seems that the test ran again just a few minutes ago and now the error is gone. Next time I will make a screenshot. Sorry for disturbing you.

[xabbuh]

Nothing to worry about. :) Just by reading the code I also don't see where the test would be skipped on PHP 5.5. But we seem to miss something as the test apparently passes right now. :)

[przemyslaw-bogusz]

As I understand this PR was about fixing tests due to the fact, that Symfony 3.4 can run on PHP 5, which means that tests from branch 3.4 should take into account, that return type declarations were introduced in PHP 7.

I thought that was why the test failed, as it hinted at : from the return type declaration.

symfony/src/Symfony/Component/DependencyInjection/Tests/Fixtures/FactoryDummy.php

Lines 16 to 18 in db6784b

	public static function createFactory(): FactoryDummy
	{
	}

But now, since suddenly the Travis CI test passes, I'm confused.

[przemyslaw-bogusz]

@xabbuh I wear glasses so that's the main problem. The failed Travis CI test is in #29944. Here's link to the test:
https://travis-ci.org/symfony/symfony/jobs/482202091

[xabbuh]

@przemyslaw-bogusz looks reasonable to me, could you do the PR?

[przemyslaw-bogusz]

@xabbuh I will do the PR

[przemyslaw-bogusz]

@xabbuh Could you please enlighten with one more thing? Since the problem resolves around using return type declarations in PHP 5 test environment, for the CI test to fail, it's enough that FactoryReturnTypePassTest has a use Symfony\Component\DependencyInjection\Tests\Fixtures\FactoryDummy;, which loads a class with a syntax error. Why Travis CI tests with PHP 5 for other PRs pass? Does it mean that not all PHPUnit tests are run on each PR?