Ingenerator\Mailhook\Assert\PositiveAssertionRunner

Complexity

Total Complexity

2

Size/Duplication

Total Lines	17
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\PositiveAssertionRunner
 *
 * @copyright  2014 inGenerator Ltd
 * @licence    BSD
 */

namespace Ingenerator\Mailhook\Assert;

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

/**
 * Runs a positive assertion - one that throws if no emails match the provided criteria
 *
 * @package Ingenerator\Mailhook\Assert
 * @see     spec\Ingenerator\Mailhook\Assert\PositiveAssertionRunnerSpec
 */
class PositiveAssertionRunner extends AssertionRunner {

	/**
	 * @param Email[]        $matched_mails
	 * @param EmailMatcher[] $matchers
	 *
	 * @return void
	 * @throws \Ingenerator\Mailhook\EmailAssertionFailedException
	 */
	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('No emails matched criteria', $matchers);
		}
	}
}