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 "be_monitored_by" matcher for Service resource #187

Merged
merged 4 commits into from Jul 2, 2013

Conversation

Projects
None yet
2 participants
@kitak
Contributor

kitak commented Jul 2, 2013

Serviceリソースで使える"be_monitored_by"マッチャを追加しました。
プロセスがプロセス監視・管理ツールで監視されていることを確認できます(現在はmonitのみの対応)。
以下が記述例になります。

describe service('unicorn') do
  it { should be_monitored_by('monit') }
end

最初はbe_running.under('monit')を用意しようと考えたのですが、supervisorが自身の子プロセスとして監視対象のプロセスを起動するのに対して、monitは監視する側される側に直接の関係がないので、be_monitored_byマッチャを用意しました。

matcherとcommandsのテストを記述して通ることを確認しています。

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Jul 2, 2013

Owner

グレート。

Owner

mizzy commented Jul 2, 2013

グレート。

@mizzy mizzy merged commit 2d0feb0 into mizzy:master Jul 2, 2013

1 check passed

default The Travis CI build passed
Details

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

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