Don't deliberately blow up on chef-solo

There's no reason to break chef-solo when we don't have to. Also, more complex configs may not be able to rely on search either.
Add real tests and missing vagrant-wrapper(someone is testing this without bundler, or they'd catch that)
Typo in variable name
One more logic error
Found older logic bomb that also was checking the wrong array
Clarify use of attribute, make it simply a default
Move loop back to original
Fix missing attribute ref
Merge pull request #206 from martinb3/fix_nil_assignment

Varnish recipe breaks when using Chef Solo