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

added 4 matchers for user related #71

Merged
merged 2 commits into from May 1, 2013

Conversation

Projects
None yet
2 participants
@studio3104
Contributor

studio3104 commented Apr 30, 2013

Hi. I have added the following items.

  • Check of user's uid
  • Check of user's home directory
  • Check of user's login shell
  • Check of group's gid

Thank you confirmation.

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Apr 30, 2013

Owner

Thanks.LGTM.

And if you will add specs to debian, gentoo and solaris, I will appreciate you.
(Because matchers added by you will be available in those platforms.)

Owner

mizzy commented Apr 30, 2013

Thanks.LGTM.

And if you will add specs to debian, gentoo and solaris, I will appreciate you.
(Because matchers added by you will be available in those platforms.)

@studio3104

This comment has been minimized.

Show comment
Hide comment
@studio3104

studio3104 May 1, 2013

Contributor

As of your advice, I added specs to the Solaris Debian, and Gentoo.
Please check again!!

Contributor

studio3104 commented May 1, 2013

As of your advice, I added specs to the Solaris Debian, and Gentoo.
Please check again!!

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy May 1, 2013

Owner

':os => :redhat' are written in all OSes ...

Owner

mizzy commented May 1, 2013

':os => :redhat' are written in all OSes ...

@studio3104

This comment has been minimized.

Show comment
Hide comment
@studio3104

studio3104 May 1, 2013

Contributor

I am sorry...
Fixed, please check again.

Contributor

studio3104 commented May 1, 2013

I am sorry...
Fixed, please check again.

mizzy added a commit that referenced this pull request May 1, 2013

Merge pull request #71 of studio3104/matchers_for_user_related
Conflicts:
	spec/solaris/commands_spec.rb

@mizzy mizzy merged commit 9ce0e1c into mizzy:master May 1, 2013

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy May 1, 2013

Owner

Good.Thanks!

Owner

mizzy commented May 1, 2013

Good.Thanks!

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