These are chat archives for BicycleStudio/Ewe

5th
Nov 2015
Sergey Vaytsel
@Telhar
Nov 05 2015 02:41
еще меня немного беспокоит тот факт, что коммиты в ветки проекта Ewe не учитываются в статистике Github. надо посмотреть, может, они будут учитываться, если сделать форк проекта, и работать там, и делать пулреквесты из форка. Надо будет сегодня разобраться, т.к. для меня статистика github'а - элемент мотивации)
Sergey Vaytsel
@Telhar
Nov 05 2015 04:16

Еще, по поводу ревью #57. Там я использовал тестовый проект, функционал к которому по умолчанию предоставляется студией и .NET Framework. Мне немного не нравится то, что приходится писать тесты на смеси языков .NET и C++, что рядом сразу и

public ref class TestClass

и

class TestData

Есть множество тестовых фреймворков на С++, и я, пожалуй, заведу задачку, чтобы исследовать популярные тестовые фреймворки, написать статью в вики с их анализом. Потом уже можно будет выбрать фреймворк и заводить задачки на переписывание тестов на него.

Sergey Vaytsel
@Telhar
Nov 05 2015 04:22
Хотя и, вполне возможно, что и текущая реализация тестов удобоварима. Дополнительных зависимостей не требует.
НО! - будет только под windows работать, т.к. .NET (хотя они и переехали на github - https://github.com/dotnet и поговаривают о том что будет реализация по *nix).
Sergey Vaytsel
@Telhar
Nov 05 2015 05:13
https://github.com/dotnet/coreclr - CLR есть под кучу платформ, проблем с переносимостью быть не должно.
Sergey Vaytsel
@Telhar
Nov 05 2015 08:09
и да, мы pull request'ы прибиваем к задачам, но задачи при мёрже не закрываются.
чтобы они закрывались, необходимо в коммите писать про неё
https://help.github.com/articles/closing-issues-via-commit-messages/
Valentin Zaborcev
@Swipes13
Nov 05 2015 14:54
по сути, тесты для меня в новинку. нехорошо, что clr, но я с ним жить могу) тем более, если только тесты на нём) как-то много магии, в сб хотел бы обсудить с тобой всю эту штуку)
один вопрос ток про вынесение getInsctance() и ещё, value в прайвате опять без подчёркивания) но это мелочь)
CLR != .NET же)
Sergey Vaytsel
@Telhar
Nov 05 2015 14:57
фреймворков горы:
google test, nunit, testng, Microsoft Unit Testing Framework, cppunit и т.д.
большой список есть тут:
https://en.wikipedia.org/wiki/List_of_unit_testing_frameworks#C.2B.2B
clr это одна из компонент
там отношение вложенности между ними
Sergey Vaytsel
@Telhar
Nov 05 2015 15:03
да давай сразу обсудим, прямо тут. в чем дело?
Valentin Zaborcev
@Swipes13
Nov 05 2015 15:06
юнит тесты на паттернс #47, реализация только синглтона, в силу того, что только синглтон имеет реализацию? а так - по идее, один проект автотестов на всю либу, в случае с какой-нибудь логикой - это будет большой зверь, верно?
Sergey Vaytsel
@Telhar
Nov 05 2015 15:07
нет, там не только синглтон имеет реализацию, просто я написал только для синглтона. на самом деле, можно в отдельную задачу вынести автотесты на другие паттерны.
да, 1 проект автотестов - 1 тестируемый проект
там будет просто много файлов тестов на разные штуки
это не проблема
Valentin Zaborcev
@Swipes13
Nov 05 2015 15:09
понял. на самом деле, прочитав в очередной раз вики, уже всё понятнее стало, видимо инфа уложилась) вчера когда мельком смотрел как-то всё было непонятно)
Sergey Vaytsel
@Telhar
Nov 05 2015 15:09
да, как то много нового зараз)
мне прям очень понравилась задача эта) сразу столько всего) там одна задача с синглтоном чего стоила, столько удовольствия получил, пока делал))
Valentin Zaborcev
@Swipes13
Nov 05 2015 15:11
ух :)) это круто)
Sergey Vaytsel
@Telhar
Nov 05 2015 15:20
Валь, не надо было мёржить..
Valentin Zaborcev
@Swipes13
Nov 05 2015 15:21
почему? создай более мелкие задачи
Sergey Vaytsel
@Telhar
Nov 05 2015 15:22
я хотел внести все те правки по комментариям, что я написал. более мелкие задачи не нужны, они всё захламляют, слишком мелкие получатся. в общем, я без задачи еще один пулреквест сделаю, где поправлю тесты.
Valentin Zaborcev
@Swipes13
Nov 05 2015 15:23
хорошо, извини) видимо, мне надо-таки посппать, за 36 часов ни часа сна)
Sergey Vaytsel
@Telhar
Nov 05 2015 15:23
йо, конечно тебе надо поспать))
Valentin Zaborcev
@Swipes13
Nov 05 2015 15:24
займусь этой задачей прямо сейчас))
Sergey Vaytsel
@Telhar
Nov 05 2015 15:24
я уверен, ты с ней справишься)) удачи)
Sergey Vaytsel
@Telhar
Nov 05 2015 16:32
#60 поправил тесты.
Sergey Vaytsel
@Telhar
Nov 05 2015 19:54
http://puu.sh/laBo0/17c2303f68.png
Наш проект при запросе 4й.
Выше нас только msdn и хабр, ну и тостер)
Sergey Vaytsel
@Telhar
Nov 05 2015 20:01
забавно, что легко ищется проект в гугле. всё таки github способствует распространению, серьезно так)