Fixed handling of non-numeric timeout/retry_interval

Change to use Timeout module rather to simplify logic somewhat...
Refactoring of tests

Not sure if this is better or not
Made TCP the default provider to avoid puppet warnings
Updated Modulefile "name" field to use new format

Otherwise librarian-puppet chokes on hyphens