Обновление BrightSide #81
Conversation
chausovSurfStudio left a comment
chausovSurfStudio left a comment There was a problem hiding this comment.
ну и подтянуть бы последние изменения
| ну и |
Stampoo force-pushed the Update-BrightSide-utility branch 2 times, most recently from a44aa3c to ed480c6 Compare 
Stampoo force-pushed the Update-BrightSide-utility branch from ed480c6 to c887977 Compare Stampoo requested review from chausovSurfStudio and removed request for LastSprint | + suspiciousAppsDir | ||
| + suspiciousSystemDir | ||
| return jailbreakRelativelyFilesAndPaths | ||
| .allSatisfy(FileManager.default.fileExists(atPath:)) |
There was a problem hiding this comment.
а тут логика точно не поменялась? раньше, получается, true возвращалось если хотя бы по одному из путей файл был, ну и в комменте так и указано к методу) а теперь - true ведь будет, если allSatisfy? -_-
There was a problem hiding this comment.
как помню из переписки - Илья сказал что да, косяк есть, но он хотел покрыть тестами, чтобы получить ошибку на них, и исправить
KASAFF force-pushed the Update-BrightSide-utility branch from 1b452c3 to 8397abb Compare 
| Добавлено еще больше директорий файлов, проверок. Протестил логику на Зените, все отрабатывает корректно По поводу Unit тестов, я не совсем понимаю нужны ли они тут? По сути вариант теста это замокать Filemanager и его ответы, не совсем понятно что нам дадут эти тестыы Передвинул проверку на симулятор в конец, тк при джейлбрейке проверки можно обойти эти проверки, и тесты остальные будут бесполезны.
Важно: Для открытия URLScheme необходимо добавить схемы в Plist, иначе 1 проверка с UIApplication.shared.canOpenURL(URL) будет всегда возвращать false |
Добавил новые директории для проверки на Jailbreak в BrightSide