BCRM\WebBundle\Tests\Functional\EventControllerTest

Complexity

Total Complexity

16

Size/Duplication

Total Lines	382
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	16

Importance

Changes	1
Bugs	0	Features	0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	createTicketsCommand()	0	4	1
A	eventUnregistration()	0	14	1
B	confirmEventUnregistration()	0	24	1
A	confirmUnregistrationCommand()	0	4	1
A	processUnregistrationsCommand()	0	4	1
A	setUpBeforeClass()	0	4	1
B	createTickets()	0	38	1
A	payRegistration()	0	51	1
A	sendTicketsCommand()	0	4	1
A	sendPayTicketCommand()	0	4	1
A	removeTickets()	0	22	1
A	thereShouldBeNoMoreTicketsThanCapacity()	0	69	2
A	cancelTicketLink()	0	22	1
B	eventRegistration()	0	31	1
A	tagsShouldAllowUmlauts()	0	21	1

<?php

/**
 * @author    Markus Tacker <m@coderbyheart.com>
 * @copyright 2013-2016 Verein zur Förderung der Netzkultur im Rhein-Main-Gebiet e.V. | http://netzkultur-rheinmain.de/
 */

namespace BCRM\WebBundle\Tests\Functional;

use BCRM\BackendBundle\Command\SendPayRegistrationMailCommand;
use BCRM\BackendBundle\Command\SendTicketsMailCommand;
use BCRM\BackendBundle\Entity\Event\Ticket;
use BCRM\BackendBundle\Entity\Event\Registration;
use BCRM\BackendBundle\Entity\Event\Unregistration;
use BCRM\BackendBundle\Command\SendConfirmUnregistrationMailCommand;
use BCRM\BackendBundle\Command\CreateTicketsCommand;
use BCRM\BackendBundle\Command\ProcessUnregistrationsCommand;
use BCRM\BackendBundle\Entity\Payment;
use BCRM\BackendBundle\Service\Payment\PayRegistrationCommand;
use Symfony\Component\DependencyInjection\ContainerInterface;

class EventControllerTest extends Base
{
    /**
     * The setUpBeforeClass() and tearDownAfterClass() template methods are called before the first test of the test
     * case class is run and after the last test of the test case class is run, respectively.
     */
    public static function setUpBeforeClass()
    {
        static::resetDatabase();
    }

    /**
     * @test
     * @group functional
     */
    public function eventRegistration()
    {
        $email   = 'name@domain.com';
        $client  = static::createClient();
        $crawler = $client->request('GET', '/anmeldung');

        $form                                    = $crawler->selectButton('event_register[save]')->form();
        $form['event_register[name]']            = 'John Doe';
        $form['event_register[email]']           = $email;
        $form['event_register[days]']            = 3;
        $form['event_register[donationEur]']     = '12,34';
        $form['event_register[payment]']         = 'paypal';
        $form['event_register[food]']            = 'default';
        $form['event_register[tags]']            = '#foo #bar #bcrm13';
        $form['event_register[twitter]']         = '@somebody';
        $form['event_register[participantList]'] = '1';
        $client->submit($form);

$form is of type array<string,string|integer,{"event_register[participantList]":"string"}>, but the function expects a object<Symfony\Component\DomCrawler\Form>.

        $response = $client->getResponse();
        $this->assertEquals(302, $response->getStatusCode());
        $this->assertTrue($response->isRedirect('/anmeldung/check'), sprintf('Unexpected redirect to %s', $response->headers->get('Location')));

        $crawler                                   = $client->followRedirect();
        $form                                      = $crawler->selectButton('event_register_review[save]')->form();
        $form['event_register_review[norefund]']   = '1';
        $form['event_register_review[autocancel]'] = '1';
        $client->submit($form);

$form is of type array<string,string,{"event_register_review[autocancel]":"string"}>, but the function expects a object<Symfony\Component\DomCrawler\Form>.

        $response = $client->getResponse();
        $this->assertEquals(302, $response->getStatusCode());
        $this->assertTrue($response->isRedirect('/anmeldung/ok'), sprintf('Unexpected redirect to %s', $response->headers->get('Location')));
        return $email;
    }

