[#425] new tech "write-file" #426

pavelpower · 2016-01-19T16:55:47Z

No description provided.

pavelpower · 2016-01-19T22:38:20Z

@blond посмотри верно ли сделан тест?

obenjiro · 2016-01-20T00:28:21Z

test/techs/write-file.test.js

+            .then(function () {
+
+                // При оспользовании "mockFS.fs().statSync" выскакивает ошибка
+                // mockFS.fs().statSync('bundle/bundle.txt').isFile().should.be.ok;


Я к сожалению так и не разобрался почему mockFS.fs() фейлится но основная проблема как мне кажется в другом. MockFS манки патчит родной fs модуль ( для этого мы вызываем mockFs() и mockFs.restore() - чтобы добавить/удалить манки патчинг )

Заменив

mockFS.fs().statSync('bundle/bundle.txt').isFile().should.be.ok;

на

require('fs').statSync('bundle/bundle.txt').isFile().should.be.ok;

Все начинает работать как нужно

Также можно убедиться в том что нужно использовать fs модуль посмотрев тесты самого mockFs
https://github.com/tschaub/mock-fs/blob/master/test/integration/filecount.js#L3

@AiBoy спасибо за ревью! Действительно, так тест проходит!

blond · 2016-01-28T13:22:33Z

techs/write-file.js

+    enb = require('../lib/api'),
+    vfs = enb.asyncFs;
+
+module.exports = inherit(enb.BaseTech, {


Можно проще переписать технологию с помощью build-flow.

blond · 2016-01-28T13:31:28Z

@pavelpower написал замечания. И ещё надо добавить документацию в REAMDE.md.

pavelpower · 2016-01-28T13:42:57Z

@blond спасибо, поправлю

pavelpower · 2016-01-30T23:25:48Z

@blond попробовал сделать так

var enb = require('../lib/api'),
    vfs = enb.asyncFs;

module.exports = enb.buildFlow.create()
    .name('write-file')
    .target('target', '?.target')
    .defineRequiredOption('target')
    .defineRequiredOption('content')
    .defineOption('fileOptions', { encoding: 'utf8' })
    .builder(function () {
        var node = this.node,
            target = node.unmaskTargetName(this._target),
            filename = node.resolvePath(target);

        return vfs.write(filename, this._content, this._fileOptions)
            .then(function () {
                node.resolveTarget(target);
            }, function () {
                node.rejectTarget(target, new Error('file not write: ' + filename));
            });
    })
    .createTech();

Но возникла проблема в том, что такая реализация вызывает 'vfs.write' много раз.

Сделал тестовый проектик, для того, чтобы понять почему не проходит тест на проверку контента.

.enb/make.js

module.exports = function(config) {
    // Настраиваем сборку бандла
    config.node('bundle', function(nodeConfig) {
        // Декларируем модули технологий,
        // которые могут участвовать в сборке таргетов.

        nodeConfig.addTechs([
            [require('enb/techs/write-file'), {
                content: 'R0lGODlhAQABAIAAAP///////yH5BAEKAAEALAAAAAABAAEAAAICTAEAOw==',
                target: '?.png',
                fileOptions: {
                    encoding: 'base64'
                }
            }],
            [require('enb/techs/write-file'), {
                content: '## Test \n this a test text',
                target: '?.md',
                fileOptions: {
                    encoding: 'utf8'
                }
            }]
        ]);

        // Объявляем таргеты, которые хотим собрать.
        nodeConfig.addTargets(['?.png', '?.md']);
    });
};

Оказалось, что в целевых файлах undefined.

В рамках проекта поставил console.log(args) вот сюда: https://github.com/dfilatov/vow-fs/blob/master/lib/fs.js#L28

Вот, что получил:

 enb make

[ ‘~/Projects/opensource/test-enb/.enb/tmp',
  undefined,
  [Function] ]
[ ‘~/Projects/opensource/test-enb/.enb/tmp',
  [Function] ]
02:54:06.249 - build started
[ ‘~/Projects/opensource/test-enb/bundle',
  undefined,
  [Function] ]
[ ‘~/Projects/opensource/test-enb/bundle',
  [Function] ]
[ ‘~/Projects/opensource/test-enb/bundle/bundle.png',
  'R0lGODlhAQABAIAAAP///////yH5BAEKAAEALAAAAAABAAEAAAICTAEAOw==',
  { encoding: 'base64' },
  [Function] ]
[ ‘~/Projects/opensource/test-enb/bundle/bundle.md',
  '## Test \n this a test text',
  { encoding: 'utf8' },
  [Function] ]
02:54:06.276 - [rebuild] [bundle/bundle.png] write-file
02:54:06.276 - [rebuild] [bundle/bundle.md] write-file
[ ‘~/Projects/opensource/test-enb/bundle/bundle.png',
  'undefined',
  'utf8',
  [Function] ]
[ ‘~/Projects/opensource/test-enb/bundle/bundle.md',
  'undefined',
  'utf8',
  [Function] ]

Заметь сколько раз вызывается плагин!

Может вариант использования enb.buildFlow не очень хорошая идея?

Или ее надо готовить иначе?

pavelpower · 2016-01-31T00:01:27Z

Сделал трейс вызова writeFile:

Trace: [ '~/Projects/opensource/test-enb/bundle/bundle.md',
  '## Test \n this a test text',
  { encoding: 'utf8' },
  [Function] ]
    at Object.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25)
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:46:35
    at Object.vow.invoke (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:1050:20)
    at Object.Queue._runTask (~/Projects/opensource/test-enb/node_modules/vow-queue/lib/queue.js:204:27)
    at Object.Queue._run (~/Projects/opensource/test-enb/node_modules/vow-queue/lib/queue.js:186:18)
    at processImmediate [as _immediateCallback] (timers.js:383:17)
03:00:16.378 - [rebuild] [bundle/bundle.md] write-file
Trace: [ '~/Projects/opensource/test-enb/bundle/bundle.md',
  'undefined',
  'utf8',
  [Function] ]
    at Object.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25)
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:46:35
    at Object.vow.invoke (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:1050:20)
    at Object.Queue._runTask (~/Projects/opensource/test-enb/node_modules/vow-queue/lib/queue.js:204:27)
    at Object.Queue._run (~/Projects/opensource/test-enb/node_modules/vow-queue/lib/queue.js:186:18)
    at processImmediate [as _immediateCallback] (timers.js:383:17)

pavelpower · 2016-01-31T00:21:39Z

@blond кстати при текущей реализации тот же trace работает два раза для папок но один раз для файла.

Выглядит так:

> enb make

Trace: [ '~/Projects/opensource/test-enb/.enb/tmp',
  undefined,
  [Function] ]
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:316:24
    at Array.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:712:56)
    at Immediate.callFns [as _onImmediate] (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:23:35)
    at processImmediate [as _immediateCallback] (timers.js:383:17)
Trace: [ '~/Projects/opensource/test-enb/.enb/tmp',
  [Function] ]
    at Object.stat (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25)
    at Object.module.exports.isDir (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:234:21)
    at onFailed (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:307:28)
    at Array.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:712:56)
    at Immediate.callFns [as _onImmediate] (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:23:35)
    at processImmediate [as _immediateCallback] (timers.js:383:17)
03:19:37.083 - build started
Trace: [ '~/Projects/opensource/test-enb/bundle',
  undefined,
  [Function] ]
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:316:24
    at Array.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:712:56)
    at Immediate.callFns [as _onImmediate] (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:23:35)
    at processImmediate [as _immediateCallback] (timers.js:383:17)
