Skip to content

Commit

Permalink
Use new configure hook instead of willDeploy
Browse files Browse the repository at this point in the history
  • Loading branch information
lukemelia committed Jun 19, 2015
1 parent 86216b5 commit 9c011f0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion index.js
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ module.exports = {
return {
name: options.name,

willDeploy: function(context) {
configure: function(context) {
var deployment = context.deployment;
var ui = deployment.ui;
var config = deployment.config[this.name] = deployment.config[this.name] || {};
Expand Down
6 changes: 3 additions & 3 deletions tests/unit/index-nodetest.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,11 @@ describe('manifest plugin', function() {
name: 'test-plugin'
});

assert.equal(typeof result.willDeploy, 'function');
assert.equal(typeof result.configure, 'function');
assert.equal(typeof result.willUpload, 'function');
});

describe('willDeploy hook', function() {
describe('configure hook', function() {
it('resolves if config is ok', function() {
var plugin = subject.createDeployPlugin({
name: 'manifest'
Expand All @@ -48,7 +48,7 @@ describe('manifest plugin', function() {
}
};

return assert.isFulfilled(plugin.willDeploy.call(plugin, context))
return assert.isFulfilled(plugin.configure.call(plugin, context))
});
});

Expand Down

0 comments on commit 9c011f0

Please sign in to comment.