Conversation
dirtymew
left a comment
dirtymew
left a comment
There was a problem hiding this comment.
Привет Вячеслав
по работе есть критичное замечание
код и тесты в go пишутся в разных файлах - если код в файле например precode.go то его тесты в pracode_test.go
нужно оформить код и тесты правильно
тест кейсы в целом ок описаны - там немного нужно поправить
| handler := http.HandlerFunc(mainHandle) | ||
| handler.ServeHTTP(responseRecorder, req) | ||
| // проверка на пустоту | ||
| require.NotEmpty(t, responseRecorder.Body.String()) |
There was a problem hiding this comment.
тут важен порядок - сначала код проверяем потом тело
require.Equal(t, responseRecorder.Code, http.StatusOK)
assert.NotEmpty(t, responseRecorder.Body)
| bodyEcpected := "wrong city value" | ||
| bodyActual := responseRecorder.Body.String() | ||
|
|
||
| assert.Equal(t, http.StatusBadRequest, responseRecorder.Code) |
| responseRecorder := httptest.NewRecorder() | ||
| handler := http.HandlerFunc(mainHandle) | ||
| handler.ServeHTTP(responseRecorder, req) | ||
|
|
There was a problem hiding this comment.
тут код ответа тоже нужно проверять
Привет, Антон! Рад Вас видеть! Исправил свои ошибки по Вашим замечаниям, новая версия уже "commit". С нетерпением, жду Вашей ревью)
dirtymew
left a comment
dirtymew
left a comment
There was a problem hiding this comment.
Привет Вячеслав
по работе все отлично - тесты описаны и работают как надо
есть один момент обрати внимание обязательно на комметарий
| @@ -0,0 +1,51 @@ | |||
| package main | |||
There was a problem hiding this comment.
файл t_test.go - нужно переименовать в precode_test.go
2 participants
No description provided.