Hw12 13 14 15 calendar #14
Conversation
| @@ -0,0 +1,85 @@ | |||
| version: "2.4" | |||
There was a problem hiding this comment.
Used this version of docker-compose to have an ability to use healthcheck together with depends_on
| "github.com/heetch/confita" | ||
| "github.com/heetch/confita/backend/env" | ||
| "github.com/heetch/confita/backend/file" | ||
| "github.com/ilyakaznacheev/cleanenv" |
There was a problem hiding this comment.
I had to migrate to this package, because of problems with confita heetch/confita#77
| @@ -35,10 +35,11 @@ func ConvertTimeToTimestamp(ntime sql.NullTime) (result *timestamp.Timestamp, er | |||
| // Didn't want to use reflection. | |||
| func ConvertEventToProto(evt repository.Event) (*Event, error) { | |||
| result := &Event{ | |||
| ID: evt.ID, | |||
| UserID: evt.UserID, | |||
| ID: int32(evt.ID), | |||
There was a problem hiding this comment.
protocolbuffers/protobuf#2679
strange "feature", so need to move to int/int32
| ) | ||
|
|
||
| var ( | ||
| now = time.Now() |
There was a problem hiding this comment.
такое бывает опасно при тестах, которые могут идти часами, параллельно и афектить друг друга
бывают очень сложноотловимые баги :( которые лечатся, конечно, переносом этого выражения в более локальную область видимости
| return | ||
| func NewCalendar(fpath string) (*Calendar, error) { | ||
| var cfg Calendar | ||
| return &cfg, readConfig(fpath, &cfg) |
There was a problem hiding this comment.
красивый приём 👍
только жалко, что приходится тогда возвращать указатель, что в данном случае выглядит сомнительно
| } | ||
|
|
||
| func (r *MemoryRepo) GetUser(id int64) (User, error) { | ||
| r.mx.Lock() |
Домашнее задание №15 «Докеризация и интеграционное тестирование Календаря»
Критерии оценки
make up/make down- 3 баллаmake bdd. Команда возвращает верный код ответа - 1 баллИнтеграционные тесты покрывают бизнес сценарии:
Зачёт от 7 баллов