From 1028e92f81f92b36f678a1b0acb3cd06788cd97a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mathieu=20Gagne=CC=81?= <mgagne@iweb.com>
Date: Wed, 5 Jun 2013 13:19:54 -0400
Subject: [PATCH] Add missing glance::params include in client

Change-Id: Ia5f99939f10a78033b743aca4a289b1e8d314e02
---
 manifests/client.pp                |  2 ++
 spec/classes/glance_client_spec.rb | 26 ++++++++++++++++++++++++--
 2 files changed, 26 insertions(+), 2 deletions(-)

diff --git a/manifests/client.pp b/manifests/client.pp
index 44e1ec17..094f803f 100644
--- a/manifests/client.pp
+++ b/manifests/client.pp
@@ -8,6 +8,8 @@ class glance::client (
   $ensure = 'present'
 ) {
 
+  include glance::params
+
   package { 'python-glance':
     name   => $::glance::params::client_package_name,
     ensure => $ensure,
diff --git a/spec/classes/glance_client_spec.rb b/spec/classes/glance_client_spec.rb
index ab61bce1..e44aeb37 100644
--- a/spec/classes/glance_client_spec.rb
+++ b/spec/classes/glance_client_spec.rb
@@ -1,5 +1,27 @@
 require 'spec_helper'
 
-describe 'glance::client' do 
-  it { should contain_package('python-glance').with_ensure('present') }
+describe 'glance::client' do
+
+  shared_examples 'glance client' do
+    it { should include_class('glance::params') }
+    it { should contain_package('python-glance').with(
+        :name   => 'python-glance',
+        :ensure => 'present'
+      )
+    }
+  end
+
+  context 'on Debian platforms' do
+    let :facts do
+      { :osfamily => 'Debian' }
+    end
+    include_examples 'glance client'
+  end
+
+  context 'on RedHat platforms' do
+    let :facts do
+      { :osfamily => 'RedHat' }
+    end
+    include_examples 'glance client'
+  end
 end