    /**
     * @test
     * @group   functional
     * @depends eventRegistration
     */
    public function createTickets($email)
    {
        $client    = static::createClient();
        $container = $client->getContainer();

        $this->createTicketsCommand($container);

It seems like $container defined by $client->getContainer() on line 77 can be null; however, BCRM\WebBundle\Tests\Functional\EventControllerTest::createTicketsCommand() does not accept null, maybe add an additional type check?

        $this->sendTicketsCommand($container); // Tickets should not be sent, because not yet paid for

It seems like $container defined by $client->getContainer() on line 77 can be null; however, BCRM\WebBundle\Tests\Functional\EventControllerTest::sendTicketsCommand() does not accept null, maybe add an additional type check?

        $this->sendPayTicketCommand($container); // but a payment notification should be sent

It seems like $container defined by $client->getContainer() on line 77 can be null; however, BCRM\WebBundle\Tests\Functional\EventControllerTest::sendPayTicketCommand() does not accept null, maybe add an additional type check?

        /* @var $em \Doctrine\Common\Persistence\ObjectManager */
        $em = $container
            ->get('doctrine')
            ->getManager();

        /* @var $tickets Ticket[] */
        $tickets = $em->getRepository('BCRMBackendBundle:Event\Ticket')->findAll();
        $this->assertEquals(2, count($tickets));
        $this->assertEquals($email, $tickets[0]->getEmail());
        $this->assertEquals($email, $tickets[1]->getEmail());
        $days = array(
            $tickets[0]->getDay(),
            $tickets[1]->getDay()
        );
        $this->assertTrue(in_array(Ticket::DAY_SATURDAY, $days));
        $this->assertTrue(in_array(Ticket::DAY_SUNDAY, $days));
        $this->assertEquals(1, $tickets[0]->getEvent()->getId());
        $this->assertEquals(1, $tickets[1]->getEvent()->getId());
        $this->assertNull($tickets[0]->getPayment());
        $this->assertNull($tickets[1]->getPayment());
        $this->assertFalse($tickets[0]->isNotified());
        $this->assertFalse($tickets[1]->isNotified());

        /** @var Registration[] $registration */
        $registration = $em->getRepository('BCRMBackendBundle:Event\Registration')->findOneByEmail($email);

The method findOneByEmail() does not exist on Doctrine\Common\Persistence\ObjectRepository. Did you maybe mean findOneBy()?

        $this->assertTrue($registration->isPaymentNotified());

The method isPaymentNotified cannot be called on $registration (of type array<integer,object<BCRM\BackendBundle\Entity\Event\Registration>>).

        return $email;
    }