Trace: [ '~/Projects/opensource/test-enb/bundle',
  [Function] ]
    at Object.stat (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25)
    at Object.module.exports.isDir (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:234:21)
    at onFailed (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:307:28)
    at Array.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:712:56)
    at Immediate.callFns [as _onImmediate] (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:23:35)
    at processImmediate [as _immediateCallback] (timers.js:383:17)
Trace: [ '~/Projects/opensource/test-enb/bundle/bundle.md',
  '## Test \n this a test text',
  { encoding: 'utf8' },
  [Function] ]
    at Object.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25)
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:46:35
    at Object.vow.invoke (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:1050:20)
    at Object.Queue._runTask (~/Projects/opensource/test-enb/node_modules/vow-queue/lib/queue.js:204:27)
    at Object.Queue._run (~/Projects/opensource/test-enb/node_modules/vow-queue/lib/queue.js:186:18)
    at processImmediate [as _immediateCallback] (timers.js:383:17)

blond · 2016-02-04T13:10:39Z

Но возникла проблема в том, что такая реализация вызывает 'vfs.write' много раз.

Это логичное поведение для build-flow. builder записывает возвращаемый в него результат.

Чтобы переопределить это поведение, нужно задекларировать метод saver:

var enb = require('../lib/api'),
    vfs = enb.asyncFs;

