Flag of Ukraine
SymfonyCasts stands united with the people of Ukraine
COURSE

PHPUnit (Legacy): Testing with a Bite

Buy Access to Course
This tutorial has a new version, check it out!

Refactoring & Dependency Injection

Keep on Learning!

If you liked what you've learned so far, dive in!
Subscribe to get access to this tutorial plus
video, code and script downloads.

Start your All-Access Pass
Buy just this tutorial for $10.00
Previous Chapter

Next Chapter
Edit on GitHub

With a Subscription, click any sentence in the script to jump to that part of the video!

Login Subscribe

In DinosaurFactory, we made a fancy private function getLengthFromSpecification. It's a great function. So great, that now, I want to be able to use it from outside this class.

To do that... and of course... to help us get to mocking, let's refactor this method into its own class. Create a new Service directory in AppBundle. And then a new class called DinosaurLengthDeterminator. That's a fun name.

Copy getLengthFromSpecification(), remove it, and paste it here. Make the method public and re-type Dinosaur to get the use statement.

34 lines src/AppBundle/Service/DinosaurLengthDeterminator.php
... lines 1 - 4
use AppBundle\Entity\Dinosaur;
... line 6
class DinosaurLengthDeterminator
{
public function getLengthFromSpecification(string $specification): int
{
$availableLengths = [
'huge' => ['min' => Dinosaur::HUGE, 'max' => 100],
'omg' => ['min' => Dinosaur::HUGE, 'max' => 100],
'?' => ['min' => Dinosaur::HUGE, 'max' => 100],
'large' => ['min' => Dinosaur::LARGE, 'max' => Dinosaur::HUGE - 1],
];
$minLength = 1;
$maxLength = Dinosaur::LARGE - 1;
foreach (explode(' ', $specification) as $keyword) {
$keyword = strtolower($keyword);
if (array_key_exists($keyword, $availableLengths)) {
$minLength = $availableLengths[$keyword]['min'];
$maxLength = $availableLengths[$keyword]['max'];
break;
}
}
return random_int($minLength, $maxLength);
}
}

We already have a bunch of tests in DinosaurFactoryTest that make sure each specification string gives us the right length. In tests, create that same Service directory and a new DinosaurLengthDeterminatorTest. We're going to migrate the existing length tests to this class.

Add public function testItReturnsCorrectLengthRange() with $spec, $minExpectedSize and $maxExpectedSize.

