New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add .under('upstart') to be_running #186

Merged
merged 2 commits into from Jul 2, 2013

Conversation

Projects
None yet
2 participants
@kitak
Contributor

kitak commented Jul 2, 2013

be_running matcher のunder chain methodに'upstart'を引数で指定できるようにしました。
upstartで起動されたプロセスかどうか確認することができます。

describe service('monit') do
  it { should be_running.under('upstart') }
end

今はunder chain methodには'supervisor'のみ指定できますが、upstartとsupervisorを同列に扱ってよいかどうかが少し気になっています(upstartはinitデーモンの代わりとなるものなので、よりシステムに近い印象があります)。

upstartが導入されているディストロのspecファイルにテストを記述して通ることを確認しています。

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Jul 2, 2013

Owner

お、すばらし。

> upstartとsupervisorを同列に扱ってよいかどうかが少し気になっています

これは別に気にしなくていいと思うよ。

というわけでLGTMなのでマージ。

Owner

mizzy commented Jul 2, 2013

お、すばらし。

> upstartとsupervisorを同列に扱ってよいかどうかが少し気になっています

これは別に気にしなくていいと思うよ。

というわけでLGTMなのでマージ。

mizzy added a commit that referenced this pull request Jul 2, 2013

Merge pull request #186 from kitak/under_upstart
add .under('upstart') to be_running

@mizzy mizzy merged commit 6091c90 into mizzy:master Jul 2, 2013

1 check passed

default The Travis CI build passed
Details

@kitak kitak deleted the kitak:under_upstart branch Jul 2, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment