]> review.fuel-infra Code Review - puppet-modules/puppetlabs-apt.git/commitdiff
(#11413) Add spec test for apt::force to verify changes to unless
authorDan Bode <dan@puppetlabs.com>
Fri, 13 Jan 2012 19:02:42 +0000 (11:02 -0800)
committerDan Bode <dan@puppetlabs.com>
Fri, 13 Jan 2012 20:04:09 +0000 (12:04 -0800)
In the Previous commits, the query being done by the
apt::force install command was not strict enough and
could lead to false positives.

These queries represented by the unless parameter have been
resolved in another commit. This commit accompanies that commit
and adds basic unit tests to correspond to the changes.

spec/defines/force_spec.rb [new file with mode: 0644]

diff --git a/spec/defines/force_spec.rb b/spec/defines/force_spec.rb
new file mode 100644 (file)
index 0000000..d040dc9
--- /dev/null
@@ -0,0 +1,23 @@
+require 'spec_helper'
+describe 'apt::force', :type => :define do
+
+  let :title do
+    'my_package'
+  end
+
+  [false, '1'].each do |version|
+    describe "with version: #{version}" do
+      let :params do
+        {:version => version, :release => 'testing'}
+      end
+      let :unless_query do
+        base_command = "/usr/bin/dpkg -s #{title} | grep -q "
+        base_command + (version ? "'Version: #{params[:version]}'" : "'Status: install'")
+      end
+      let :exec_title do
+        "/usr/bin/aptitude -y -t #{params[:release]} install #{title}"
+      end
+      it { should contain_exec(exec_title).with_unless(unless_query) }
+    end
+  end
+end