что за скин? не болят глаза?

gocover.io

зависимость от фастхттп это конечно провал )

Тут-то эти возвраты ошибок тестировать как-то совсем не интересно, да и чему там ломаться.

зависимость от фастхттп это конечно провал )

Ну обычный http не оч же

зависимость от фастхттп это конечно провал )

список импортов, названный Requirements — вот это провал

но это уже обсуждалось

ну да мы уже выяснили что этот хайлоад он не тянет )

не знаю, где ты это выяснил и где он что не потянул

сарказм был

А в fasthttp не надо закрывать соединение после запроса?

очевидно ж что родного хттп там с головой хватит

А в fasthttp не надо закрывать соединение после запроса?

после запроса в какую сторону?

А в fasthttp не надо закрывать соединение после запроса?

Я пробовал, fasthttp намекает что "не парься, я сам всё закрываю"

после запроса в какую сторону?

Отсюда туда. Впрочем, я сходил доки почитал, не надо

угу

Ага, качется начинаю понимать

я б забил на 100% покрытие

да не будет 100%

но тут есть довольно большие участки не оттестированного кода

я б забил на 100% покрытие

Ну я прям по-хардкору до 100 выбивать и не буду, тяжеловато.

да не будет 100%

точнее, будет, если задрачивать покрытие

Но хотя бы процентов 85-90, думаю, будет достаточно

Нагенерить тестов

Нагенерить тестов

не прокатит

Потом тестов для тестов

Потом тестов для тестов

Рекурсия

Но хотя бы процентов 85-90, думаю, будет достаточно

да, будет достаточно.

фастхттп, документация и норм

я бы разбивал, а ты как хочешь

я бы разбивал, а ты как хочешь

Ну, ТЫЖЭКСПЕРТ, потому спрашиваю, чтобы опираться на твои привычки

угу, нарисуй красные линии, одну из них прозрачным, и одну из них зелёным

угу, нарисуй красные линии, одну из них прозрачным, и одну из них зелёным

Ну не начинай, а

85.2%

вообще — передав неправильный json. но тебе не нужно тестировать батарейки. лучше посмотри, нет ли других важных участков кода, которые не протестированы.

это все тоже батарейки, по сути.

ты можешь проверить часть, что в цикле?

ты можешь проверить часть, что в цикле?

Не уверен. Вообще если атрибуты есть, то они проходят тот самый кусок вполне успешно. Если их нет, то кусок игнорируется

да понятно.

окей, тащи PR

Сейчас ещё гляну пару мест

Маршлинг это тоже "батарейка"?

батарейки == встроенные библиотеки

подскажите как в го реализовать (?!word) в регулярном выражении?

lookahead нет в regexp

используй https://github.com/dlclark/regexp2

87,0%

используй https://github.com/dlclark/regexp2

спасибо

87,0%

PR притащил?

PR притащил?

Добивал последние тесты. Сайт не обновлял результаты, так что пришлось на версию Go постарше переключиться

Зато теперь 88.3%

Выше уже не смогу

PR притащил?

Так, а в какой раздел это лучше пихать? о_0

Так, а в какой раздел это лучше пихать? о_0

https://github.com/avelino/awesome-go/#third-party-apis

https://github.com/avelino/awesome-go/#third-party-apis

Simple wrapper around Telegraph API publishing platform. Сойдёт?

скорее Telegraph publishing platform API client

Блин, я только сейчас заметил что у меня ошибка билда из-за тестов на ошибки: https://travis-ci.org/toby3d/telegraph/jobs/186560278

Я или неправильные тесты написал (потому что я специально забил тесты с ошибочными данными для проверки реакции), либо надо как-то тревису сказать, чтобы он не ругался на это.

? перестарался где-то

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

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

Ну блин 9_6

Секунду, а в ошибочных тестах вмеcто t.Error надо юзать t.Log?

ERROR: S client not available

t.Error - для вывода ошибок тестов t.Log - просто для какой-нибудь информации

t.Error - для вывода ошибок тестов t.Log - просто для какой-нибудь информации

Ну это я знаю

t.Error - + инкрементит счетчик ошибок

Но для учёта теста как успешный тип лога надо менять?

нет

Я имею ввиду в тестах, где заведомо кривые входные данные и типа если правильно отыскалась ошибка, то юзать Log

выявлена ошибка во время теста - t.Error все норм, ничего делать не нужно

Хм, тогда чего Travis ругается, я понять не могу

t.Log - просто для какойнить хрени, дапустим отожрали столько-то памяти, метод вернул такой-то результат, и т.д.

t.Log - просто для какойнить хрени, дапустим отожрали столько-то памяти, метод вернул такой-то результат, и т.д.

Ну я понял, да.

Просто уточнил, а то мало ли

ДА ПОЧЕМУ, БЛИН https://travis-ci.org/toby3d/telegraph/jobs/186566652

Ну вот что ему не нравится? Линтер говорит всё в порядке, тесты даже для удобства разбил

Только PR открыть захотел, называется.

Или Я не так тесты пишу или что вообще.

Что делать-то? ?

Доки перерыл даже, ничего вразумительного

Что делать-то? ?

вот здесь https://github.com/toby3d/telegraph/blob/master/invalid_test.go#L14 должно быть != или ==? т.е по логике, создаем заведомо левый акк, знаем, что упадет, и таки оно падет, но это не должно быть ошибкой. С учетом того, что t.Error инкременитит кол-во ошибок, как было сказано ранее

Только PR открыть захотел, называется.

а когда откроешь?

вот здесь https://github.com/toby3d/telegraph/blob/master/invalid_test.go#L14 должно быть != или ==? т.е по логике, создаем заведомо левый акк, знаем, что упадет, и таки оно падет, но это не должно быть ошибкой. С учетом того, что t.Error инкременитит кол-во ошибок, как было сказано ранее

По логике !=, так как всё зависит от ответа сервера. Если запрос кривой, то вернется ошибка, что токена нет

получается там?

а когда откроешь?

Когда пофикшу

получается там?

Не очень.

По логике !=, так как всё зависит от ответа сервера. Если запрос кривой, то вернется ошибка, что токена нет

правильно. так как аккаунт заведомо неверен, ошибка в клиенте тогда, когда ошибка не вернулась

По логике !=, так как всё зависит от ответа сервера. Если запрос кривой, то вернется ошибка, что токена нет

там --- FAIL: TestCreateInvalidAccount (0.28s) invalid_test.go:15: SHORT_NAME_REQUIRED и ошибка там будет всегда, но тест-то пройден, и t.Error звать не надо, не?

правильно. так как аккаунт заведомо неверен, ошибка в клиенте тогда, когда ошибка не вернулась

Но она вернулась же

там --- FAIL: TestCreateInvalidAccount (0.28s) invalid_test.go:15: SHORT_NAME_REQUIRED и ошибка там будет всегда, но тест-то пройден, и t.Error звать не надо, не?

Почему не надо, если есть инфа, что аккаунт не создался по такой-то причине?

Почему не надо, если есть инфа, что аккаунт не создался по такой-то причине?

но ты же говоришь, что аккаунт не создан и это ошибка

в тесте с заведомо неправильными данными

но ты же говоришь, что аккаунт не создан и это ошибка

Бляяяять

Вы чертовски правы.

То есть мне надо равенство в сломаных тестах менять, да?