    /**
     * @test
     * @group   functional
     * @depends createTickets
     */
    public function payRegistration($email)
    {
        $client    = static::createClient();
        $container = $client->getContainer();
        /* @var $em \Doctrine\Common\Persistence\ObjectManager */
        $em = $container
            ->get('doctrine')
            ->getManager();
        /** @var Registration $registration */
        $registration = $em->getRepository('BCRMBackendBundle:Event\Registration')->findOneByEmail($email);

The method findOneByEmail() does not exist on Doctrine\Common\Persistence\ObjectRepository. Did you maybe mean findOneBy()?

        $this->assertEquals('paypal', $registration->getPaymentMethod());
        $this->assertNull($registration->getPayment());

        $client = static::createClient();
        $client->request('GET', '/anmeldung/' . $registration->getUuid() . '/payment');
        $response = $client->getResponse();
        $this->assertEquals(200, $response->getStatusCode());
        preg_match('/data-number="([^"]+)"/', $response->getContent(), $registrationUuidMatch);
        preg_match('/data-amount="([^"]+)"/', $response->getContent(), $registrationAmountMatch);
        $this->assertEquals($registration->getUuid(), $registrationUuidMatch[1]);
        $this->assertEquals(sprintf('%.2f', ceil((2000 + 1234) * 1.019 + 35) / 100), $registrationAmountMatch[1]);

        // Add payment
        $payment = new Payment();
        $payment->setMethod('cash');
        $payment->setTransactionId($email);
        $em->persist($payment);
        $em->flush();

        $command               = new PayRegistrationCommand();
        $command->registration = $registration;
        $command->payment      = $payment;
        /** @var \LiteCQRS\Bus\CommandBus $commandBus */
        $commandBus = $container->get('command_bus');
        $commandBus->handle($command);
        $registration = $em->getRepository('BCRMBackendBundle:Event\Registration')->findOneByEmail($email);

The method findOneByEmail() does not exist on Doctrine\Common\Persistence\ObjectRepository. Did you maybe mean findOneBy()?

        $this->assertEquals($payment, $registration->getPayment());

        // Send tickets
        $this->sendTicketsCommand($container);

It seems like $container defined by $client->getContainer() on line 121 can be null; however, BCRM\WebBundle\Tests\Functional\EventControllerTest::sendTicketsCommand() does not accept null, maybe add an additional type check?

        /* @var $tickets Ticket[] */
        $tickets = $em->getRepository('BCRMBackendBundle:Event\Ticket')->findAll();
        $this->assertEquals(2, count($tickets));
        $this->assertEquals($payment, $tickets[0]->getPayment());
        $this->assertEquals($payment, $tickets[1]->getPayment());
        $this->assertTrue($tickets[0]->isNotified());
        $this->assertTrue($tickets[1]->isNotified());

        return $email;
    }

    protected function createTicketsCommand(ContainerInterface $container)
    {
        $this->runCommand($container, new CreateTicketsCommand(), 'bcrm:tickets:create');
    }

    protected function sendTicketsCommand(ContainerInterface $container)
    {
        $this->runCommand($container, new SendTicketsMailCommand(), 'bcrm:tickets:send');
    }

    protected function sendPayTicketCommand(ContainerInterface $container)
    {
        $this->runCommand($container, new SendPayRegistrationMailCommand(), 'bcrm:registration:pay');
    }

    /**
     * @test
     * @group   functional
     * @depends payRegistration
     */
    public function eventUnregistration($email)
    {
        $client  = static::createClient();
        $crawler = $client->request('GET', '/stornierung');

        $form                            = $crawler->selectButton('event_unregister[save]')->form();
        $form['event_unregister[email]'] = $email;
        $form['event_unregister[days]']  = 3;
        $client->submit($form);

$form is of type array<string,?,{"event_unregister[days]":"integer"}>, but the function expects a object<Symfony\Component\DomCrawler\Form>.

        $response = $client->getResponse();
        $this->assertEquals(302, $response->getStatusCode());
        $this->assertTrue($response->isRedirect('/stornierung/ok'), sprintf('Unexpected redirect to %s', $response->headers->get('Location')));
        return $email;
    }

    /**
     * @test
     * @group   functional
     * @depends eventUnregistration
     */
    public function confirmEventUnregistration($email)
    {
        $client    = static::createClient();
        $container = $client->getContainer();

        $this->confirmUnregistrationCommand($container);

It seems like $container defined by $client->getContainer() on line 213 can be null; however, BCRM\WebBundle\Tests\Functional\EventControllerTest::confirmUnregistrationCommand() does not accept null, maybe add an additional type check?

        /* @var $em \Doctrine\Common\Persistence\ObjectManager */
        $em = $container
            ->get('doctrine')
            ->getManager();

        /* @var $registration Registration */
        $registration = $em->getRepository('BCRMBackendBundle:Event\Unregistration')->findOneBy(array(
            'email' => $email
        ));

        // Confirm
        $client->request('GET', sprintf('/stornierung/bestaetigen/%d/%s', $registration->getId(), $registration->getConfirmationKey()));

The method getConfirmationKey() does not seem to exist on object<BCRM\BackendBundle\Entity\Event\Registration>.

        $response = $client->getResponse();
        $this->assertEquals(302, $response->getStatusCode());
        $this->assertTrue($response->isRedirect('/stornierung/aktiviert'), sprintf('Unexpected redirect to %s', $response->headers->get('Location')));
        return $email;
    }

