Merge pull request #1 from lukemelia/configure-hook

Use new `configure` hook instead of `willDeploy`
ember-cli-deploy · Jun 19, 2015 · 23088f3 · 23088f3
2 parents 86216b5 + 9c011f0
commit 23088f3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/index.js b/index.js
@@ -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] || {};

diff --git a/tests/unit/index-nodetest.js b/tests/unit/index-nodetest.js
@@ -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'
@@ -48,7 +48,7 @@ describe('manifest plugin', function() {
         }
       };
 
-      return assert.isFulfilled(plugin.willDeploy.call(plugin, context))
+      return assert.isFulfilled(plugin.configure.call(plugin, context))
     });
   });