-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Обновление BrightSide #81
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ну и подтянуть бы последние изменения
ну и |
a44aa3c
to
ed480c6
Compare
ed480c6
to
c887977
Compare
+ suspiciousAppsDir | ||
+ suspiciousSystemDir | ||
return jailbreakRelativelyFilesAndPaths | ||
.allSatisfy(FileManager.default.fileExists(atPath:)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
а тут логика точно не поменялась? раньше, получается, true возвращалось если хотя бы по одному из путей файл был, ну и в комменте так и указано к методу) а теперь - true ведь будет, если allSatisfy? -_-
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
как помню из переписки - Илья сказал что да, косяк есть, но он хотел покрыть тестами, чтобы получить ошибку на них, и исправить
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Поправил
1b452c3
to
8397abb
Compare
Добавлено еще больше директорий файлов, проверок. Протестил логику на Зените, все отрабатывает корректно По поводу Unit тестов, я не совсем понимаю нужны ли они тут? По сути вариант теста это замокать Filemanager и его ответы, не совсем понятно что нам дадут эти тестыы Передвинул проверку на симулятор в конец, тк при джейлбрейке проверки можно обойти эти проверки, и тесты остальные будут бесполезны.
Важно: Для открытия URLScheme необходимо добавить схемы в Plist, иначе 1 проверка с UIApplication.shared.canOpenURL(URL) будет всегда возвращать false
|
Добавил новые директории для проверки на Jailbreak в BrightSide