Buy Access to Course
18.

Describing for Exception Messages

Share this awesome video!

|

Keep on Learning!

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

Login Subscribe

In the Exception/ directory that we copied a few minutes ago, there's another exception class: the DinosaursAreRunningRampantException.

// ... lines 1 - 4
final class DinosaursAreRunningRampantException extends \Exception
{
}

Here's the problem we're facing: we have these enclosures, but... they don't have any security - no electric fences, no guard towers, nothing! We need to add that capability to enclosures and throw this new exception if we try to add a Dinosaur to an Enclosure that has no active security. Because... honestly... we're having a real problem where people add dinosaurs to an enclosure and then just leave the door wide open.

In EnclosureSpec, let's create a new example to describe this: it_should_not_allow_to_add_dinosaurs_to_unsecure_enclosures(). I want you to temporarily ignore all the other examples that we've been working on so far, because this example is going to temporarily break... all of them.

48 lines | spec/Entity/EnclosureSpec.php
// ... lines 1 - 11
class EnclosureSpec extends ObjectBehavior
{
// ... lines 14 - 40
function it_should_not_allow_to_add_dinosaurs_to_unsecure_enclosures()
{
// ... lines 43 - 45
}
}

First... how does "security" for our enclosures need to be designed? Is it a boolean property on Enclosure so we can just turn security on or off? Something more complex? Actually, right now, it doesn't matter!

Check it out: in this example, I want to describe that if you simply create a new Enclosure, that is not enough: it's not secure. Describe that by saying $this->shouldThrow(). And this time, instead of passing the class name of the exception that should be thrown, I'll say new DinosaursAreRunningRampantException() and pass this a message: Are you craaazy?!?.

48 lines | spec/Entity/EnclosureSpec.php
// ... lines 1 - 40
function it_should_not_allow_to_add_dinosaurs_to_unsecure_enclosures()
{
$this
->shouldThrow(new DinosaursAreRunningRampantException('Are you craaazy?!?'))
// ... line 45
}
// ... lines 47 - 48

Why am I doing this differently than before? It's really up to you: you can pass the class name to shouldThrow() if you only need to make sure the exception is an instance of that class or if you want to make sure that the message is also correct, you can create the exception object with the message you expect.

Next, the exception should be thrown ->duringAddDinosaur(new Dinosaur()) with Velociraptor and true.

48 lines | spec/Entity/EnclosureSpec.php
// ... lines 1 - 40
function it_should_not_allow_to_add_dinosaurs_to_unsecure_enclosures()
{
$this
// ... line 44
->duringAddDinosaur(new Dinosaur('Velociraptor', true));
}
// ... lines 47 - 48

Oh, and this language is also a bit different than before. Earlier, we used during() and passed addDinosaur as an argument to that method. That's fine, but you can also use this more magical way: duringAddDinosaur(). It's a bit more natural because you can then pass each argument one-by-one, instead of putting them in an array. They do the same thing - so it's up to you.

The point is: we now have a test that describes that you can't just create an Enclosure and start putting dinosaurs into it. Somehow, security needs to be activated... whatever that means.

Let's move over and run phpspec:

./vendor/bin/phpspec run

Awesome! That does fail because it is still possible to add dinosaurs to enclosures without activating security.

Designing the Security

Ok... so how do we need enclosure security to work? The right answer depends on your dinosaur park. But the process is universal: think about what requirements you have. Is security just something you turn on or off - like with some activateSecurity() method on Enclosure? Or, is it more complex? Based on talking to our security experts, I've determined that we need the ability to add different types of security to different Enclosures - electric fences around some, guard towers around others and maybe just a sign that says "Please stay inside" if we get really busy. Oh, and each Enclosure can have 0 or many pieces of security.

Back in the tutorial/ directory, check out the Entity/ directory. See that Security class? Copy that and put into our src/Entity/ folder. Not spec/Entity, I'm totally messing this up right now... and will pay for it later.

25 lines | src/Entity/Security.php
// ... lines 1 - 4
class Security
{
private $name;
private $isActive;
private $enclosure;
public function __construct(string $name, bool $isActive, Enclosure $enclosure)
{
$this->name = $name;
$this->isActive = $isActive;
$this->enclosure = $enclosure;
}
public function getIsActive(): bool
{
return $this->isActive;
}
}

And, yes, yes, we're going to cheat a bit: we're going to skip the spec process for the Security class and start with something I've already created.

Each Security has a name - like "electric fence" or "guard tower" and a boolean for whether it's active or not. And we pass in the Enclosure that this Security will be attached to. For the methods - just one: getIsActive().

To get the example in EnclosureSpec to pass, somehow, we need a way to attach Security objects to our Enclosure class. And then, when we add a dinosaur, we can check to make sure the Enclosure has at least one active Security.

Ok... cool! To hold the securities, let's create a $securities property and set it to an empty array. This will be an array of Security objects, so let's document that.

50 lines | src/Entity/Enclosure.php
// ... lines 1 - 7
class Enclosure
{
// ... lines 10 - 12
/** @var Security[] */
private $securities = [];
// ... lines 15 - 48
}

Now, this is interesting. If we're adding a securities property, shouldn't we describe this more directly with some examples that show... I don't know... some addSecurity() or getSecurities() methods? Well... maybe? Maybe because... we might not need these methods! Right now, what we do know is that, if there are no active securities, an exception should be thrown. And of course, we will need to update some of our examples from earlier once we get this working so that they also have some active security.

Anyways, down in addDinosaur(), let's call another new method if (!$this->isSecurityActive()) we will throw a new DinosaursAreRunningRampantException() and pass it the same message that we described in our example - because we're testing for this exact string.

50 lines | src/Entity/Enclosure.php
// ... lines 1 - 20
public function addDinosaur(Dinosaur $dinosaur)
{
if (!$this->isSecurityActive()) {
throw new DinosaursAreRunningRampantException('Are you craaazy?!?');
}
// ... lines 26 - 31
}
// ... lines 33 - 50

In reality, this is a bit silly. In real life, I probably wouldn't care enough to test for that exact message - the class is enough.

To add the missing method, I'll put my cursor on the method name, hit Alt + Enter and click "Add Method". Cool! This will return a bool and inside, we can loop over the $securities with $this->securities as $security. If at least one Security object attached to this enclosure is active, then our enclosure is secure. So if ($security->getIsActive()), then return true. And if none of them are active, return false.

50 lines | src/Entity/Enclosure.php
// ... lines 1 - 38
private function isSecurityActive(): bool
{
foreach ($this->securities as $security) {
if ($security->getIsActive()) {
return true;
}
}
return false;
}
// ... lines 49 - 50

Okay, that should work! Move back to the terminal. Oh, see this 41? That means the example lives on line 41 of the spec class. Re-run phpspec:

./vendor/bin/phpspec run

It works! Sort of. Notice, line 41 is gone - that example is passing! By the way, instead of running all your spec classes, you can run just one by passing the filename to the command:

./vendor/bin/phpspec run spec/Entity/EnclosureSpec.php

Or, you can run just one example by adding colon then the line number. The example we're working on should be line 41 - yep! There it is. Try it:

./vendor/bin/phpspec run spec/Entity/EnclosureSpec.php:41

Cool! 1 passed. But if we run all of them, we have a few failures.

We made a few changes to our app that broke our existing examples. Next, let's think about the correct way to handle this and add a few more nice features to our Enclosure... including testing exceptions that happen during object construction.