Fix some style issues in Puppet manifests

These checks are performed on the Puppet Forge to give quality scores to
modules, and are failing for the choria repositories:
* The `ensure` parameter must be the first one in resource declaration;
* Non-interpolated strings should be single, not double-quoted.

A bunch of other issues are reported by puppet-lint, but as of today
they do not seem to be part of the module score calculation, so in order
to minimize changes, this commit do not address them.

While these changes are gratuitous, they will allow the scoring of the
modules to increase, giving users a more positive view of the care taken
to maintain them.

Author: smortex

moduleroot/manifests/init.pp.erb

@@ -24,9 +24,10 @@ class <%= choria_module_name %> (
   Hash $server_config = {},
   Boolean $client = $mcollective::client,
   Boolean $server = $mcollective::server,
-  Enum["present", "absent"] $ensure = "present"
+  Enum['present', 'absent'] $ensure = 'present'
 ) {
   mcollective::module_plugin{$name:
+    ensure                      => $ensure,
     config_name                 => $config_name,
     client_files                => $client_files,
     server_files                => $server_files,
@@ -48,6 +49,5 @@ class <%= choria_module_name %> (
     server_config               => $server_config,
     client                      => $client,
     server                      => $server,
-    ensure                      => $ensure
   }
 }