TBolier\RethinkQL\IntegrationTest\Aggregation\GroupTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	testGroupAndMaxByNumberReductionByNumberAndUngroupOrderByNumber()	0	27	1
B	testGroupByNumber()	0	24	1
A	testGroupByDescription()	0	18	1

<?php
declare(strict_types = 1);

namespace TBolier\RethinkQL\IntegrationTest\Aggregation;

use TBolier\RethinkQL\IntegrationTest\Query\AbstractTableTest;
use TBolier\RethinkQL\Response\ResponseInterface;

class GroupTest extends AbstractTableTest
{
    /**
     * @return void
     * @throws \Exception
     */
    public function testGroupByDescription(): void
    {
        $this->insertDocumentWithNumber(5, 10);
        $this->insertDocumentWithNumber(4, 10);
        $this->insertDocumentWithNumber(3, 20);
        $this->insertDocumentWithNumber(2, 20);
        $this->insertDocumentWithNumber(1, 30);

        /** @var ResponseInterface $res */
        $res = $this->r()
            ->table('tabletest')
            ->group('description')
            ->run();

        $this->assertCount(1, $res->getData());

It seems like $res->getData() can also be of type string; however, parameter $haystack of PHPUnit\Framework\Assert::assertCount() does only seem to accept Countable|iterable, maybe add an additional type check?

        $this->assertEquals('A document description.', $res->getData()[0]['group']);
        $this->assertCount(1, $res->getData());
    }

    /**
     * @return void
     * @throws \Exception
     */
    public function testGroupByNumber(): void
    {
        $this->insertDocumentWithNumber(5, 10);
        $this->insertDocumentWithNumber(4, 10);
        $this->insertDocumentWithNumber(3, 20);
        $this->insertDocumentWithNumber(2, 20);
        $this->insertDocumentWithNumber(1, 30);

        /** @var ResponseInterface $res */
        $res = $this->r()
            ->table('tabletest')
            ->group('number')
            ->run();

        $this->assertCount(3, $res->getData());

It seems like $res->getData() can also be of type string; however, parameter $haystack of PHPUnit\Framework\Assert::assertCount() does only seem to accept Countable|iterable, maybe add an additional type check?

        $this->assertEquals(10, $res->getData()[0]['group']);
        $this->assertCount(2, $res->getData()[0]['reduction']);

        $this->assertEquals(20, $res->getData()[1]['group']);
        $this->assertCount(2, $res->getData()[1]['reduction']);

        $this->assertEquals(30, $res->getData()[2]['group']);
        $this->assertCount(1, $res->getData()[2]['reduction']);
    }

    /**
     * @return void
     * @throws \Exception
     */
    public function testGroupAndMaxByNumberReductionByNumberAndUngroupOrderByNumber(): void
    {
        $this->insertDocumentWithNumber('Bob', 10);
        $this->insertDocumentWithNumber('Alice', 20);
        $this->insertDocumentWithNumber('Bob', 30);
        $this->insertDocumentWithNumber('Alice', 40);
        $this->insertDocumentWithNumber('Harry', 50);
        $this->insertDocumentWithNumber('Harry', 45);

        /** @var ResponseInterface $res */
        $res = $this->r()
            ->table('tabletest')
            ->group('id')->max('number')->getField('number')
            ->ungroup()
            ->orderBy($this->r()->desc('number'))
            ->run();

        $this->assertCount(3, $res->getData());

It seems like $res->getData() can also be of type string; however, parameter $haystack of PHPUnit\Framework\Assert::assertCount() does only seem to accept Countable|iterable, maybe add an additional type check?

        $this->assertEquals('Alice', $res->getData()[0]['group']);
        $this->assertEquals(20, $res->getData()[0]['reduction']);

        $this->assertEquals('Bob', $res->getData()[1]['group']);
        $this->assertEquals(10, $res->getData()[1]['reduction']);

        $this->assertEquals('Harry', $res->getData()[2]['group']);
        $this->assertEquals(50, $res->getData()[2]['reduction']);
    }
}