    protected function confirmUnregistrationCommand(ContainerInterface $container)
    {
        $this->runCommand($container, new SendConfirmUnregistrationMailCommand(), 'bcrm:unregistration:confirm');
    }

    /**
     * @test
     * @group   functional
     * @depends confirmEventUnregistration
     */
    public function removeTickets($email)
    {
        $client    = static::createClient();
        $container = $client->getContainer();

        $this->processUnregistrationsCommand($container);

It seems like $container defined by $client->getContainer() on line 248 can be null; however, BCRM\WebBundle\Tests\Functional\EventControllerTest::processUnregistrationsCommand() does not accept null, maybe add an additional type check?

        /* @var $em \Doctrine\Common\Persistence\ObjectManager */
        $em = $container
            ->get('doctrine')
            ->getManager();

        /* @var $registration Registration */
        $registration = $em->getRepository('BCRMBackendBundle:Event\Registration')->findOneById(2);

The method findOneById() does not exist on Doctrine\Common\Persistence\ObjectRepository. Did you maybe mean findOneBy()?

        $this->assertEquals(0, $registration->getSaturday());
        $this->assertEquals(0, $registration->getSunday());

        /* @var $tickets Ticket[] */
        $tickets = $em->getRepository('BCRMBackendBundle:Event\Ticket')->findAll();
        $this->assertEquals(0, count($tickets));
        return $email;
    }

    protected function processUnregistrationsCommand(ContainerInterface $container)
    {
        $this->runCommand($container, new ProcessUnregistrationsCommand(), 'bcrm:tickets:process-unregistrations');
    }

    /**
     * @test
     * @group   functional
     * @depends removeTickets
     */
    public function thereShouldBeNoMoreTicketsThanCapacity()
    {
        $client    = static::createClient();
        $container = $client->getContainer();
        /* @var $em \Doctrine\Common\Persistence\ObjectManager */
        $em = $container
            ->get('doctrine')
            ->getManager();

        $this->assertEquals(0, count($em->getRepository('BCRMBackendBundle:Event\Ticket')->findAll()));

        $event = $em->getRepository('BCRMBackendBundle:Event\Event')->findAll()[0];

        // Create registrations
        for ($i = 1; $i <= 5; $i++) {
            $email        = 'john.doe.198' . $i . '@domain.com';
            $registration = new Registration();
            $registration->setName($i);
            $registration->setEvent($event);
            $registration->setEmail($email);
            $registration->setSaturday(true);
            $registration->setUuid($email);
            $payment = new Payment();
            $payment->setMethod('cash');
            $payment->setTransactionId($email);
            $em->persist($payment);
            $registration->setPayment($payment);
            $em->persist($registration);
        }
        $em->flush();

        $this->createTicketsCommand($container);

It seems like $container defined by $client->getContainer() on line 281 can be null; however, BCRM\WebBundle\Tests\Functional\EventControllerTest::createTicketsCommand() does not accept null, maybe add an additional type check?

        $tickets = $em->getRepository('BCRMBackendBundle:Event\Ticket')->findBy(array(
            'event' => $event,
            'day'   => Ticket::DAY_SATURDAY,
        ));
        $this->assertEquals(3, count($tickets));
        $this->assertInArray('john.doe.1981@domain.com', array_map(function (Ticket $t) {
            return $t->getEmail();
        }, $tickets));

        // Unregister Tickets
        $unregistration = new Unregistration();
        $unregistration->setEvent($event);
        $unregistration->setEmail('john.doe.1981@domain.com');
        $unregistration->setConfirmed(true);
        $unregistration->setSaturday(true);
        $em->persist($unregistration);
        $em->flush();

        $this->processUnregistrationsCommand($container);

It seems like $container defined by $client->getContainer() on line 281 can be null; however, BCRM\WebBundle\Tests\Functional\EventControllerTest::processUnregistrationsCommand() does not accept null, maybe add an additional type check?

        $this->assertEquals(2, count($em->getRepository('BCRMBackendBundle:Event\Ticket')->findBy(array(
            'event' => $event,
            'day'   => Ticket::DAY_SATURDAY,
        ))));

        $this->createTicketsCommand($container);

It seems like $container defined by $client->getContainer() on line 281 can be null; however, BCRM\WebBundle\Tests\Functional\EventControllerTest::createTicketsCommand() does not accept null, maybe add an additional type check?

        $tickets = $em->getRepository('BCRMBackendBundle:Event\Ticket')->findBy(array(
            'event' => $event,
            'day'   => Ticket::DAY_SATURDAY,
        ));
        $this->assertEquals(3, count($tickets));
        $this->assertNotInArray('john.doe.1981@domain.com', array_map(function (Ticket $t) {
            return $t->getEmail();
        }, $tickets));
    }

