Use new openstackclient tag

The new openstackclient tag was added so that we can get all resources
about openstack CLI more easily. This adds this tag to designateclient
because the package provides some sub-commands.

Change-Id: I882545f76950d6521f4e40c3ae23689c2eeda7d3

manifests/client.pp

@@ -13,7 +13,7 @@
 #  Defaults to $::designate::params::client_package_name
 #
 class designate::client (
-  $package_ensure = 'present',
+  $package_ensure      = 'present',
   $client_package_name = $::designate::params::client_package_name,
 ) inherits designate::params {
 
@@ -22,7 +22,7 @@ class designate::client (
   package { 'python-designateclient':
     ensure => $package_ensure,
     name   => $client_package_name,
-    tag    => 'openstack',
+    tag    => ['openstack', 'openstackclient'],
   }
 
   include openstacklib::openstackclient

spec/classes/designate_client_spec.rb

@@ -4,9 +4,6 @@
 require 'spec_helper'
 
 describe 'designate::client' do
-  let :params do
-    { }
-  end
 
   shared_examples 'designate-client' do
 
@@ -18,7 +15,7 @@ describe 'designate::client' do
         is_expected.to contain_package('python-designateclient').with(
           :ensure => 'present',
           :name   => platform_params[:client_package_name],
-          :tag    => 'openstack'
+          :tag    => ['openstack', 'openstackclient']
         )
       end
 
@@ -26,15 +23,15 @@ describe 'designate::client' do
     end
 
     context 'with custom package name' do
-      before do
-        params.merge!({ :client_package_name => 'designate-client-custom-name' })
+      let :params do
+        { :client_package_name => 'designate-client-custom-name' }
       end
 
       it 'configures using custom name' do
         is_expected.to contain_package('python-designateclient').with(
           :ensure    => 'present',
           :name      => 'designate-client-custom-name',
-          :tag       => 'openstack',
+          :tag       => ['openstack', 'openstackclient'],
         )
       end
     end