phpunit Первые шаги

От: Из: PHP

megagramm

Здравствуйте, уважаемое сообщество.

Решил что нужно освоить unit tests. Хочу научиться писать тесты и применять их к уже написанному коду.

Начал изучать с видеоуроков, бред бесполезный в основном.
Перешел на текстовое освоение и тут в разных источниках (сайтах_ инструкции немного отличаются. Кажется суть понимать начал. Но ни одна инструкция мной не пройдена без ошибок от и до. Все время что-то стопорится. Проверяю по несколько раз и понимаю что устаревшие где-то.

Прошу помощи. Что я делаю не так, почему выдаются ошибки?

Итак вот одна из инструкций.
https://lexusalex.ru/5-php-phpunit

1. установил composer
2 установил phpunit

Код (Text):
  1. composer require —dev phpunit/phpunit:^9

(У меня версия php 7.3.27, потому ставлю 9 юнит)

Тестирование доступно по пути: ./vendor/bin/phpunit
Хорошо. Дальше в инструкции говорится что мол так неудобно запускать, можно подругому.
Для этого нужно дополнить раздел scripts в composer.json

Код (Text):
  1. {
  2.     «require»: {
  3.         «monolog/monolog»: «2.0.*»,
  4.         «phpmailer/phpmailer»:»^5.2″
  5.     },
  6.     «require-dev»: {
  7.         «phpunit/phpunit»: «^9»
  8.     },
  9.     «scripts»: {
  10.         «test»: «phpunit —colors=always»
  11.     }
  12.  
  13. }

После этого по инструкции нужно запустить команду:

Код (Text):
  1. composer test

Запускаю и получаю:

Вопросы:
1. Что я пропустил или не учел?
2. Может быть вам известна более точная инструкция пошаговой работы с для новичка?

 

ADSoft

похоже что phpunit только в своей папке работает… пропишите путь к ней в системные переменные (Винда) или создайте линк на неё (Линукс)

 

megagramm

Линк на phpunit куда создать?

Я сделал следующее:

Код (Text):
  1. export PATH=$PATH:/home/agrey/composer-test/vendor/bin/

Также не помогло.

 

megagramm

Вроде бы разобрался, В секции scripts нужно было указать так:

«test»: «phpunit —colors=always tests»,
т.е. в конце добавить папку или файлы для теста

после изменений заработало

 

don.bidon

phpunit ещё конфиг из xml-файла подхватывать умеет

Код (Text):
  1. php «vendor/bin/phpunit» -c «build/config/phpunit.xml» —log-junit «build/logs/junit.xml»
Код (Text):
  1. <?xml version=»1.0″ encoding=»UTF-8″?>
  2. <phpunit xmlns:xsi=»http://www.w3.org/2001/XMLSchema-instance» xsi:noNamespaceSchemaLocation=»https://schema.phpunit.de/9.3/phpunit.xsd» bootstrap=»../../vendor/autoload.php» colors=»true» verbose=»true»>
  3.   <coverage>
  4.     <include>
  5.       <directory suffix=».php»>../../src</directory>
  6.     </include>
  7.     <report>
  8.       <clover outputFile=»../output/phpunit.coverage-clover.xml»/>
  9.       <html outputDirectory=»../output/code-coverage»/>
  10.     </report>
  11.   </coverage>
  12.   <php>
  13.     <ini name=»error_reporting» value=»-1″/>
  14.   </php>
  15.   <testsuites>
  16.     <testsuite name=»Tests»>
  17.       <directory>../../tests</directory>
  18.       <exclude>../../tests/config</exclude>
  19.       <exclude>../../tests/src</exclude>
  20.     </testsuite>
  21.   </testsuites>
  22.   <logging/>
  23. </phpunit>
 

megagramm

Спасибо. Видел, но пока еще не изучал эту часть.

Вопрос есть по групповым проверкам. А как делать их?
Вот например. У меня есть валидация пароля. Я хочу проверить пару десятков разных паролей. Неужели на каждый пароль мне свой метод писать нужно test_password2, test_password3 etc????

PHP:
  1. public function test_password()
  2.     {
  3.         $message = «»;
  4.         $this->form[‘password_confirm’] = $this->form[‘password’] = ‘123QWEqwe’;
  5.         $formuser = new Cformuser($this->form);
  6.         $message_arr = $formuser->validate();
  7.      
  8.         if(isset($message_arr[‘password’])){
  9.             $message = $message_arr[‘password’];
  10.         }
  11.         $this->assertEmpty($message_arr, $message);
  12.     }

Мне каежстя это же можно как-то в цикле сделать? Я могу конечно напиисать массив из паролей в метод и просто запускать его в цикле и выводить в ошибку ключи массива неправильные, но мне кажется в phpunit что-то должно быть такое. Задача выгляидт слишком тривиально.

Пока еще не нашел примеров по моему вопросу, буду благодарен за пендель в нужном направлении

 

valentinnew

attributes — data-provider
Но внутри будет тот же массив.

 

artoodetoo

Ну конечно можно. Но это несущественный чисто технический вопрос. Каждый тест должен решать отдельную задачу, доказывая что целевой класс или модуль выполняет некую цель заданным способом. Если для этого надо проверить десять паролей окей, фигачь их в одном тесте. Если каждый пароль проверяет какой-то свой нюанс, можно и на несколько тестов разбить. Но у этих тестов должны быть осмысленные имена чтобы по ним сразу было понятно что пошло не так.
— Добавлено —
То есть testPassword1, testPassword2 это пример неправильного подхода.

 

megagramm

Всем спасибо за предыдущие комментарии. Был перерыв с тестами, сейчас снова вернулся к ним. Все туториалы идут на примерах простых тестов. Но мне вот сходу нужно простетировать модуль «Менеджер задач». Смотрю на констурктор класса и не понимаю с какого бока тут браться за тестирования. Может кто-то скажет логику, что бы вы тестировали у конструктора

PHP:
  1. class bcron
  2. {
  3. …
  4.     public function __construct()
  5.     {
  6.         global $config;
  7.         if (!isset($config[‘log_dir’])) {
  8.             $config[‘log_dir’] = $config[‘install_dir’] . «/logs»;
  9.         }
  10.  
  11.         if (!isset($config[‘log_file’])) {
  12.             $config[‘log_file’] = $config[‘log_dir’] . «/» . $config[‘project_id’] . «.log»;
  13.         }
  14.  
  15.         //Фиксация времени старта
  16.         $this->get_date_array();
  17.  
  18.         //проверка существования таблицы,
  19.         if (!$this->check_exist_table(self::TABLE_CRON_CMD) || !$this->check_exist_table(self::TABLE_CRON)) {
  20.  
  21.             $message = «The tables of this module are missing. Apply the migration mig20230117_000000.inc.php»;
  22.             self::tellMe([«text» => $message, «status» => «error»]);
  23.             exit;
  24.         }
  25.  
  26.         $this->stack_range = 4;
  27.         $this->descriptorspec = [
  28.             0 => [‘pipe’, ‘r’],
  29.             1 => [‘pipe’, ‘w’],
  30.             2 => [‘file’, ‘bcron-log’, «a»]
  31.         ];
  32.         if (is_writable($config[‘log_file’])) {
  33.             $this->descriptorspec[2] = [‘file’, $config[‘log_file’], ‘a’];
  34.         }
  35.     }
  36. …
  37. }

— Добавлено —
Вот мое предположение:

  • Вызывается 1 раз ::get_date_array() и что массив с датой будет не пустой.
  • Никогда не вызывается ::tellMe
  • Вызывается 2 раза ::check_exist_table(). Как понял из туториалов, необходимо сделать дубль метода check_exist_table() чтобы не велось обращение к БД.

Поправьте, если я не прав.

 

valentinnew

Возможно, это оффтопик, но код должен быть написан так, чтоб его можно тестировать, иначе это будет боль и страдание и только видимость тестов, не имеющая практической ценности.
Для того, чтобы научиться писать код так, чтобы он был тестируемым, я рекомендую TDD.
Для того, чтобы не обращаться к БД в тестах, нужны моки, а для этого нужно выносить зависимости.
Ну и psr12.

 

Похожие записи:

  1. Почему установка phpunit через composer выдает ошибку?
  2. Генератор строк документации в типе Google
  3. Шаблон проектирования Factory / Builder в Python
  4. Как составлять регулярные выражения для Яндекс.Метрики
  5. Почему после настройки новой сети соединение с моим сервером minecraft (через nginx) было потеряно, в то время как все другие службы, обслуживаемые nginx, работают нормально?

Добавить комментарий

Ваш адрес email не будет опубликован. Обязательные поля помечены *