TraceableCommandTestingCapabilities::handleCommandAndAssertTraced() - Code Metrics - hmlb/user-bundle - Measure and Improve Code Quality continuously with Scrutinizer

handleCommandAndAssertTraced() A
last analyzed 2017-03-29 12:56 UTC

↳ Parent: TraceableCommandTestingCapabilities

Complexity

Conditions	2
Paths	2

Size

Total Lines	18
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	18
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	12
nc	2
nop	3

<?php

declare (strict_types = 1);

namespace HMLB\UserBundle\Tests\Command;

use HMLB\DDD\Entity\Identity;
use HMLB\DDD\Persistence\PersistentMessage;
use HMLB\DDDBundle\Repository\PersistentCommandRepository;
use HMLB\UserBundle\Message\Trace\Context;
use HMLB\UserBundle\Message\Trace\Trace;
use HMLB\UserBundle\Message\TraceableCommand;
use SimpleBus\Message\Bus\MessageBus;

/**
 * Trait TraceableCommandTestingCapabilities.
 *
 * @author Hugues Maignol <[email protected]>
 */
trait TraceableCommandTestingCapabilities
{
    /**
     * @param MessageBus                  $commandBus
     * @param PersistentMessage           $command
     * @param PersistentCommandRepository $commandRepository
     */
    protected function handleCommandAndAssertTraced(
        MessageBus $commandBus,
        PersistentMessage $command,
        PersistentCommandRepository $commandRepository
    ) {
        $commandBus->handle($command);

        $this->assertInstanceOf(Identity::class, $command->getId());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        $foundCommand = $commandRepository->get($command->getId());
        $this->assertSame($command, $foundCommand);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        if (!$command instanceof TraceableCommand) {
            $this->fail(sprintf('The message %s is not Traceable', get_class($command)));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }
        $this->assertInstanceOf(Trace::class, $trace = $command->getTrace());
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $this->assertInstanceOf(Context::class, $trace->getContext());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }
}


1			<?php
2
3			declare (strict_types = 1);
4
5			namespace HMLB\UserBundle\Tests\Command;
6
7			use HMLB\DDD\Entity\Identity;
8			use HMLB\DDD\Persistence\PersistentMessage;
9			use HMLB\DDDBundle\Repository\PersistentCommandRepository;
10			use HMLB\UserBundle\Message\Trace\Context;
11			use HMLB\UserBundle\Message\Trace\Trace;
12			use HMLB\UserBundle\Message\TraceableCommand;
13			use SimpleBus\Message\Bus\MessageBus;
14
15			/**
16			* Trait TraceableCommandTestingCapabilities.
17			*
18			* @author Hugues Maignol <[email protected]>
19			*/
20			trait TraceableCommandTestingCapabilities
21			{
22			/**
23			* @param MessageBus $commandBus
24			* @param PersistentMessage $command
25			* @param PersistentCommandRepository $commandRepository
26			*/
27			protected function handleCommandAndAssertTraced(
28			MessageBus $commandBus,
29			PersistentMessage $command,
30			PersistentCommandRepository $commandRepository
31			) {
32			$commandBus->handle($command);
33
34			$this->assertInstanceOf(Identity::class, $command->getId());
			0 ignored issues – show Bug introduced 2016-02-13 15:32 UTC by Report Bug Copy Issue Report It seems like `assertInstanceOf()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
35
36			$foundCommand = $commandRepository->get($command->getId());
37			$this->assertSame($command, $foundCommand);
			0 ignored issues – show Bug introduced 2016-02-13 15:32 UTC by Report Bug Copy Issue Report It seems like `assertSame()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
38
39			if (!$command instanceof TraceableCommand) {
40			$this->fail(sprintf('The message %s is not Traceable', get_class($command)));
			0 ignored issues – show Bug introduced 2016-02-13 15:32 UTC by Report Bug Copy Issue Report It seems like `fail()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
41			}
42			$this->assertInstanceOf(Trace::class, $trace = $command->getTrace());
			0 ignored issues – show Bug introduced 2016-02-14 13:51 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `HMLB\DDD\Persistence\PersistentMessage` as the method `getTrace()` does only exist in the following implementations of said interface: `HMLB\UserBundle\Command\ChangeEmail`, `HMLB\UserBundle\Command\ChangePassword`, `HMLB\UserBundle\Command\ConfirmEmail`, `HMLB\UserBundle\Command\RegisterUser`, `HMLB\UserBundle\Command\RequestEmailValidation`, `HMLB\UserBundle\Command\RequestPasswordReset`, `HMLB\UserBundle\Command\ResetPassword`, `HMLB\UserBundle\Command\UserCommand`, `HMLB\UserBundle\Event\EmailChanged`, `HMLB\UserBundle\Event\EmailConfirmed`, `HMLB\UserBundle\Event\EmailValidationRequested`, `HMLB\UserBundle\Event\PasswordChanged`, `HMLB\UserBundle\Event\PasswordReset`, `HMLB\UserBundle\Event\PasswordResetRequested`, `HMLB\UserBundle\Event\UserDisabled`, `HMLB\UserBundle\Event\UserEnabled`, `HMLB\UserBundle\Event\UserEvent`, `HMLB\UserBundle\Event\UserRegistered`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history... Bug introduced 2016-02-13 15:32 UTC by Report Bug Copy Issue Report It seems like `assertInstanceOf()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
43			$this->assertInstanceOf(Context::class, $trace->getContext());
			0 ignored issues – show Bug introduced 2016-02-13 15:32 UTC by Report Bug Copy Issue Report It seems like `assertInstanceOf()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
44			}
45			}
46

hmlb / user-bundle

handleCommandAndAssertTraced() A last analyzed 2017-03-29 12:56 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like

handleCommandAndAssertTraced() A
last analyzed 2017-03-29 12:56 UTC