Всем привет, есть вопрос как правильно проверять файл на максимальный

Question

Всем привет, есть вопрос как правильно проверять файл на максимальный

размер, на расширение, прописывать эту логику в сервисе или с помощью анотации и писать кастомный валидатор в спринге, как чаще всего делают в реальных проектах?
Ещё вопрос есть сколько времени обычно уходит на покрытие кода тестами, например чтобы покрыть тестами 5 классов с 5 методами?

#java #programming #russian

0

27.07.2023

16 ответов

19 просмотров

Дмитрий

У нас в проекте есть white/black list, проверяем наличие расширения после имени файла. По тестам — бессмысленный вопрос. Что тебе эта метрика даст?

0

27.07.2023

Георгий Автор вопроса

Дмитрий
У нас в проекте есть white/black list, проверяем н...

А где проверяете расширение прямо в сервисе? По тестами да понимаю, просто когда пишу тесты уходит почему-то очень много времени, поэтому интересно

0

27.07.2023

Дмитрий

Георгий
А где проверяете расширение прямо в сервисе? По те...

Очень много времени - это сколько? Тоже ни о чем. ты же только учишься. С опытом времени будет уходить меньше.

0

27.07.2023

Дмитрий

Георгий
А где проверяете расширение прямо в сервисе? По те...

Можно сказать, в сервисе. Перед тем как что-то делать с файлом, сначала его проверяем.

0

27.07.2023

Георгий Автор вопроса

Ну вот на это я потратил минут 15, можете оценить правильность тесты и что бы вы тут изменили? Тут логика добавления объявления с изображением

0

27.07.2023

Дмитрий

Георгий

Ну вот на это я потратил минут 15, можете оценить ...

Я бы не писал файлы в базу. ассерт каждого поля делать не вижу смысла. Либо используй ReflectionEquals, либо делай ассерт того что записалось в базу с тем, что пришло из реквеста. Трай кетч я бы унес куда то вглубь, ближе к потенциальному источнику исключения. По моканию сервисов — можно сделать мок в @BeforeAll один раз, так как тебе с этими моками скорее всего нужно тестить и другие методы

0

27.07.2023

Георгий Автор вопроса

Дмитрий
Я бы не писал файлы в базу. ассерт каждого поля де...

У меня файл репозиторий пишет в minio, я не очень понял про проверку того что записалось в базу, сейчас как раз проверяю что записалось, ReflectionEquals не могу использовать, у меня там поля по разному у объектов называются, я же правильно понимаю что он по полям одинаковым просто сравнивает? вот трай кетч я тоже не могу убрать, потому что я тут получаю inputstream и отсюда ошибка, мне стоит тогда в filerepository не inputStream а MultipartFile сразу передавать?

0

27.07.2023

Георгий Автор вопроса

Дмитрий
Я бы не писал файлы в базу. ассерт каждого поля де...

про моки, это ты про то где у меня большой блок с when?

0

27.07.2023

Дмитрий

Георгий
У меня файл репозиторий пишет в minio, я не очень ...

ты ж маппером создаешь тип, который пишется в базу, как там могут быть поля разные?

0

27.07.2023

Георгий Автор вопроса

Дмитрий
ты ж маппером создаешь тип, который пишется в базу...

А получается это нету смысла проверять, только поля которые в сервисе как-то создались?

0

27.07.2023

Дмитрий

Георгий
У меня файл репозиторий пишет в minio, я не очень ...

В репозиторий лучше передавать сразу объект который надо сохранить. Если репозиторий сохраняет файлы, значит надо передавать в него файлы. Хотя, может бывает и другой подход. У меня нет опыта сохранения файлов в репозиторий

0

27.07.2023

Дмитрий

Я не понял этого сообщения

0

27.07.2023

Георгий Автор вопроса

Дмитрий
Я не понял этого сообщения

Ну я про то что если в базе сохраняется то что я передаю в дто, то на выходе не нужно проверять что выходные параметры равны входным?

0

27.07.2023

Дмитрий