Ingenerator\Mailhook\Assert\NegativeAssertionRunner

Complexity

Total Complexity

2

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	do_assert()	0	7	2

<?php
/**
 * Defines Ingenerator\Mailhook\Assert\NegativeAssertionRunner
 *
 * @copyright  2014 inGenerator Ltd
 * @licence    BSD
 */

namespace Ingenerator\Mailhook\Assert;

use Ingenerator\Mailhook\Email;
use Ingenerator\Mailhook\EmailAssertionFailedException;
use Ingenerator\Mailhook\EmailMatcher;

/**
 * Runs a negative assertion - throws if any mails match the criteria
 *
 * @package Ingenerator\Mailhook\Assert
 * @see     spec\Ingenerator\Mailhook\Assert\NegativeAssertionRunnerSpec
 */
class NegativeAssertionRunner extends AssertionRunner {

	/**
	 * @param Email[]        $matched_mails
	 * @param EmailMatcher[] $matchers
	 *
	 * @throws \Ingenerator\Mailhook\EmailAssertionFailedException
	 * @return void
	 */
	protected function do_assert($matched_mails, $matchers)
	{
		if ($matched_mails)

The expression $matched_mails of type Ingenerator\Mailhook\Email[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

		{
			throw new EmailAssertionFailedException('Unexpected matching emails', $matchers);
		}
	}

}