Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@blond
Copy link
Member

@blond blond commented Feb 20, 2016

No description provided.

@blond blond changed the title Support node 4 Support node 4 and 5 Feb 20, 2016
@blond blond changed the title Support node 4 and 5 Support node 4 Feb 20, 2016
});

expect(require.cache[modulePath]).to.be.not.equal('foo');
expect(clearRequire.calledWith('/path/to/project/.enb/make.js')).to.be.true;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

expect(clearRequire).to.be.calledWith() ?

@blond blond force-pushed the support-node-4 branch 2 times, most recently from 586d355 to 7d0dbf6 Compare February 20, 2016 17:48
@blond blond mentioned this pull request Feb 20, 2016
@blond blond force-pushed the support-node-4 branch 2 times, most recently from 3933b8b to c341519 Compare February 20, 2016 18:08
'make.js': ''
}
}
});
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А mockFs({ path.normalize('full/path/to/file.js'): '' }); не работает?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправил.

А тут зачем path.normalize?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хз. если под вендой окей, то мне ок)

@blond blond force-pushed the support-node-4 branch 2 times, most recently from d1dc0af to da2c498 Compare February 21, 2016 08:43
projectConfig.setLanguages(['ru']);
});
mockRequire(path.normalize('/path/to/project/.bem/make.js'), function (projectConfig) {
projectConfig.setLanguages(['ru']);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be 'en'

blond added a commit that referenced this pull request Feb 24, 2016
@blond blond merged commit 8f53f93 into master Feb 24, 2016
@blond blond deleted the support-node-4 branch February 24, 2016 10:22
@blond blond removed the in progress label Feb 24, 2016
@blond blond mentioned this pull request Mar 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants