BitBagCommerce / SyliusMolliePlugin

spec/Action/CaptureActionSpec.php

<?php

/*
 * This file has been created by developers from BitBag.
 * Feel free to contact us once you face any issues or want to start
 * another great project.
 * You can find more information about us on https://bitbag.shop and write us
 * an email on mikolaj.krol@bitbag.pl.
 */

declare(strict_types=1);

namespace spec\BitBag\SyliusMolliePlugin\Action;

use BitBag\SyliusMolliePlugin\Action\CaptureAction;
use BitBag\SyliusMolliePlugin\Client\MollieApiClient;
use Mollie\Api\Endpoints\PaymentEndpoint;
use Payum\Core\Action\ActionInterface;
use Payum\Core\ApiAwareInterface;
use Payum\Core\Bridge\Spl\ArrayObject;
use Payum\Core\GatewayAwareInterface;
use Payum\Core\GatewayInterface;
use Payum\Core\Payum;
use Payum\Core\Reply\HttpRedirect;
use Payum\Core\Request\Capture;
use Payum\Core\Security\GenericTokenFactory;
use Payum\Core\Security\GenericTokenFactoryAwareInterface;
use Payum\Core\Security\TokenInterface;
use PhpSpec\ObjectBehavior;
use Sylius\Component\Core\Model\PaymentInterface;

final class CaptureActionSpec extends ObjectBehavior
{
    function it_is_initializable(): void

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->shouldHaveType(CaptureAction::class);
    }

    function it_implements_action_interface(): void
    {
        $this->shouldHaveType(ActionInterface::class);
    }

    function it_implements_generic_token_factory_aware(): void
    {
        $this->shouldHaveType(GenericTokenFactoryAwareInterface::class);
    }

    function it_implements_api_aware_interface(): void
    {
        $this->shouldHaveType(ApiAwareInterface::class);
    }

    function it_implements_gateway_aware_interface(): void
    {
        $this->shouldHaveType(GatewayAwareInterface::class);
    }

    function it_executes(
        Capture $request,
        ArrayObject $arrayObject,
        PaymentInterface $payment,
        TokenInterface $token,
        TokenInterface $notifyToken,
        Payum $payum,
        GenericTokenFactory $genericTokenFactory,
        GatewayInterface $gateway,
        MollieApiClient $mollieApiClient,
        PaymentEndpoint $paymentEndpoint
    ): void {
        $this->setGateway($gateway);

The method setGateway() does not exist on spec\BitBag\SyliusMolliePlugin\Action\CaptureActionSpec. Since you implemented __call, consider adding a @method annotation.

        $mollieApiClient->isRecurringSubscription()->willReturn(false);
        $this->setApi($mollieApiClient);

The method setApi() does not exist on spec\BitBag\SyliusMolliePlugin\Action\CaptureActionSpec. Since you implemented __call, consider adding a @method annotation.

        $notifyToken->getTargetUrl()->willReturn('url');
        $notifyToken->getHash()->willReturn('test');
        $token->getTargetUrl()->willReturn('url');
        $token->getAfterUrl()->willReturn('url');
        $token->getGatewayName()->willReturn('test');
        $token->getDetails()->willReturn([]);
        $token->getHash()->willReturn('test');
        $genericTokenFactory->createNotifyToken('test', [])->willReturn($notifyToken);

The method willReturn() does not exist on Payum\Core\Security\TokenInterface.

        $genericTokenFactory->createRefundToken('test', [])->willReturn($notifyToken);
        $this->setGenericTokenFactory($genericTokenFactory);

The method setGenericTokenFactory() does not exist on spec\BitBag\SyliusMolliePlugin\Action\CaptureActionSpec. Since you implemented __call, consider adding a @method annotation.

        $payum->getTokenFactory()->willReturn($genericTokenFactory);

The method willReturn() does not exist on Payum\Core\Security\GenericTokenFactoryInterface.

        $arrayObject->toUnsafeArray()->willReturn([]);
        $request->getModel()->willReturn($arrayObject);
        $request->getFirstModel()->willReturn($payment);
        $request->getToken()->willReturn($token);
        $payment = \Mockery::mock('payment');
        $payment->id = 1;

Accessing id on the interface Mockery\MockInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

Accessing id on the interface Mockery\LegacyMockInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        $payment->shouldReceive('getCheckoutUrl')->andReturn('https://thisisnotanemptyurl.com');
        $paymentEndpoint->create([
            'amount' => null,
            'description' => null,
            'redirectUrl' => 'url',
            'webhookUrl' => null,
            'metadata' => null,
        ])->willReturn($payment);

The method willReturn() does not exist on Mollie\Api\Resources\Payment.

        $mollieApiClient->payments = $paymentEndpoint;

        $arrayObject->offsetGet('description')->shouldBeCalled();
        $arrayObject->offsetGet('webhookUrl')->shouldBeCalled();
        $arrayObject->offsetGet('amount')->shouldBeCalled();
        $arrayObject->offsetExists('subscription_mollie_id')->shouldBeCalled();
        $arrayObject->offsetExists('payment_mollie_id')->shouldBeCalled();
        $arrayObject->offsetExists('payment_mollie_id')->shouldBeCalled();
        $arrayObject->offsetGet('metadata')->shouldBeCalled();
        $arrayObject->offsetSet('metadata', ['refund_token' => 'test'])->shouldBeCalled();

Are you sure the usage of $arrayObject->offsetSet('metadata', array('refund_token' => 'test')) targeting Payum\Core\Bridge\Spl\ArrayObject::offsetSet() seems to always return null.

        $arrayObject->offsetSet('payment_mollie_id', 1)->shouldBeCalled();

Are you sure the usage of $arrayObject->offsetSet('payment_mollie_id', 1) targeting Payum\Core\Bridge\Spl\ArrayObject::offsetSet() seems to always return null.

        $arrayObject->offsetSet('webhookUrl', 'url')->shouldBeCalled();

Are you sure the usage of $arrayObject->offsetSet('webhookUrl', 'url') targeting Payum\Core\Bridge\Spl\ArrayObject::offsetSet() seems to always return null.

        $this
            ->shouldThrow(HttpRedirect::class)
            ->during('execute', [$request])
        ;
    }

    function it_supports_only_capture_request_and_array_access(
        Capture $request,
        \ArrayAccess $arrayAccess
    ): void {
        $request->getModel()->willReturn($arrayAccess);

        $this->supports($request)->shouldReturn(true);

The method supports() does not exist on spec\BitBag\SyliusMolliePlugin\Action\CaptureActionSpec. Since you implemented __call, consider adding a @method annotation.

    }
}