
guga
14.10.2016
11:03:09
?

Мытко
14.10.2016
11:03:19
Создавать отдельный конструктор, где я буду передавать объект с датой не принимается как ответ

Митко Соловец?
14.10.2016
11:03:28
я думаю гуга прав и надо потом отдельно посмотреть

Google

Митко Соловец?
14.10.2016
11:03:36
у меня частенько звук глючит

Pavel
14.10.2016
11:03:59
я буду потом смотреть

sss3 ?
14.10.2016
11:04:37
а когда появится доступ к видюшкам для тех кто покупал билет на саму конфу?

Митко Соловец?
14.10.2016
11:04:50
конкретно

Pavel
14.10.2016
11:05:03

Мытко
14.10.2016
11:05:16
окей
только я не могу это протестировать

Митко Соловец?
14.10.2016
11:05:32
в конструктор передай вчерашнюю дату, объект создай сегодняшний

Мытко
14.10.2016
11:05:34
если не поменяю через рефлексию

Митко Соловец?
14.10.2016
11:05:37
почему не можешь?

Мытко
14.10.2016
11:05:52

Google

Мытко
14.10.2016
11:05:55
они не нужны

Митко Соловец?
14.10.2016
11:06:08
в смысле лишний?

Мытко
14.10.2016
11:06:14
лишний

Митко Соловец?
14.10.2016
11:06:17
ты сказал, у тебя инициализируется в конструкторе

Мытко
14.10.2016
11:06:34
https://i.imgur.com/kCaOtYC.png
ага
вот так
https://i.imgur.com/MUbTaB7.png

Митко Соловец?
14.10.2016
11:07:25
это у объекта, да?

Мытко
14.10.2016
11:07:28
да

Pavel
14.10.2016
11:07:36

Митко Соловец?
14.10.2016
11:07:43
да погоди
тут можно и так
а логика какая на проверку и удаление?

Мытко
14.10.2016
11:08:12
старее чем день - удаляется

James Tiberius Kirk ?
14.10.2016
11:08:14
мы же в кевасе вроде Instant пользуемся, а не LocalDateTime

Мытко
14.10.2016
11:08:15
сейчас так

James Tiberius Kirk ?
14.10.2016
11:08:22
или у вас за такое по попе не шлепают?

Мытко
14.10.2016
11:08:41

Митко Соловец?
14.10.2016
11:08:45
ты можешь замокать вызов метода проверки

Google

James Tiberius Kirk ?
14.10.2016
11:08:48
мб

Митко Соловец?
14.10.2016
11:09:07
в идеале переопределить метод в тестовом бине

Мытко
14.10.2016
11:09:11

Митко Соловец?
14.10.2016
11:09:12
но я кода не вижу, мне тяжко
скинь код проверки
можно в лс

Мытко
14.10.2016
11:09:59
currentTime.minusDays(1).isBefore(object.getTimeCreation())
просто вызов такого дерьма

Митко Соловец?
14.10.2016
11:10:20
единица захардкожена
при этом что-то подменить мы не можем

Мытко
14.10.2016
11:10:43
я пока думаю
что делать

Митко Соловец?
14.10.2016
11:10:46
т.к. дата зашита
как и единица

Мытко
14.10.2016
11:11:06
вообще

Митко Соловец?
14.10.2016
11:11:10
я бы поступил так: еще один конструктор для теста
или вынести единицу как параметр метода

Мытко
14.10.2016
11:11:29
ну я могу через рефлексию поменять это гавно

Митко Соловец?
14.10.2016
11:11:38
это хуже
чем просто еще один конструктор

Google

Pavel
14.10.2016
11:11:55
ЗАЧЕМ МНЕ КОНСТРУКТОР
Я ЖЕ ОХУЕННЕН, Я ЛУЧШЕ ИСПОЛЬЗУЮ ОТРАЖЕНИЕ...

Митко Соловец?
14.10.2016
11:12:03
или поменять сигнатуру
но рефлексия - худший выбор
хоть и не запрещен в тестировании

Pavel
14.10.2016
11:12:16
рефлексия для этого - ппц

Мытко
14.10.2016
11:12:17
не понимаю почему

Igor
14.10.2016
11:12:30

Admin
ERROR: S client not available

Pavel
14.10.2016
11:12:32
я не понимаю, чем тебе конструктор для тестов мешает?

Митко Соловец?
14.10.2016
11:12:33
скорость работы теста, лишний код

Pavel
14.10.2016
11:12:33
это раз

Митко Соловец?
14.10.2016
11:12:34
гемморой
конструктор для тестов - абсолютно нормально
это ничего не аффектит и лучший выбор
можно поменять сигнатуру - но это может что-то заффектить

Pavel
14.10.2016
11:13:04
Напиши просто
/*
* this constructor made for test
*
*/

Мытко
14.10.2016
11:13:14
мне не нравится

Pavel
14.10.2016
11:13:14
телеграм дура, не умеет в хоткеи идеи

Митко Соловец?
14.10.2016
11:13:16
ну и рефлексия тут - самое зашкварное

Google

Мытко
14.10.2016
11:13:28
рефлексия тоже не нравится

Igor
14.10.2016
11:13:29

Мытко
14.10.2016
11:13:32
вот и спрашиваю

Pavel
14.10.2016
11:13:38
мне не нравится, что мне будут в рот совать железки, Удаляйте гланды через жопу.

Мытко
14.10.2016
11:14:01
вот это ор
пойду спрошу про конструктор для тестов у коворкеров

guga
14.10.2016
11:15:38
смузи не забудь

Мытко
14.10.2016
11:24:15
получил ответ

guga
14.10.2016
11:26:25
и что сказали?

Мытко
14.10.2016
11:26:36
ну они сказали, то что и паша сказал
но объяснили почему это лучше
и конструктор без даты я просто убрал

guga
14.10.2016
11:27:49
Вот и отлично.

Мытко
14.10.2016
11:28:16
и ещё вынес создание даты в отдельный метод, который я мокаю, чтобы в тестах всегда была одна и таже дата

guga
14.10.2016
11:29:55
А bootique не плохо так выглядит
нужно потыкать в него палочкой

Митко Соловец?
14.10.2016
11:44:42
Волков прозрел