    /**
     * @test
     * @group   functional
     * @depends thereShouldBeNoMoreTicketsThanCapacity
     */
    public function cancelTicketLink()
    {
        $client    = static::createClient();
        $container = $client->getContainer();
        /* @var $em \Doctrine\Common\Persistence\ObjectManager */
        $em     = $container
            ->get('doctrine')
            ->getManager();
        $ticket = $em->getRepository('BCRMBackendBundle:Event\Ticket')->findOneBy(array('day' => 1));

        // Confirm
        $crawler = $client->request('GET', sprintf('/stornierung/%d/%s', $ticket->getId(), $ticket->getCode()));
        $form    = $crawler->selectButton('cancel_confirm')->form();
        $client->submit($form);
        $response = $client->getResponse();
        $this->assertEquals(302, $response->getStatusCode());
        $this->assertTrue($response->isRedirect('/stornierung/aktiviert'), sprintf('Unexpected redirect to %s', $response->headers->get('Location')));

        $this->processUnregistrationsCommand($container);

It seems like $container defined by $client->getContainer() on line 356 can be null; however, BCRM\WebBundle\Tests\Functional\EventControllerTest::processUnregistrationsCommand() does not accept null, maybe add an additional type check?

        $this->assertEquals(2, count($em->getRepository('BCRMBackendBundle:Event\Ticket')->findAll()));
    }

    /**
     * Test for https://github.com/BCRM/www/issues/1
     *
     * @test
     * @group functional
     */
    public function tagsShouldAllowUmlauts()
    {
        $email   = 'name@domain.com';
        $client  = static::createClient();
        $crawler = $client->request('GET', '/anmeldung');

        $form                                    = $crawler->selectButton('event_register[save]')->form();
        $form['event_register[name]']            = 'John Doe';
        $form['event_register[email]']           = $email;
        $form['event_register[days]']            = 3;
        $form['event_register[payment]']         = 'paypal';
        $form['event_register[food]']            = 'default';
        $form['event_register[tags]']            = '#zauberwürfel #bar #bcrm13';
        $form['event_register[twitter]']         = '@somebody';
        $form['event_register[participantList]'] = '1';
        $client->submit($form);

$form is of type array<string,string|integer,{"event_register[participantList]":"string"}>, but the function expects a object<Symfony\Component\DomCrawler\Form>.

        $response = $client->getResponse();
        $this->assertEquals(302, $response->getStatusCode());
        $this->assertTrue($response->isRedirect('/anmeldung/check'), sprintf('Unexpected redirect to %s', $response->headers->get('Location')));
        return $email;
    }
}