Added devMode option to bh-bundle tech

enb · May 25, 2015 · 9e7b7c4 · 9e7b7c4
1 parent b01227a
commit 9e7b7c4
Show file tree

Hide file tree

Showing 3 changed files with 54 additions and 5 deletions.
diff --git a/README.md b/README.md
@@ -57,6 +57,7 @@ nodeConfig.addTech(require('enb-bh/techs/bh-bundle'));
 * *String* **target** — Результирующий таргет. По умолчанию — `?.bh.js`.
 * *String* **filesTarget** — files-таргет, на основе которого получается список исходных файлов (его предоставляет технология `files`). По умолчанию — `?.files`.
 * *String* **sourceSuffixes** — суффиксы файлов, по которым строится `files`-таргет. По умолчанию — `['bh.js']`.
+* *Boolean* **devMode** — режим сборки. По умолчанию — `true`.
 * *String|Array* **mimic** — имена переменных для экспорта.
 * *String* **jsAttrName** — атрибут блока с параметрами инициализации. По умолчанию — `data-bem`.
 * *String* **jsAttrScheme** — Схема данных для параметров инициализации. По умолчанию — `json`. Форматы: `js` — Получаем `return { ... }`. `json` — JSON-формат. Получаем `{ ... }`.

diff --git a/techs/bh-commonjs.js b/techs/bh-commonjs.js
@@ -13,6 +13,7 @@
  *   (его предоставляет технология `files`). По умолчанию — `?.files`.
  * * *String* **sourceSuffixes** — суффиксы файлов, по которым строится `files`-таргет. По умолчанию — ['bh.js'].
  * * *String|Array* **mimic** — имена модулей для экспорта.
+ * * *Boolean* **devMode** — режим сборки. По умолчанию — `true`.
  * * *String* **jsAttrName** — атрибут блока с параметрами инициализации. По умолчанию — `data-bem`.
  * * *String* **jsAttrScheme** — Cхема данных для параметров инициализации. По умолчанию — `json`.
  * *                             Форматы:
@@ -41,10 +42,12 @@ module.exports = require('enb/lib/build-flow').create()
     .defineOption('jsAttrName', 'data-bem')
     .defineOption('jsAttrScheme', 'json')
     .defineOption('jsCls', 'i-bem')
+    .defineOption('devMode', true)
     .defineOption('escapeContent', false)
     .useFileList(['bh.js'])
     .builder(function (bhFiles) {
-        var node = this.node;
+        var node = this.node,
+            devMode = this._devMode;
 
         /**
          * Генерирует `require`-строку для подключения исходных bh-файлов.
@@ -63,7 +66,7 @@ module.exports = require('enb/lib/build-flow').create()
             }
 
             return [
-                'dropRequireCache(require, require.resolve("' + relPath + '"));',
+                devMode ? 'dropRequireCache(require, require.resolve("' + relPath + '"));' : '',
                 (pre || '') + 'require("' + relPath + '")' + (post || '') + ';'
             ].join(EOL);
         }
@@ -88,7 +91,7 @@ module.exports = require('enb/lib/build-flow').create()
         ].join(EOL);
 
         return [
-            dropRequireCacheFunc,
+            devMode ? dropRequireCacheFunc : '',
             buildRequire(coreFilename, 'var BH = '),
             'var bh = new BH();',
             'bh.setOptions({',

diff --git a/test/techs/bh-commonjs.test.js b/test/techs/bh-commonjs.test.js
@@ -1,10 +1,12 @@
-var fs = require('fs'),
+var path = require('path'),
+    fs = require('fs'),
     mock = require('mock-fs'),
     TestNode = require('enb/lib/test/mocks/test-node'),
     Tech = require('../../techs/bh-commonjs'),
     FileList = require('enb/lib/file-list'),
     bhCoreFilename = require.resolve('bh/lib/bh.js'),
     writeFile = require('../lib/write-file'),
+    dropRequireCache = require('enb/lib/fs/drop-require-cache'),
     EOL = require('os').EOL;
 
 describe('bh-commonjs', function () {
@@ -30,6 +32,42 @@ describe('bh-commonjs', function () {
         return assert(bemjson, html, templates);
     });
 
+    describe('mode', function () {
+        it('must drop require cache in dev mode', function () {
+            var opts = {
+                    devMode: true
+                },
+                bemjson = { block: 'block' },
+                html = '<a class="block"></a>';
+
+            return build([], opts)
+                .then(function () {
+                    return build(['bh.match("block", function(ctx) {ctx.tag("a");});'], opts);
+                })
+                .then(function (BH) {
+                    BH.apply(bemjson).must.equal(html);
+                });
+        });
+
+        it('must not drop require cache in prod mode', function () {
+            var opts = {
+                    devMode: false
+                },
+                bemjson = { block: 'block' },
+                html = '<div class="block"></div>';
+
+            dropRequireCache(require, path.resolve('blocks', 'block-0.bh.js'));
+
+            return build(['bh.match("block", function() {});'], opts)
+                .then(function () {
+                    return build(['bh.match("block", function(ctx) {ctx.tag("a");});'], opts);
+                })
+                .then(function (BH) {
+                    BH.apply(bemjson).must.equal(html);
+                });
+        });
+    });
+
     describe('jsAttr params', function () {
         it('must apply default jsAttrName and jsAttrScheme params', function () {
             var bemjson = { block: 'block', js: true },
@@ -265,7 +303,7 @@ function bhWrap(str) {
     return 'module.exports = function(bh) {' + str + '};';
 }
 
-function assert(bemjson, html, templates, options) {
+function build(templates, options) {
     var scheme = {
             blocks: {},
             bundle: {}
@@ -287,6 +325,13 @@ function assert(bemjson, html, templates, options) {
 
     return bundle.runTechAndRequire(Tech, options)
         .spread(function (BH) {
+            return BH;
+        });
+}
+
+function assert(bemjson, html, templates, options) {
+    return build(templates, options)
+        .then(function (BH) {
             BH.apply(bemjson).must.be(html);
 
             options && options.mimic && [].concat(options.mimic).forEach(function (name) {