Skip to content

Добавлен вариант использования go-chi/render в качестве библиотеки для работы с JSON endpoint#41

Open
sergey-vaysman wants to merge 1 commit into
Yandex-Practicum:mainfrom
sergey-vaysman:feature/iter4_test_edit_json_lib_list
Open

Добавлен вариант использования go-chi/render в качестве библиотеки для работы с JSON endpoint#41
sergey-vaysman wants to merge 1 commit into
Yandex-Practicum:mainfrom
sergey-vaysman:feature/iter4_test_edit_json_lib_list

Conversation

@sergey-vaysman

@sergey-vaysman sergey-vaysman commented Jan 6, 2023

Copy link
Copy Markdown

В рамках третьего инкремента при работе с go-chi от наставника Дениса Алексеева был совет реализовывать rest-endpoint по примеру из библиотеки (https://github.com/go-chi/chi/blob/master/_examples/rest/main.go). Соответственно, преобразование запроса в объект и обратно было переведено в go-chi/render (который в свою очередь использует encoding/json).

Предлагаю добавить еще один вариант для проверки использования библиотеки JSON в тест. Т.к. текущее поведение теста (ошибка Не найдено использование известных библиотек кодирования JSON) не стыкуется с рекоммендациями наставника. Приходится делать workaround с добавлением неиспользуемого кода с нужной для теста зависимостью.

…я работы с JSON endpoint

В рамках третьего спринта при работе с go-chi от наставника Дениса Алексеева был совет реализовывать rest-endpoint по примеру из библиотеки (https://github.com/go-chi/chi/blob/master/_examples/rest/main.go). Соответственно, преобразование запроса в объект и обратно было переведено в go-chi/render (который в свою очередь использует encoding/json).

Предлагаю добавить еще один вариант для проверки использования библиотеки JSON в тест. Т.к. текущее поведение теста (ошибка Не найдено использование известных библиотек кодирования JSON) не стыкуется с рекоммендациями наставника. Приходится делать workaround с добавлением неиспользуемого кода с нужной для теста зависимостью.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant