BEMXJST: production option (fix for #298)

lib/bemxjst/index.js

@@ -401,7 +401,10 @@ BEMXJST.prototype.runOne = function runOne(json) {
     ent.init(block, elem);
   }
 
-  var res = ent.run(context);
+  var res = this.options.production === true ?
+    this.tryRun(context, ent) :
+    ent.run(context);
+
   context.ctx = oldCtx;
   context.block = oldBlock;
   context.elem = oldElem;
@@ -415,6 +418,21 @@ BEMXJST.prototype.runOne = function runOne(json) {
   return res;
 };
 
+BEMXJST.prototype.tryRun = function tryRun(context, ent) {
+  try {
+    return ent.run(context);
+  } catch (e) {
+    console.error('BEMXJST ERROR: cannot render ' +
+      [
+        'block ' + context.block,
+        'elem ' + context.elem,
+        'mods ' + JSON.stringify(context.mods),
+        'elemMods ' + JSON.stringify(context.elemMods)
+      ].join(', '), e);
+    return '';
+  }
+};
+
 BEMXJST.prototype.renderContent = function renderContent(content, isBEM) {
   var context = this.context;
   var oldPos = context.position;

test/api-apply-test.js

@@ -1,21 +1,10 @@
 var assert = require('assert');
-var bemxjst = require('../').bemhtml;
+var bemhtml = require('../').bemhtml;
 
 describe('API apply', function() {
-  it('should throw error with one apply', function() {
-    // There is no bemxjst.compile()
-    assert.throws(function() {
-      bemxjst.apply({ block: 'b' });
-    });
-  });
-
-  it('should throw errors with many applies', function() {
-    // There is no bemxjst.compile()
-    assert.throws(function() {
-      bemxjst.apply({ block: 'b' });
-    });
-    assert.throws(function() {
-      bemxjst.apply({ block: 'b' });
-    });
+  it('should support apply', function() {
+    var templates = bemhtml.compile();
+    var html = templates.apply({ block: 'b' });
+    assert.equal(html, '<div class="b"></div>');
   });
 });

test/api-compile-test.js

@@ -116,4 +116,35 @@ describe('API compile', function() {
     assert.equal(template.apply({ block: 'a' }),
       '<a class="a"></a>');
   });
+
+  describe('Production mode', function() {
+    it('should render even if error in one block', function() {
+      var template = bemxjst.compile(function() {
+        block('b1').attrs()(function() {
+          var attrs = applyNext();
+          attrs.undef.foo = 'bar';
+          return attrs;
+        });
+      }, { production: true });
+
+      assert.equal(template.apply({
+        block: 'page',
+        content: { block: 'b1' }
+      }), '<div class="page"></div>');
+    });
+
+    it('should throw error with one apply if production mode off', function() {
+      var template = bemxjst.compile(function() {
+        block('b1').attrs()(function() {
+          var attrs = applyNext();
+          attrs.foo = 'bar';
+          return attrs;
+        });
+      });
+
+      assert.throws(function() {
+        template.apply({ block: 'b1' });
+      });
+    });
+  });
 });