search

enb / enb-bh

BH support for ENB
Other
15 stars 12 forks source link

Add tests for bh-server and bh-server-include techs #38

Closed sipayRT closed 9 years ago

sipayRT commented 9 years ago

По замечаниям из #32 :

IMPORTANT: тут тесты не собираются, т.к. нет инфраструктуры

blond commented 9 years ago

Отребейзь от мастера.

sipayRT commented 9 years ago

да, я сейчас еще внесу правки и закоммичу

blond commented 9 years ago

Написал небольшие замечания, в остальном :+1:

sipayRT commented 9 years ago

@blond внес поправки. В bh-server-include падает один тест (а не должен)

blond commented 9 years ago

@sipayRT, а тест для CommonJS? :)

sipayRT commented 9 years ago

@blond он есть в bh-server

blond commented 9 years ago

В bh-server-include падает один тест (а не должен)

Это баг в технологии.

Нужно добавить ещё тест на проверку, что если мы добавили/удалили шаблон, то всё пересобралось.

sipayRT commented 9 years ago

Нужно добавить ещё тест на проверку, что если мы добавили/удалили шаблон, то всё пересобралось.

@blond добавил

sipayRT commented 9 years ago

@blond сосквошил коммиты, тесты проходят