Skip to content
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

Тесты и корректная обработка символов для имен папок и файлов #128

Closed
MonkAlex opened this issue Jan 23, 2019 · 5 comments
Labels
area/core any issue related with core logic bug
Milestone

Comments

@MonkAlex
Copy link
Owner

Для линукса сейчас считается что можно использовать любые символы.

Для винды список частично с АПИ, частично забит руками. В целом, лучше реализовать и проверить список отсюда

@MonkAlex MonkAlex added the bug label Jan 23, 2019
@MonkAlex MonkAlex added this to the Deffered milestone Jan 23, 2019
@MonkAlex MonkAlex modified the milestones: Deffered, 1.48 Feb 2, 2019
MonkAlex added a commit that referenced this issue Mar 30, 2019
@MonkAlex
Copy link
Owner Author

Возможно стоит пересмотреть апи по работе с файлами-папками и их нормализацией.
Правда, это усложнит хранение путей, а этого бы сильно не хотелось.

@MonkAlex
Copy link
Owner Author

Печальный проверочный кейс, где всё навернулось:

  1. Добавить новую мангу
  2. Переименовать её в ...
  3. Скачать (или сжать, оба действия что-то наделают в корне)
  4. Вернуть родное имя

В итоге на 4 циклически само себя копирует сидит
на 3 пункте тоже не круто, что работает в корне папки загрузки -- сжимает рандомные папки (почему то не все)

@MonkAlex MonkAlex added the area/core any issue related with core logic label Mar 31, 2019
@MonkAlex
Copy link
Owner Author

MonkAlex commented Apr 6, 2019

Итого, что делаем:

  1. Валидируем папки настроек. Папки должны существовать или находиться внутри папки программы. Не должны оканчиваться относительными путями.
  2. Валидируем имена манги и её папок отдельно - никаких относительных путей и разделителей.

ПС: дополнительно - не даем мувать в папку и саму папку настроек. аналогично со сжатием.

MonkAlex added a commit that referenced this issue Apr 21, 2019
… folders - must be normalized and create on download.
@MonkAlex
Copy link
Owner Author

Валидации сделаны.
Перемещения и упаковку надо делать, плюс на них тоже нужны тесты.
Из явно кривых кейсов - перемещение "настроек" в папку конкретной манги. А потом обратно.

@MonkAlex
Copy link
Owner Author

MonkAlex commented May 7, 2019

В итоге просто запретил перемещать папку настроек в папку манги. Обратно кейса в итоге тоже не должно быть.
ПС: сжатие в итоге работает ровно с теми же папками, отдельно тестами не покрыто.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/core any issue related with core logic bug
Projects
None yet
Development

No branches or pull requests

1 participant