module.exports = enb.buildFlow.create()
    .name('write-file')
    .target('target', '?.target')
    .defineRequiredOption('target')
    .defineRequiredOption('content')
    .defineOption('fileOptions', { encoding: 'utf8' })
    .builder(function () {
        return this._content;
    })
    .saver(function (filename, content) {
        return vfs.write(filename, content, this._fileOptions);
    })
    .createTech();

blond · 2016-02-04T13:12:38Z

кстати при текущей реализации тот же trace работает два раза для папок но один раз для файла.

Это связано с технологией write-file или отдельно?

pavelpower · 2016-02-05T22:12:58Z

@blond Решение с .saver сработало. все теперь работает как надо.

Это связано с технологией write-file или отдельно?

Похоже на то, что это связано не с технологией write-file.

Бандлы для нод так же создаются по два раза:

.enb/make.js:

module.exports = function(config) {
    // Настраиваем сборку бандла
    config.node('bundle', function(nodeConfig) {

    });
};

console:

> enb make

Trace: ----> [ '~/Projects/opensource/test-enb/.enb/tmp',
  undefined,
  [Function] ]
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:316:24
    at Array.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:712:56)
    at Immediate.callFns [as _onImmediate] (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:23:35)
    at processImmediate [as _immediateCallback] (timers.js:383:17)
Trace: ----> [ '~/Projects/opensource/test-enb/.enb/tmp',
  [Function] ]
    at Object.stat (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25)
    at Object.module.exports.isDir (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:234:21)
    at onFailed (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:307:28)
    at Array.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:712:56)
    at Immediate.callFns [as _onImmediate] (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:23:35)
    at processImmediate [as _immediateCallback] (timers.js:383:17)
01:04:53.166 - build started
Trace: ----> [ '~/Projects/opensource/test-enb/bundle',
  undefined,
  [Function] ]
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25
    at ~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:316:24
    at Array.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:712:56)
    at Immediate.callFns [as _onImmediate] (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:23:35)
    at processImmediate [as _immediateCallback] (timers.js:383:17)
Trace: ----> [ '~/Projects/opensource/test-enb/bundle',
  [Function] ]
    at Object.stat (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:28:25)
    at Object.module.exports.isDir (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:234:21)
    at onFailed (~/Projects/opensource/test-enb/node_modules/vow-fs/lib/fs.js:307:28)
    at Array.<anonymous> (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:712:56)
    at Immediate.callFns [as _onImmediate] (~/Projects/opensource/test-enb/node_modules/vow/lib/vow.js:23:35)
    at processImmediate [as _immediateCallback] (timers.js:383:17)

blond · 2016-02-18T13:36:49Z

Бандлы для нод так же создаются по два раза:

Почему так мыслей нет, предлагаю поискать это отдельно. Например, может кто-то вызывает makePlatform.init 2 раза. Или проблема в projectConfig.

pavelpower · 2016-02-18T13:38:42Z

@blond у меня предположение, что проблема в vow-fs. На самом деле в текущей версии тоже самое происходит.

blond · 2016-02-18T13:41:47Z

test/techs/write-file.test.js

+
+        bundle = new MockNode('bundle');
+        resolveSpy = sinon.spy(bundle, 'resolveTarget');
+        rejectSpy = sinon.spy(bundle, 'rejectTarget');


resolveSpy и rejectSpy не нужены

blond · 2016-02-18T13:43:06Z

Написал замечания по коду.

blond · 2016-02-18T13:43:31Z

у меня предположение, что проблема в vow-fs. На самом деле в текущей версии тоже самое происходит.

