verifies algorithm before evaluating keyfinder

[jb08]

issue 343 PR (coordinated with @zinosama)

[sourcelevel-bot]

Hello, @jb08! This is your first Pull Request that will be reviewed by SourceLevel, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.

[jb08]

CI failing unrelated to PR, no?

Finished in 1.65 seconds
154 examples, 0 failures
bundler: failed to load command: codeclimate-test-reporter
The command "bundle exec rspec && bundle exec codeclimate-test-reporter" exited with 1.

[jb08]

@ab320012 fyi here is the PR that was discussed as part of issue 343.

[jb08]

cc: @laurawyse @pjo336
getting this merged prior to Peter's ongoing work will save network calls.
(thanks to @zinosama for grooming this work)

[jb08]

@excpt @ab320012 any advice on how to push this forward?

[martinemde]

This looks good to me also. I made a change to the key loader and I'd also like to see this rolled into the next release with both our changes.

See #348

[jb08]

thanks @bdewater for the approval.