Как протестировать код который работает с shell командами?

Question

[Askar Fuzaylov]

Как протестировать классы и методы которые используют такой метод?

protected function exec()
    {
        $descriptorSpec = array(
            0 => array('pipe', 'r'),
            1 => array('pipe', 'w'),
            2 => array('pipe', 'a')
        );

        Response::addRecord('Command: ' . $this->getCommand());
        $process = proc_open($this->getCommand(), $descriptorSpec, $pipes, $this->directory);
        if (is_resource($process)) {
            Response::addRecord('Response: ' . stream_get_contents($pipes[1]));
            fclose($pipes[1]);

            return proc_close($process);
        }

        return false;
    }

Answer 1

[D']

Я бы сделал обертку для работы с proc_open\proc_close\etc...
Тогда можно просто замокать эту обертку, и протестить только логику.

Comments

[Askar Fuzaylov]

Вся проблема в том что я не смогу протестировать правильная команда передана или нет. Потому что буду получать всегда один результат

[D']

Askar Fuzaylov: Т.е? Делаем класс FileSystem, который будет вызывать нативные команды. В функции exec нет смысла тестировать вызов нативных команд, так как это не его забота.
В exec нужно протестировать только то, что он вызывает нужные методы из FileSystem, в правильной последовательности, и с правильными данными.

А сам класс FileSystem нужно тестировать отдельно, со всеми необходимыми условиями.

[Askar Fuzaylov]

D' Normalization: Можете показать набросок такого теста и обертки? Я не совсем понял как вы предлагаете решить задачу

Answer 2

[Дмитрий]

Ну вот Вы сами и сформулировали ответ на свой вопрос. Попробую пояснить. Модульный тест, хороший тест, обладает качеством: его просто написать! Даже вопроса не возникнет, о том, а как написать-то? Если же написать сложно, то Вы, возможно, пытаетесь написать тест на код в котором выполняется более чем одна задача!

Сейчас вижу, что у Вас в методе выполнется минимум два не связанных между собою действия:
1. Создание дескриптора
2. Выполнение какого-то действия с применением этого дескриптора

Их надо тестировать по отдельности! Для действия №1 Вы можете написать тесты:
1. Позитивный: это когда 0, 1, 2
2. Негативный: когда меньше нул или больше 2

Для действия №2: Вы тоже можете написать минимум два теста с правильным и не правильным дескриптором

Comments

[Askar Fuzaylov]

Я согласен с вами попробую разделить ответственности. А как мне протестировать команды для linux если использую винду? К примеру для №2 при создании дескрипторов

[Дмитрий]

Askar Fuzaylov: Любое действие можно разбить на более мелкие действия. Но в Вашем случае Вы делаете бред. Поясню : Вы Вызываете команды ОС и функции из стандартной библиотеки PHP . Вопросы: 1) Вы действительно так сильно не доверяете командам разработчиков разрабатывающих MS Windows или Linux? 2) Вы действительно не доверяете команде разрабатывающих PHP ?

Я к тому, что декомпозируя свою задачу на более мелкие Вы рано или поздно упираетесь в атомарные и элементарные действия, которые выполняются стандартной библиотекой или другой библиотекой, которая уже протестирована!

Тестировать надо ЗНАчимые действия. К примеру от чего может сломаться почти вся система!

[Askar Fuzaylov]

Дмитрий: Я вроде понял вашу мысль. Т.е. тесты должны писать так что бы не были зависимы от окружения и тестами должны быть покрыты только важные участки содержащую пользовательскую бизнес логику.

[Дмитрий]

Askar Fuzaylov: Почти, но не совсем! ;) Однако с подобными знаниями Вы уже сможете писать более менее приемлимое тестовое покрытие. Если интересно, то можем пообщаться в скайпе, беру не дорого ;)