diff options
| author | Daniel Pittman <daniel@puppetlabs.com> | 2011-04-29 17:24:18 -0700 |
|---|---|---|
| committer | Daniel Pittman <daniel@puppetlabs.com> | 2011-05-02 10:08:26 -0700 |
| commit | b23cc8abec1a1ec41b554b4e72f9a3c21feaf9da (patch) | |
| tree | a0c8310b9a36b51c2566dfa7ef86ab421658ddd7 /spec/lib | |
| parent | 6815044fbc94c0d502b1061309d5aaeb8f791660 (diff) | |
| download | puppet-b23cc8abec1a1ec41b554b4e72f9a3c21feaf9da.tar.gz puppet-b23cc8abec1a1ec41b554b4e72f9a3c21feaf9da.tar.xz puppet-b23cc8abec1a1ec41b554b4e72f9a3c21feaf9da.zip | |
(#7282) action without `when_invoked` should fail...
We used to let actions be declared without the `when_invoked` block, which was
usually a sign of either someone writing their method code direct in action
declaration, or someone forgetting to add their code at all.
This was just let silently by: the error only showed up when you finally tried
to invoke the action, and a NoMethod error was raised by the face.
...except for our own testing. We took advantage of this a whole pile of
times in there; fixing the original UI issue means fixing all those too.
Reviewed-By: Nick Lewis <nick@puppetlabs.com>
Diffstat (limited to 'spec/lib')
| -rwxr-xr-x | spec/lib/puppet/face/huzzah.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/spec/lib/puppet/face/huzzah.rb b/spec/lib/puppet/face/huzzah.rb index 2c2b7aa8d..6593d358a 100755 --- a/spec/lib/puppet/face/huzzah.rb +++ b/spec/lib/puppet/face/huzzah.rb @@ -3,5 +3,5 @@ Puppet::Face.define(:huzzah, '2.0.1') do copyright "Puppet Labs", 2011 license "Apache 2 license; see COPYING" summary "life is a thing for celebration" - action :bar do "is where beer comes from" end + script :bar do "is where beer comes from" end end |