Как написать юнит тест для ввода с клавиатуры?

Question

[PJeys]

using System;
using System.Threading;
using System.Collections.Generic;

public class EventLoop
{
    private ManualResetEventSlim _messageEvent = new ManualResetEventSlim(false);
    private Queue<Int32> _messageQueue = new Queue<Int32>();
    public int sum = 0;
    public void Threadmain()
    {
        var thrd = new Thread(PumpMessage);
        thrd.Start();
        WriteLine("Enter first number to summarize, enter q or Q to stop");
        while (true)
        {
            var message = WaitMessage();
            sum += message;
            WriteLine("Current sum:" + Convert.ToString(sum));
            if (Convert.ToString(message) == "q" || Convert.ToString(message) == "Q")
            {
                WriteLine("The final sum is" + Convert.ToString(sum));
                break;
            }

        }

    }
    private int WaitMessage()
    {
        _messageEvent.Wait();
        var message = _messageQueue.Dequeue();
        _messageEvent.Reset();
        return Convert.ToInt32(message);
    }
    private void PumpMessage()
    {
        while (true)
        {
            var num = ReadLine();
            if (num == "q" || num == "Q")
            {
                WriteLine("The final sum is" + Convert.ToString(sum));
                break;
            }
            _messageQueue.Enqueue(Convert.ToInt32(num));
            _messageEvent.Set();
        }
    }
    public virtual string ReadLine()
    {
        return Console.ReadLine();
    }
    public virtual void WriteLine(string strline)
    {
        Console.WriteLine(strline);
    }
}
public class Program
{
    static void Main(string[] args)
    {
        EventLoop obj = new EventLoop();
        obj.Threadmain();
    }
}

Имеется такая программа, к которой нужно оформить юнит тесты. Загвоздка в том, что данные для программы пользователь вводит с клавиатуры, как можно имитировать ввод для успешного тестирования?

Answer 1

[Алексей Березников]

Если вы действительно хотите тестировать ввод с клавиатуры, это значит что вам нужно замокать статический метод.
У Fody есть add-in Fody.Ionad, который позволяет делать как раз именно это.
Но, конечно, будет правильнее воспользоваться советом предыдущих ответчиков, и переделать код так, чтобы он был более тестируем.

Answer 2

[cicatrix]

Пример, когда платят за строку кода :)
Код абсолютно НЕ тестопригодный, вы постарались.
На ввод с клавиатуры юнит-тест не нужен. Нужен юнит-тест на метод, который производит с этим вводом какие-либо действия. Основная логика у вас происходит именно в Threadmain() - методе, который ничего не принимает и ничего не возвращает, кроме возможных исключений.

1. Зачем ввод с клавиатуры в отдельном потоке, да ещё и с очередью, если ReadLine() блокирующий метод.
2. Convert.ToInt32() выдаст исключение, если аргументом будет строка, не содержащая цифры.
3. num == "q" || num == "Q" - есть методы .ToLower(), ToUpper(). Название переменной num абсолютно нелогично, если оно не числовое.
4. Ещё тестом надо покрыть PumpMessage(), чтобы проверить, выходит ли программа из цикла

UPD: Вот рефактор, который делает более-менее то же самое, и его можно тестировать:

using System;
using System.Linq;
using System.Collections.Generic;

class Program {

  static List<int> _numbers = new List<int>();  
  static void Main() {
    while(UserInput(Console.ReadLine()));
    Console.WriteLine($"Sum is: {CalcSum(_numbers)}");
  }
  
  static bool UserInput(string input)
  {
      if(string.IsNullOrEmpty(input)) return false;
      if(!int.TryParse(input, out int number)) return false;
      _numbers.Add(number);
      return true;
  }
  
  static int CalcSum(IEnumerable<int> numbers)
  {
      return numbers.Sum();
  }
}

Тесты для UserInput
UserInput получает на вход null и должен вернуть false
UserInput получает на вход "" и должен вернуть false
UserInput получает на вход что угодно, кроме числа и должен вернуть false
UserInput на вход число и возвращает true

Тесты для CalcSum
Получает на вход пустой список и возвращает 0
Получает на вход массив [1, 2] возвращает 3
(ну для пущей надёжности можно ещё добавить проверку входящего аргумента на null) и проверить, выбрасывается ли исключение

Comments

[PJeys]

Добрый вечер! Спасибо за комментарий, но в моём задании суть состояла в том, чтобы реализовать event loop любым способом. Проблемы возникли на этапе написания тестов.

[cicatrix]

PJeys, Ну, я догадывался, что это было нечто подобное, однако, тут ещё можно поспорить над определением EventLoop, в какой-то степени, мой пример, это тоже он :)
Но суть не в этом. Есть такая парадигма - Test Driven Development. Огромное количество книг и материалов написано, можете погуглить. Однако, она накладывает определённые ограничения и требования. Самое главное требование - сделать ваш код тестируемым. Юнит-тест - это проверка некоторого условия, AAA - Arrange, Act, Assert. Подготовка, действие, проверка, если хотите.
Иными словами, вы должны написать программу, которая проверит, как работает ваш код. При этом для нужд тестирования вы не должны никаким образом ничего добавлять или иным образом модифицировать код бизнес-логики. Все операции по тестированию должны проводиться извне, как с "чёрным ящиком". Человек, который пишет юнит-тест вообще не должен знать, что там внутри у вашего метода понаписано.
Соответственно, ваш код должен быть представлен в виде методов, которые что-то берут на вход и что-то выдают в качестве результата (либо бросают исключение - на это тоже можно тестировать).
Ваш код инкапсулировал программную логику так, что протестировать что-либо извне без модификации методов изнутри не представляется возможным.
Пусть, мой пример не отвечает требованиям задания, зато он гораздо лучше приспособлен для юнит-тестов.
И последнее - для TDD (Test Driven Development) лучше всего подходить к разработке "Test First". Сначала тесты, потом код. Сначала вы пишете набор тестов, исполнение которых гарантирует вам правильную работу программы, а уж затем вы пишете код так, чтобы эти тесты выполнялись.

Answer 3

[Роман]

Unit тесты, судя из названия, предназначены для тестирования функциональности модулей и отдельных методов, ни одним unittest фреймворком вы не провернете это, а у вас получается интеграционный тест. Пишем еще одно приложение, которое будет перехватывать ввод/вывод с консоли и тестировать ваше приложение. Вот для примера

https://www.codeproject.com/Questions/720259/How-t...