Add handling of error cases

eGavr commented 9 years ago

closes #11 #14

/cc @j0tunn

j0tunn commented 9 years ago

Вместо вручную написанных моков лучше запользовать sinon

j0tunn commented 9 years ago

По поводу флага isCompressionError и reduce. Во-первых вопрос: какая логика сжатия? "Натравливаем на файл алгоритмы и смотрим какой лучше сжал (независимо друг от друга)" или "натравливаем алгоритмы по очереди и смотрим какая комбинация алгоритмов лучше жмет"?

Натравливаем на файл алгоритмы и смотрим какой лучше сжал (независимо друг от друга)

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

упаковываем логику про reduce промисов в функцию runSequentially, она принимает список алгоритмов и файл, выдает список пожатых файлов (эти файлы при этом физически остаются на диске)
после этого ты выбираешь минимальный из списка (или выбираешь любые данные, которые тебе нужны), остальные удаляешь

натравливаем алгоритмы по очереди и смотрим какая комбинация алгоритмов лучше жмет

тогда ты не учитываешь разные комбинацие алгоритмов и тут нужно вообще много чего менять

eGavr commented 9 years ago

Логика сжатия - первый описанный алгоритм.

Перебирать все варианты комбинаций?) Сейчас пока пять алгоритмлв, а это уже 5! = 120 вариантов сжатий одного файла ^^ Мы будем ждать опцтимизацию нескольких файлов до утра)

Итак...

Натравливаем на файл алгоритмы и смотрим какой лучше сжал (независимо друг от друга)

Только подумал о твоем решении.) Спасибо большое за помошь

bem-archive / image-optim