GuillermoandraeTest\Highrise\Repositories\AccountTest

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	testFind()	0	4	1
A	testSearch()	0	4	1
A	setUp()	0	8	1
A	testShow()	0	4	1
A	testFindAll()	0	4	1

<?php

namespace GuillermoandraeTest\Highrise\Repositories;

use Guillermoandrae\Highrise\Repositories\AccountRepository;
use GuillermoandraeTest\Highrise\TestCase;

class AccountTest extends TestCase
{
    /**
     * @var AccountRepository
     */
    private $repository;

The private property $repository is not used, and could be removed.

    public function testShow()
    {
        $account = $this->resource->show();
        $this->assertSame('Your Company', $account->getName());
    }

    public function testFind()
    {
        $this->expectExceptionMessage('The findById method of this repository is not supported');
        $this->resource->find('test');
    }

    public function testFindAll()
    {
        $this->expectExceptionMessage('The findAll method of this repository is not supported');
        $this->resource->findAll();
    }

    public function testSearch()
    {
        $this->expectExceptionMessage('The search method of this repository is not supported');
        $this->resource->search();
    }

    protected function setUp()
    {
        $client = $this->getMockClient(
            200,
            [],
            $this->getMockModelXml('account')
        );
        $this->resource = new AccountRepository($this->getAdapter($client));

The property resource does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }
}