Поправить TaskManagerTests

[aleksandra-riazanova]

Ответ от преподавателя:

Замечания:

1. func makeSUT(taskList: [Task] = [Task]()) -> ITaskManager - мы тестируем конкретные классы
2. Не надо расширять тестируемые классы ради тестов, тестовые данные нужно готовить в тестах и размещать по месту, а это плохая практика:

// MARK: - TaskStub

extension Task {

    static func getTasksStub() -> [Task] {
        let tasks = [
            Task(title: "01 Task", completed: true),
            Task(title: "02 Task"),
            Task(title: "03 Task", completed: true),
            Task(title: "04 Task"),
        ]

        return tasks
    }
}

// MARK: - ImportantTaskStub

extension ImportantTask {

    static func getImportantTasksStub() -> [ImportantTask] {
        let tasks = [
            ImportantTask(
                title: "01 Task",
                taskPriority: .medium
            ),
            ImportantTask(
                title: "02 Task",
                taskPriority: .low
            ),
            ImportantTask(
                title: "03 Task",
                taskPriority: .high
            )
        ]

        return tasks
    }
}

Памятка по выполнению задания:

TaskManagerTests Проверяем инициализацию

• без задач
• со списком задач / через метод allTasks

Каждый метод должен быть проверен

• добавление одной задачи
• добавление списка задач
• удаление задачи
• получение выполненных задач (точно выполненные, количество)
• получение невыполненных зада (точно невыполненные, количество)
• Также можно проверить наличие конкретных задач в полученных списках

Тестовые данные можно создавать/хранить

• в приватных свойствах класса теста
• в самом тестовом методе
• в приватных расширениях: как вычисляемые или через enum + static

При этом важно помнить про дублирование кода, если методы используют одинаковые тестовые данные, то лучше выносить их за методы.