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

Some tests for attributes of last merged PRs #151

Merged
merged 3 commits into from Dec 23, 2016

Conversation

Projects
None yet
3 participants
@artem-sidorenko
Member

artem-sidorenko commented Dec 22, 2016

No description provided.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Dec 22, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling d2f18d1 on artem-sidorenko:tests into c1fbf18 on dev-sec:master.

coveralls commented Dec 22, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling d2f18d1 on artem-sidorenko:tests into c1fbf18 on dev-sec:master.

@chris-rock

This comment has been minimized.

Show comment
Hide comment
@chris-rock

chris-rock Dec 23, 2016

Member

Great addition @artem-sidorenko Should we add tests for our automatic detection as well?

Member

chris-rock commented Dec 23, 2016

Great addition @artem-sidorenko Should we add tests for our automatic detection as well?

@chris-rock chris-rock merged commit 3c3503a into dev-sec:master Dec 23, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 100.0%
Details
@artem-sidorenko

This comment has been minimized.

Show comment
Hide comment
@artem-sidorenko

artem-sidorenko Dec 23, 2016

Member

@chris-rock automatic detection? Do you mean the ssh version, crypto parameters or something else? We have the tests for ssh version and for crypto parameters

Member

artem-sidorenko commented Dec 23, 2016

@chris-rock automatic detection? Do you mean the ssh version, crypto parameters or something else? We have the tests for ssh version and for crypto parameters

@artem-sidorenko artem-sidorenko deleted the artem-forks:tests branch Dec 23, 2016

artem-sidorenko added a commit to artem-forks/chef-ssh-hardening that referenced this pull request Dec 23, 2016

Fix the broken master
Due to the parallel merge of GH-151 and GH-150

artem-sidorenko added a commit to artem-forks/chef-ssh-hardening that referenced this pull request Dec 23, 2016

Fix the broken master
Due to the parallel merge of GH-151 and GH-150
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment