OCA\Mail\Db\AliasMapperTest

Complexity

Total Complexity

5

Size/Duplication

Total Lines	55
Duplicated Lines	21.82 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	2
Bugs	1	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setup()	0	5	1
A	testFind()	0	14	1
A	tearDown()	12	12	3

<?php

/**
 * @author Tahaa Karim <tahaalibra@gmail.com>
 *
 * ownCloud - Mail
 *
 * This code is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License, version 3,
 * as published by the Free Software Foundation.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License, version 3,
 * along with this program.  If not, see <http://www.gnu.org/licenses/>
 *
 */

namespace OCA\Mail\Db;

use OC\AppFramework\Db\Db;

class AliasMapperTest extends \PHPUnit_Framework_TestCase {

	/**
	 * @var AliasMapper
	 */
	private $mapper;

	/**
	 * @var \OC\DB\Connection
	 */
	private $db;

	/**
	 * @var Alias
	 */
	private $alias;

	/**
	 * Initialize Mapper
	 */
	public function setup(){
		$db = \OC::$server->getDatabaseConnection();
		$this->db = new Db($db);

It seems like new \OC\AppFramework\Db\Db($db) of type object<OC\AppFramework\Db\Db> is incompatible with the declared type object<OC\DB\Connection> of property $db.

		$this->mapper = new AliasMapper($this->db);
	}

	public function testFind(){

		$this->alias = new Alias();
		$this->alias->setAccountId(1);
		$this->alias->setAlias('alias@marvel.com');
		$this->alias->setName('alias');

		/** @var Alias $b */
		$b = $this->mapper->insert($this->alias);

		$result = $this->mapper->find($b->getId(), 'user12345');

		$this->assertEquals($b, $result);
	}

	protected function tearDown() {

This method seems to be duplicated in your project.

		parent::tearDown();

		$sql = 'DELETE FROM *PREFIX*mail_aliases WHERE `id` = ?';
		$stmt = $this->db->prepare($sql);
		if (!empty($this->alias)) {
			$stmt->execute([$this->alias->getId()]);
		}
		if (!empty($this->address2)) {
			$stmt->execute([$this->alias->getId()]);
		}
	}

}