Short story about design using tests
...basically is about how i do design using tests
Let's imagine that we need register user in our app...
yeah sophisticated task
How do you design register user feature?
Yay let's plan the database schema...
and use getters and setters everywhere
No, let's plan the domain first...
bin/phpspec desc "PHPBenelux\Domain\User"
What data are needed for User to use our system?
<?php
namespace spec\PHPBenelux\Domain;
use PhpSpec\ObjectBehavior;
class UserSpec extends ObjectBehavior
{
function it_has_email_and_password()
{
$this->setEmail('leszek.prabucki@gmail.com');
$this->email()->shouldBe('leszek.prabucki@gmail.com');
$hash = password_hash('secretPass', PASSWORD_BCRYPT);
$this->setPasswordHash();
$this->passwordHash()->shouldBe($hash);
}
}
Why not use constructor instead?
<?php
namespace spec\PHPBenelux\Domain;
use PhpSpec\ObjectBehavior;
class UserSpec extends ObjectBehavior
{
function it_has_email_and_password()
{
$hash = password_hash('secretPass', PASSWORD_BCRYPT);
$this->beConstructedWith('leszek.prabucki@gmail.com', $hash);
$this->email()->shouldBe('leszek.prabucki@gmail.com');
$this->passwordHash()->shouldBe($hash);
}
}
We can add Value Objects to protect invariants
<?php
namespace spec\PHPBenelux\Domain;
use PHPBenelux\Domain\Email;
use PHPBenelux\Domain\PasswordHash;
use PHPBenelux\Domain\UserId;
use PhpSpec\ObjectBehavior;
class UserSpec extends ObjectBehavior
{
function it_has_email_and_password()
{
$hash = password_hash('secretPass', PASSWORD_BCRYPT);
$this->beConstructedWith(
UserId::generate(),
new Email('leszek.prabucki@gmail.com'),
PasswordHash::fromHash($hash)
);
$this->id()->shouldHaveType(UserId::class);
$this->email()->shouldBeLike(new Email('leszek.prabucki@gmail.com'));
$this->passwordHash()->shouldBeLike(PasswordHash::fromHash($hash));
}
}
bin/phpspec run
<?php
declare(strict_types=1);
namespace PHPBenelux\Domain;
class User
{
public function __construct($argument1, $argument2, $argument3)
{
// TODO: write logic here
}
public function id()
{
// TODO: write logic here
}
}
<?php
declare(strict_types=1);
namespace PHPBenelux\Domain;
class User
{
private $id;
private $email;
private $passwordHash;
public function __construct(UserId $id, Email $email, PasswordHash $passwordHash)
{
$this->id = $id;
$this->email = $email;
$this->passwordHash = $passwordHash;
}
public function id(): UserId
{
return $this->id;
}
public function email(): Email
{
return $this->email;
}
public function passwordHash(): PasswordHash
{
return $this->passwordHash;
}
}
Ok, what now?
Register User service
<?php
namespace spec\PHPBenelux\Application\UseCase;
use Exception;
use InvalidArgumentException;
use PHPBenelux\Application\UseCase\RegisterUser;
use PHPBenelux\Application\TransactionManager;
use PHPBenelux\Domain\Users;
use PHPBenelux\Domain\UserFactory;
use PHPBenelux\Domain\User;
use PhpSpec\ObjectBehavior;
class RegisterUserSpec extends ObjectBehavior
{
function let(
TransactionManager $transactionManager,
Users $users,
UserFactory $userFactory
) {
$this->beConstructedWith($transactionManager, $users, $userFactory);
}
function it_creates_and_store_user_in_repository(
TransactionManager $transactionManager,
Users $users,
UserFactory $userFactory,
User $user
) {
$transactionManager->begin()->shouldBeCalled();
$userFactory->create(
'leszek.prabucki@gmail.com',
'password'
)->willReturn($user);
$users->has($user)->willReturn(false);
$users->add($user)->shouldBeCalled();
$transactionManager->commit()->shouldBeCalled();
$this->execute(new RegisterUser\Command(
'leszek.prabucki@gmail.com',
'password'
));
}
}
<?php
declare(strict_types=1);
namespace PHPBenelux\Application\UseCase;
use PHPBenelux\Application\TransactionManager;
use PHPBenelux\Domain\UserFactory;
use PHPBenelux\Domain\Users;
use Throwable;
class RegisterUser
{
private $transactionManager;
private $users;
private $userFactory;
public function __construct(TransactionManager $transactionManager, Users $users, UserFactory $userFactory)
{
$this->transactionManager = $transactionManager;
$this->users = $users;
$this->userFactory = $userFactory;
}
public function execute(RegisterUser\Command $command): void
{
$this->transactionManager->begin();
try {
$user = $this->userFactory->create($command->email(), $command->password());
if ($this->users->has($user)) {
$this->transactionManager->rollback();
return;
}
$this->users->add($user);
$this->transactionManager->commit();
} catch (Throwable $exception) {
$this->transactionManager->rollback();
throw $exception;
}
}
}
Doctrine Adapter
<?php
declare(strict_types=1);
namespace PHPBenelux\Adapter\Doctrine\Domain;
use Doctrine\ORM\EntityManagerInterface;
use PHPBenelux\Domain\Email;
use PHPBenelux\Domain\Exception\UserNotFound;
use PHPBenelux\Domain\Users as UsersInterface;
final class Users implements UsersInterface
{
private $entityManager;
public function __construct(EntityManagerInterface $entityManager)
{
$this->entityManager = $entityManager;
}
public function add(User $user): void
{
$this->entityManager->persist($user);
}
public function has(User $user): bool
{
return (bool) $this
->entityManager
->getRepository(User::class)
->findOneBy(['email.email' => (string) $user->email()])
;
}
public function get(Email $email): User
{
/** @var User $user */
$user = $this->entityManager->getRepository(User::class)->findOneBy(['email.email' => (string) $email]);
if (!$user) {
throw new UserNotFound();
}
return $user;
}
}
<?php
declare(strict_types=1);
namespace PHPBenelux\Adapter\Doctrine\Application;
use Doctrine\ORM\EntityManagerInterface;
use PHPBenelux\Application\TransactionManager as TransactionManagerInterface;
final class TransactionManager implements TransactionManagerInterface
{
private $entityManager;
public function __construct(EntityManagerInterface $entityManager)
{
$this->entityManager = $entityManager;
}
public function begin(): void
{
$this->entityManager->beginTransaction();
}
public function commit(): void
{
$this->entityManager->flush();
$this->entityManager->commit();
}
public function rollback(): void
{
$this->entityManager->rollback();
}
}
Thank you!
Short story about design using tests PHPBenelux uncon
By Leszek Prabucki
Short story about design using tests PHPBenelux uncon
- 1,843