Add tests for bh-client and bh-client-module techs #39
Conversation
|
@blond посмотри, плз. Для |
|
|
||
| it('mimic', function () { | ||
| var test = [ | ||
| 'chai.should();', |
There was a problem hiding this comment.
Почему тут не используется generateTest?
There was a problem hiding this comment.
потому что в этом тесте я вместо bh.apply() использую BEMHTML.apply()
There was a problem hiding this comment.
А если сделать третим аргументом? )
There was a problem hiding this comment.
там уже есть третий необязательный аргумент :) Добавлять еще один аргумент ради одного теста посчитал неразумным.
|
Нету тестов на кэш. |
@blond добавил |
sipayRT
changed the title
| }); | ||
|
|
||
| describe('caches', function () { | ||
| var mockBhCore = [ |
sipayRT
changed the title
| <script src="mocha.js"></script> | ||
| <script src="chai.js"></script> | ||
| <script src="ym.js"></script> | ||
| <script src="a.js"></script> |
There was a problem hiding this comment.
Давай назовём как-то внятнее :) Например, some-ym-lib.
|
Тесты падают, надо исправить ;) |
|
Не забудь добавить коментариев про запись из-за баги. |
понятия не имею почему они падают - локально все собирается (да и ошибки какие-то странные). Добавил комментарии и перезапустил сборку |
|
одну из ошибок поправил (мой косяк был), а остальные ошибки типа |
|
@blond всё! Если интересно в чем была проблема, то завтра расскажу :) |
Add tests for bh-client and bh-client-module techs
No description provided.