class-builder: elem === '' means no elem

Fix: #152
bem · Dec 22, 2015 · 6c427cc · 6c427cc
1 parent 199c068
commit 6c427cc
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/lib/bemxjst/class-builder.js b/lib/bemxjst/class-builder.js
@@ -5,7 +5,7 @@ function ClassBuilder(options) {
 exports.ClassBuilder = ClassBuilder;
 
 ClassBuilder.prototype.build = function build(block, elem) {
- if (elem === undefined)
+ if (!elem)
  return block;
  else
  return block + this.elemDelim + elem;

diff --git a/test/bemhtml/runtime-test.js b/test/bemhtml/runtime-test.js
@@ -154,6 +154,17 @@ describe('BEMHTML compiler/Runtime', function() {
  ], '<div class="b1"></div>');
  });
 
+ it('should assume elem=\'\' is a falsey value', function () {
+ test(function() {
+ block('b1').elem('e1').def()(function() {
+ return applyCtx(this.extend(this.ctx, {
+ block: 'b2',
+ elem: ''
+ }));
+ });
+ }, { block: 'b1' }, '<div class="b1"></div>');
+ });
+
  it('should properly save context while render plain html items', function() {
  test(function() {
  }, {