Правильный способ тестирования методов, которые могут возвращать исключения

Question

Правильный способ тестирования методов, которые могут возвращать исключения

0

Я тестирую метод User::validateAdmin() который может выполнять три функции:

Если действительный администратор, верните true
Если истек срок действия admin, верните false
Если не объявление, бросьте NotAnAdminException

Пока у меня это:

$result = $this->User->validateAdmin($validAdmin);
$espected = true;
$this->assertEquals($result, $espected);

$result = $this->User->validateAdmin($disabledAdmin);
$espected = false;
$this->assertEquals($result, $espected);

$this->setExpectedException('NotAnAdminException');
$result = $this->User->validateAdmin($anotherUserRole);
$espected = null;
$this->assertEquals($result, $espected);
$this->setExpectedException(null);

... но это заставляет PHPUnit игнорировать все последующие использования NotAnAdminException, ожидаемые или нет.

Что правильно правильно проверить мои три сценария?

Álvaro González 07 янв. 2015, в 15:13

Источник

0

Похоже, вы поставили все это в одном тесте?
PeeHaa 07 янв. 2015, в 13:16
1

stackoverflow.com/questions/14561908/… ?
ndm 07 янв. 2015, в 13:20
0

Да, поместите материал, который генерирует исключения, в один метод контрольного примера и подтвердите его с помощью аннотаций блока документов.
mark 07 янв. 2015, в 14:25
0

@ndm - Спасибо за ссылку, она ясно объясняет, почему мой подход неверен. Я понимаю, что мне нужно либо попробовать / поймать себя, либо разбить тестовый код на разные методы (не знаю, есть ли предпочтительный способ или это просто вопрос предпочтений).
Álvaro González 07 янв. 2015, в 15:08
0

@mark mark - я думаю, что это будет проверять только первое утверждение в каждом методе тестирования, не так ли? Кроме того, мне нужно, чтобы тесты не выполнялись, если исключение не выдается, когда это необходимо.
Álvaro González 07 янв. 2015, в 15:11
0

@ÁlvaroG.VicarioÁlvaroG.Vicario ваш тест проверяет несколько вещей. Вы сделаете это намного проще для себя, в будущем вам и кому-либо еще придется работать с тестом, чтобы тестировать только одну вещь в тесте.
PeeHaa 07 янв. 2015, в 22:06
0

@PeeHaa - я учел ваш совет, хотя на первый взгляд кажется беспорядочным писать дюжину методов тестирования, просто чтобы протестировать одну и ту же функцию с разными наборами данных (а встроенные тесты ядра CakePHP фактически группируют связанные утверждения). До сих пор, я думаю, я получил ответ на свой текущий вопрос, и я составил (надеюсь, правильный) правильный ответ, чтобы помочь другим.
Álvaro González 08 янв. 2015, в 08:39

Показать ещё 5 комментариев

Теги:

php

cakephp

cakephp-2.0

phpunit

2 ответа

2

Вы должны держать свои тесты короткими и чистыми. Таким образом, вы должны обязательно разделить свой тест на несколько тестов, которые проверяют различные аспекты вашего метода.

Я также рекомендую вам писать тесты в стиле BDD. Поэтому вы должны проверить ожидаемое поведение, например:

public function shouldReturnTrueForAdmin() {
    //given
    $validAdmin = ........;

    //when
    $result = $this->User->validateAdmin($validAdmin);

    //then
    $this->assertTrue($result);
}

Тестирование исключений в этом стиле затруднено в PhpUnit.

Вы можете попробовать CatchException от узо лакомства:

public function shouldFailIfNotAdmin() {
    //given
    $anotherUserRole = ........;

    //when
    CatchException::when($this->User)>validateAdmin($anotherUserRole);

    //then
    CatchException::assertThat()->isInstanceOf("NotAnAdminException");
}

woru 13 янв. 2015, в 07:04

Ещё вопросы

Похоже, вы поставили все это в одном тесте?
Да, поместите материал, который генерирует исключения, в один метод контрольного примера и подтвердите его с помощью аннотаций блока документов.
@ndm - Спасибо за ссылку, она ясно объясняет, почему мой подход неверен. Я понимаю, что мне нужно либо попробовать / поймать себя, либо разбить тестовый код на разные методы (не знаю, есть ли предпочтительный способ или это просто вопрос предпочтений).
@mark mark - я думаю, что это будет проверять только первое утверждение в каждом методе тестирования, не так ли? Кроме того, мне нужно, чтобы тесты не выполнялись, если исключение не выдается, когда это необходимо.
@ÁlvaroG.VicarioÁlvaroG.Vicario ваш тест проверяет несколько вещей. Вы сделаете это намного проще для себя, в будущем вам и кому-либо еще придется работать с тестом, чтобы тестировать только одну вещь в тесте.
@PeeHaa - я учел ваш совет, хотя на первый взгляд кажется беспорядочным писать дюжину методов тестирования, просто чтобы протестировать одну и ту же функцию с разными наборами данных (а встроенные тесты ядра CakePHP фактически группируют связанные утверждения). До сих пор, я думаю, я получил ответ на свой текущий вопрос, и я составил (надеюсь, правильный) правильный ответ, чтобы помочь другим.

Álvaro González · Accepted Answer · 2015-01-08T06-08-00.000Z

Поскольку PHPUnit не продолжает проверку после ожидания исключения, подход, показанный в вопросе, является ошибочным, потому что PHPUnit обертывает тест в регулярном блоке try/catch, поэтому, как только генерируется исключение, остальные утверждения в тесте метод игнорируются. Кроме того, метод \PHPUnit_Framework_TestCase::setExpectedException работает так, как ожидалось: тест проходит тогда и только тогда, когда \PHPUnit_Framework_TestCase::setExpectedException данное исключение.

Самый чистый подход состоял бы в разделении утверждений различными способами:

public function testvalidateAdminValid(){
    $validAdmin = ........;
    $result = $this->User->validateAdmin($validAdmin);
    $espected = true;
    $this->assertEquals($result, $espected);
}

public function testvalidateAdminExpired(){
    $disabledAdmin = ........;
    $result = $this->User->validateAdmin($disabledAdmin);
    $espected = false;
    $this->assertEquals($result, $espected);
}

public function testvalidateAdminNotAnAdmin(){
    $anotherUserRole = ........;
    $this->setExpectedException('NotAnAdminException');
    $this->User->validateAdmin($anotherUserRole);
}

Если вы предпочитаете придерживаться всех связанных тестов в одном и том же методе, вы должны сами эмулировать setExpectedException, например:

try{
    $result = $this->User->validateAdmin($anotherUserRole);
    $this->fail('Failed to throw NotAnAdminException');
}catch(NotAnAdminException $e){
    $this->assertEmpty($result);
}

Или просто используйте стиль аннотации блочного документа @expectedException .