34 lines tests/AppBundle/Service/DinosaurLengthDeterminatorTest.php
... lines 1 - 8
class DinosaurLengthDeterminatorTest extends TestCase
{
... lines 11 - 13
public function testItReturnsCorrectLengthRange($spec, $minExpectedSize, $maxExpectedSize)
{
... lines 16 - 20
}
... lines 22 - 34

This test will be similar to the one in DinosaurFactoryTest, but a bit simpler: it only needs to test the length. Create a new determinator. And then set $actualSize to $determinator->getLengthFromSpecification($spec).

34 lines tests/AppBundle/Service/DinosaurLengthDeterminatorTest.php
... lines 1 - 8
class DinosaurLengthDeterminatorTest extends TestCase
{
... lines 11 - 13
public function testItReturnsCorrectLengthRange($spec, $minExpectedSize, $maxExpectedSize)
{
$determinator = new DinosaurLengthDeterminator();
$actualSize = $determinator->getLengthFromSpecification($spec);
... lines 18 - 20
}
... lines 22 - 34

To make sure this is within the range, add $this->assertGreaterThanOrEqual(). Oh wait! No auto-completion! Bah! Extend TestCase!

34 lines tests/AppBundle/Service/DinosaurLengthDeterminatorTest.php
... lines 1 - 8
class DinosaurLengthDeterminatorTest extends TestCase
{
... lines 11 - 34

Now add $this->assertGreaterThanOrEqual() with $minExpectedSize and $actualSize. You need to read this... backwards: this asserts that $actualSize is greater than or equal to $maxExpectedSize.

Repeat that with $this->assertLessThanOrEqual() and $maxExpectedSize, $actualSize.

34 lines tests/AppBundle/Service/DinosaurLengthDeterminatorTest.php
... lines 1 - 8
class DinosaurLengthDeterminatorTest extends TestCase
{
... lines 11 - 13
public function testItReturnsCorrectLengthRange($spec, $minExpectedSize, $maxExpectedSize)
{
... lines 16 - 18
$this->assertGreaterThanOrEqual($minExpectedSize, $actualSize);
$this->assertLessThanOrEqual($maxExpectedSize, $actualSize);
}
... lines 22 - 34

Adding the Length Data Provider

The real work is done in the data provider. Add public function getSpecLengthTests(). If you look at DinosaurFactoryTest, the getSpecificationTests() already has great examples. Copy those, go back to the new test, and paste. We need almost the same thing: just change the comments to specification, min length and max length.

Then, for a large dinosaur, it should be between Dinosaur::LARGE and DINOSAUR::HUGE - 1. For a small dinosaur, the range is 0 to Dinosaur::LARGE - 1. Copy the large dino range and use that for the last one too,

38 lines tests/AppBundle/Service/DinosaurLengthDeterminatorTest.php
... lines 1 - 8
class DinosaurLengthDeterminatorTest extends TestCase
{
... lines 11 - 22
public function getSpecLengthTests()
{
return [
// specification, min length, max length
['large carnivorous dinosaur', Dinosaur::LARGE, Dinosaur::HUGE - 1],
'default response' => ['give me all the cookies!!!', 0, Dinosaur::LARGE - 1],
['large herbivore', Dinosaur::LARGE, Dinosaur::HUGE - 1],
... lines 30 - 34
];
}
}

We can also move the huge dinosaur tests here. Copy them, move back, and paste! This time, the range should be Dinosaur::HUGE to 100. Copy that and use it for all of them.

38 lines tests/AppBundle/Service/DinosaurLengthDeterminatorTest.php
... lines 1 - 8
class DinosaurLengthDeterminatorTest extends TestCase
{
... lines 11 - 22
public function getSpecLengthTests()
{
return [
... lines 26 - 29
['huge dinosaur', Dinosaur::HUGE, 100],
['huge dino', Dinosaur::HUGE, 100],
['huge', Dinosaur::HUGE, 100],
['OMG', Dinosaur::HUGE, 100],
['?', Dinosaur::HUGE, 100],
];
}
}

And finally, hook this all up with @dataProvider getSpecLengthTests(). I'll even fix my typo!

34 lines tests/AppBundle/Service/DinosaurLengthDeterminatorTest.php
... lines 1 - 8
class DinosaurLengthDeterminatorTest extends TestCase
{
/**
* @dataProvider getSpecLengthTests
*/
public function testItReturnsCorrectLengthRange($spec, $minExpectedSize, $maxExpectedSize)
{
... lines 16 - 20
}
... lines 22 - 34

Perfect! Because we deleted some code, the DinosaurFactory is temporarily broken. So let's execute just this test:

./vendor/bin/phpunit tests/AppBundle/Service/DinosaurLengthDeterminatorTest.php

It passes!

Refactoring to use a Dependency

Time to fix the factory and get those dinosaurs growing again! The getLengthFromSpecification() method is gone. To use the new determinator class, use dependency injection: add public function __construct() with a DinosaurLengthDeterminator argument. I'll press alt+enter and select "Initialize fields" as a shortcut to create the property and set it below.

47 lines src/AppBundle/Factory/DinosaurFactory.php
... lines 1 - 7
class DinosaurFactory
{
private $lengthDeterminator;
public function __construct(DinosaurLengthDeterminator $lengthDeterminator)
{
$this->lengthDeterminator = $lengthDeterminator;
}
... lines 16 - 45
}

Back in growFromSpecification(), use $this->lengthDeterminator to call the method.

47 lines src/AppBundle/Factory/DinosaurFactory.php
... lines 1 - 7
class DinosaurFactory
{
... lines 10 - 21
public function growFromSpecification(string $specification): Dinosaur
{
... lines 24 - 25
$length = $this->lengthDeterminator->getLengthFromSpecification($specification);
... lines 27 - 35
}
... lines 37 - 45
}

And... that's it! We haven't run the tests yet, but this class now probably works again. You need to pass in a dependency, but as long as you do that, it's all basically the same.

Run all the tests:

./vendor/bin/phpunit

Woh! That is a lot of failures - all with the same message:

Too few arguments passed to DinosaurFactory::__construct() on DinosaurFactoryTest line 18.

Scroll up to line 18. Of course: we're missing the constructor argument to the factory in our test.

For the second time, we have dependent objects. The object we're testing - DinosaurFactory - is dependent on another object - DinosaurLengthDeterminator. We're going to fix this with a mock.

Leave a comment!

Most Useful Newest Oldest
4
Login or Register to join the conversation
danielchadwick Avatar

Emojis are breaking the code display. See anything including the size specification function.

Reply
MolloKhan Avatar

Hey @Daniel

Thanks for reporting it to us. I've already fixed

Cheers!

Reply
Default user avatar
Default user avatar toporovvv | posted 5 years ago

Two typos here: mixExpectedSize and mizLength.

Reply
Victor Avatar

Hey toporovvv,

Thanks for reporting it, nice catch! Now it's fixed in https://github.com/knpunive...

Cheers!

1 Reply
Cat in space

"Houston: no signs of life"
Start the conversation!

While the fundamentals of PHPUnit haven't changed, this tutorial *is* built on an older version of Symfony and PHPUnit.
Symfony 3 PHPUnit 6

What PHP libraries does this tutorial use?

// composer.json
{
    "require": {
        "php": "^7.0, <7.4",
        "composer/package-versions-deprecated": "^1.11", // 1.11.99
        "doctrine/doctrine-bundle": "^1.6", // 1.10.3
        "doctrine/orm": "^2.5", // v2.7.2
        "incenteev/composer-parameter-handler": "^2.0", // v2.1.2
        "sensio/distribution-bundle": "^5.0.19", // v5.0.21
        "sensio/framework-extra-bundle": "^3.0.2", // v3.0.28
        "symfony/monolog-bundle": "^3.1.0", // v3.1.2
        "symfony/polyfill-apcu": "^1.0", // v1.6.0
        "symfony/swiftmailer-bundle": "^2.3.10", // v2.6.7
        "symfony/symfony": "3.3.*", // v3.3.13
        "twig/twig": "^1.0||^2.0" // v2.4.4
    },
    "require-dev": {
        "doctrine/data-fixtures": "^1.3", // 1.3.3
        "doctrine/doctrine-fixtures-bundle": "^2.3", // v2.4.1
        "liip/functional-test-bundle": "^1.8", // 1.8.0
        "phpunit/phpunit": "^6.3", // 6.5.2
        "sensio/generator-bundle": "^3.0", // v3.1.6
        "symfony/phpunit-bridge": "^3.0" // v3.4.30
    }
}
Edit on GitHub
Previous Chapter
Next Chapter
userVoice