Conversation
chausovSurfStudio
left a comment
There was a problem hiding this comment.
ну и подтянуть бы последние изменения
|
ну и |
a44aa3c to
ed480c6
Compare
ed480c6 to
c887977
Compare
| + 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.
как помню из переписки - Илья сказал что да, косяк есть, но он хотел покрыть тестами, чтобы получить ошибку на них, и исправить
1b452c3 to
8397abb
Compare
|
Добавлено еще больше директорий файлов, проверок. Протестил логику на Зените, все отрабатывает корректно По поводу Unit тестов, я не совсем понимаю нужны ли они тут? По сути вариант теста это замокать Filemanager и его ответы, не совсем понятно что нам дадут эти тестыы Передвинул проверку на симулятор в конец, тк при джейлбрейке проверки можно обойти эти проверки, и тесты остальные будут бесполезны.
Важно: Для открытия URLScheme необходимо добавить схемы в Plist, иначе 1 проверка с UIApplication.shared.canOpenURL(URL) будет всегда возвращать false |
Добавил новые директории для проверки на Jailbreak в BrightSide