Support only_if and not_if inside a define

[sue445]

c.f. #262

Example

define :some_definition do
end

# this is executed
some_definition do
  only_if "exit 0"
end

# this is not executed
some_definition do
  only_if "exit 1"
end

Close #262
Close #161

[sue445]

@unasuke I removed WIP. Please review this 🙇

[ryotarai]

This implementation evaluates only_if and not_if in the phase of loading recipes but only_if and not_if of normal resources (!= definition) are evaluated in the running phase.
For instance, in this recipe, echo is not executed:

execute 'touch /tmp/a'

define :echo do
  execute 'echo B'
end

execute 'echo A' do
  only_if 'test -f /tmp/a'
end

echo 'echo' do
  only_if 'test -f /tmp/a'
end

 INFO : Starting Itamae...
 INFO : Recipe: /Users/ryotarai/src/github.com/itamae-kitchen/itamae/tmp/a.rb
 INFO :   execute[touch /tmp/a] executed will change from 'false' to 'true'
 INFO :   execute[echo A] executed will change from 'false' to 'true'

[sue445]

@ryotarai oops... I'll fix this.

[sue445]

@ryotarai @unasuke I fixed this problem. Please review again 🙇

log that executed #271 (comment)

$ be ./bin/itamae local /tmp/test.rb
 INFO : Starting Itamae...
 INFO : Recipe: /tmp/test.rb
 INFO :   execute[touch /tmp/a] executed will change from 'false' to 'true'
 INFO :   execute[echo A] executed will change from 'false' to 'true'
 INFO :     execute[echo B] executed will change from 'false' to 'true'

[unasuke]

👀

https://travis-ci.org/itamae-kitchen/itamae/jobs/450251581#L4897

[unasuke]

LGTM 👍

[ryotarai]

LGTM 👍