Repository navigation
Conversation
…entManager. A checker for the minimum number of images has been added (regexp does not work correctly)
c703b04 to
a1a9c07
Compare
HadronCollider
left a comment
HadronCollider
left a comment
There was a problem hiding this comment.
- не нужно создавать новый критерий при наличии существующего (который уже считает кол-во рисунков) - это указано в задаче
- см. ниже
| if self.file.page_counter() < 4: | ||
| return answer(False, "В отчете недостаточно страниц. Нечего проверять.") |
There was a problem hiding this comment.
замените на новый метод if not self.has_enough_pages(): - md почти никогда не будет содержать больше 4х страниц
| if image[1]: | ||
| continue | ||
| image_xrefs.add(image[0]) |
There was a problem hiding this comment.
не совсем понятная логика - о чем сигнализирует image[1] != 0?
для pdf с png с прозрачностью метод работает некорректно, поэтому кажется не стоит отбрасывать изображение из-за наличия маски
| from tests.util import create_report_file_info | ||
|
|
||
|
|
||
| class TestMinCountImageReportChecker: |
There was a problem hiding this comment.
Тестируйте PDF-метод без мока - нам нужно проверить как он работает с разными PDF (а не то как критерий будет отвечать на заготовленные ответы): альфа-канал, повтор одного xref, несколько размещений, неиспользуемые ресурсы, изображения в приложении
Плюс проверьте, считаются ли векторные схемы рисунками (текущий метод учитывает растровые)
No description provided.