modify specifying version flag to deal with new repo

Modulefile

@@ -1,5 +1,5 @@
 name        'garethr-docker'
-version     '0.7.1'
+version     '0.8.0'
 source      'git://github.com/garethr/garethr-docker.git'
 author      'Gareth Rushgrove'
 summary     'Module for installing docker from repository on get.docker.io'

README.md

@@ -32,6 +32,14 @@ socket if required.
       socket_bind => 'unix:///var/run/docker.sock',
     }
 
+Unless specified this installs the latest version of docker from the
+lxc-docker package. However if you want to specify a specific version
+you can do so:
+
+    class { 'docker':
+      version => '0.5.5',
+    }
+
 ### Images
 
 The next step is probably to install a docker image, for this we have a defined type which can be used like so:

manifests/init.pp

@@ -28,6 +28,7 @@
 #
 class docker(
   $version                 = $docker::params::version,
+  $ensure                  = $docker::params::ensure,
   $tcp_bind                = $docker::params::tcp_bind,
   $socket_bind             = $docker::params::socket_bind,
   $use_upstream_apt_source = $docker::params::use_upstream_apt_source,

manifests/install.pp

@@ -45,7 +45,14 @@
     }
   }
 
+  if $docker::version {
+    $dockerpackage = "lxc-docker-${docker::version}"
+  } else {
+    $dockerpackage = "lxc-docker"
+  }
+
   package { 'lxc-docker':
-    ensure  => $docker::version,
+    name   => $dockerpackage,
+    ensure => $docker::ensure,
   }
 }

manifests/params.pp

@@ -1,5 +1,6 @@
 class docker::params {
-  $version                 = present
+  $version                 = undef
+  $ensure                  = present
   $tcp_bind                = undef
   $socket_bind             = 'unix:///var/run/docker.sock'
   $use_upstream_apt_source = true

spec/classes/docker_spec.rb

@@ -20,7 +20,12 @@
   end
 
   context 'with a custom version' do
-    let(:params) { {'version' => 'absent' } }
+    let(:params) { {'version' => '0.5.5' } }
+    it { should contain_package('lxc-docker').with_name('lxc-docker-0.5.5').with_ensure('present') }
+  end
+
+  context 'with ensure absent' do
+    let(:params) { {'ensure' => 'absent' } }
     it { should contain_package('lxc-docker').with_ensure('absent') }
   end