@pavelpower, не очень понимаю как это связано =)

tadatuta · 2016-02-18T16:11:18Z

Бандлы для нод так же создаются по два раза:
Почему так мыслей нет, предлагаю поискать это отдельно. Например, может кто-то вызывает makePlatform.init 2 раза. Или проблема в projectConfig.

Они всегда вызываются по 2 раза, воспроизводится давно и при любых вариациях конфига.
Почему — не знаю, но оно мне изрядно портило жизнь.

blond · 2016-02-22T07:59:47Z

@pavelpower 👍

Соскваш коммиты и вливаем.

pavelpower · 2016-02-22T08:00:48Z

:okay:

pavelpower · 2016-02-22T08:09:21Z

@blond done

[#425] new tech "write-file"

blond · 2016-02-22T08:18:19Z

pavelpower · 2016-02-22T08:23:51Z

👍

blond added the review label Jan 19, 2016

pavelpower closed this Jan 19, 2016

pavelpower reopened this Jan 19, 2016

blond added review and removed review labels Jan 19, 2016

obenjiro reviewed Jan 20, 2016
View reviewed changes

pavelpower mentioned this pull request Jan 20, 2016

Добавить технологию write-file #425

Closed

blond reviewed Jan 28, 2016
View reviewed changes

blond added ready and removed review labels Jan 28, 2016

pavelpower mentioned this pull request Jan 31, 2016

Bag or Feature? bem-site/bem-forum-content-ru#844

Open

blond added review and removed ready labels Feb 15, 2016

blond reviewed Feb 18, 2016
View reviewed changes

[enb#425] create tech write-file

4812715

pavelpower force-pushed the 425_techs-write-file branch from 5f5690c to 4812715 Compare February 22, 2016 08:07

blond added a commit that referenced this pull request Feb 22, 2016

Merge pull request #426 from pavelpower/425_techs-write-file

8394e12

[#425] new tech "write-file"

blond merged commit 8394e12 into enb:master Feb 22, 2016

blond removed the review label Feb 22, 2016

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[#425] new tech "write-file" #426

[#425] new tech "write-file" #426

pavelpower commented Jan 19, 2016

pavelpower commented Jan 19, 2016

obenjiro Jan 20, 2016

pavelpower Jan 20, 2016

blond Jan 28, 2016

blond commented Jan 28, 2016

pavelpower commented Jan 28, 2016

pavelpower commented Jan 30, 2016

pavelpower commented Jan 31, 2016

pavelpower commented Jan 31, 2016

blond commented Feb 4, 2016

blond commented Feb 4, 2016

pavelpower commented Feb 5, 2016

blond commented Feb 18, 2016

pavelpower commented Feb 18, 2016

blond Feb 18, 2016

blond commented Feb 18, 2016

blond commented Feb 18, 2016

tadatuta commented Feb 18, 2016

blond commented Feb 22, 2016

pavelpower commented Feb 22, 2016

pavelpower commented Feb 22, 2016

blond commented Feb 22, 2016

pavelpower commented Feb 22, 2016

[#425] new tech "write-file" #426

[#425] new tech "write-file" #426

Conversation

pavelpower commented Jan 19, 2016

pavelpower commented Jan 19, 2016

obenjiro Jan 20, 2016

Choose a reason for hiding this comment

pavelpower Jan 20, 2016

Choose a reason for hiding this comment

blond Jan 28, 2016

Choose a reason for hiding this comment

blond commented Jan 28, 2016

pavelpower commented Jan 28, 2016

pavelpower commented Jan 30, 2016

pavelpower commented Jan 31, 2016

pavelpower commented Jan 31, 2016

blond commented Feb 4, 2016

blond commented Feb 4, 2016

pavelpower commented Feb 5, 2016

blond commented Feb 18, 2016

pavelpower commented Feb 18, 2016

blond Feb 18, 2016

Choose a reason for hiding this comment

blond commented Feb 18, 2016

blond commented Feb 18, 2016

tadatuta commented Feb 18, 2016

blond commented Feb 22, 2016

pavelpower commented Feb 22, 2016

pavelpower commented Feb 22, 2016

blond commented Feb 22, 2016

pavelpower commented Feb 22, 2016