diff --git a/spec/acceptance/redis_cli_task_spec.rb b/spec/acceptance/redis_cli_task_spec.rb index c563162e..3703f0cd 100644 --- a/spec/acceptance/redis_cli_task_spec.rb +++ b/spec/acceptance/redis_cli_task_spec.rb @@ -8,21 +8,17 @@ let(:task_name) { 'redis::redis_cli' } - it 'install redis-cli with the class' do - pp = <<-EOS - include redis - EOS - - apply_manifest(pp, catch_failures: true) - apply_manifest(pp, catch_changes: true) + include_examples 'an idempotent resource' do + let(:manifest) { 'include redis' } end describe 'ping' do let(:params) { 'command="ping"' } it 'execute ping' do - is_expected.to match(%r{{\s*"status":\s*"PONG"\s*}}) - is_expected.to match(%r{Ran on 1 target in .+ sec}) + is_expected. + to match(%r{{\s*"status":\s*"PONG"\s*}}). + and match(%r{Ran on 1 target in .+ sec}) end end @@ -31,8 +27,9 @@ let(:params) { 'command="ping; cat /etc/passwd"' } it 'stops script injections and escapes' do - is_expected.to match(%r!{\s*"status":\s*"ERR unknown command ('|`)ping; cat /etc/passwd('|`)!) - is_expected.to match(%r{Ran on 1 target in .+ sec}) + is_expected. + to match(%r!{\s*"status":\s*"ERR unknown command ('|`)ping; cat /etc/passwd('|`)!). + and match(%r{Ran on 1 target in .+ sec}) end end @@ -40,8 +37,9 @@ let(:params) { 'command="ping && cat /etc/passwd"' } it 'stops script injections and escapes' do - is_expected.to match(%r!{\s*"status":\s*"ERR unknown command ('|`)ping && cat /etc/passwd('|`)!) - is_expected.to match(%r{Ran on 1 target in .+ sec}) + is_expected. + to match(%r!{\s*"status":\s*"ERR unknown command ('|`)ping && cat /etc/passwd('|`)!). + and match(%r{Ran on 1 target in .+ sec}) end end end diff --git a/spec/acceptance/suites/default/redis_adminstration_spec.rb b/spec/acceptance/suites/default/redis_adminstration_spec.rb index 2e850fcd..ae0a356e 100644 --- a/spec/acceptance/suites/default/redis_adminstration_spec.rb +++ b/spec/acceptance/suites/default/redis_adminstration_spec.rb @@ -1,28 +1,30 @@ require 'spec_helper_acceptance' +RSpec::Matchers.define_negated_matcher :execute_without_warning, :execute_with_warning + # systcl settings are untestable in docker describe 'redis::administration', unless: default['hypervisor'] =~ %r{docker} do - it 'runs successfully' do - pp = <<-EOS - include redis - include redis::administration - EOS + def execute_with_warning + have_attributes(stderr: %r{WARNING}) + end - # Apply twice to ensure no errors the second time. - apply_manifest(pp, catch_failures: true) - apply_manifest(pp, catch_changes: true) + include_examples 'an idempotent resource' do + let(:manifest) { 'include redis, redis::administration' } end - describe file('/proc/sys/vm/overcommit_memory') do - its(:content) { is_expected.to eq("1\n") } + specify do + expect(file('/proc/sys/vm/overcommit_memory')). + to have_attributes(content: "1\n") end - describe file('/proc/sys/net/core/somaxconn') do - its(:content) { is_expected.to eq("65535\n") } + specify do + expect(file('/proc/sys/net/core/somaxconn')). + to have_attributes(content: "65535\n") end - describe command('timeout 1s redis-server --port 7777 --loglevel verbose') do - its(:stderr) { is_expected.not_to match(%r{WARNING}) } - its(:exit_status) { is_expected.to eq(124) } + specify do + expect(command('timeout 1s redis-server --port 7777 --loglevel verbose')). + to execute_without_warning. + and have_attributes(exit_status: 124) end end diff --git a/spec/acceptance/suites/default/redis_multi_instances_one_host_spec.rb b/spec/acceptance/suites/default/redis_multi_instances_one_host_spec.rb index 99896385..597f04e3 100644 --- a/spec/acceptance/suites/default/redis_multi_instances_one_host_spec.rb +++ b/spec/acceptance/suites/default/redis_multi_instances_one_host_spec.rb @@ -23,67 +23,59 @@ 'redis' end - it 'runs successfully' do - pp = <<-EOS - $listening_ports = #{instances} + include_examples 'an idempotent resource' do + let(:manifest) do + <<-PUPPET + $listening_ports = #{instances} - class { '::redis': - default_install => false, - service_enable => false, - service_ensure => 'stopped', - } - - $listening_ports.each |$port| { - $port_string = sprintf('%d',$port) - redis::instance { $port_string: - service_enable => true, - service_ensure => 'running', - port => $port, - bind => $facts['networking']['ip'], - dbfilename => "${port}-dump.rdb", - appendfilename => "${port}-appendonly.aof", - appendfsync => 'always', - require => Class['Redis'], + class { '::redis': + default_install => false, + service_enable => false, + service_ensure => 'stopped', } - } - - EOS - # Apply twice to ensure no errors the second time. - apply_manifest(pp, catch_failures: true) - apply_manifest(pp, catch_changes: true) + $listening_ports.each |$port| { + $port_string = sprintf('%d',$port) + redis::instance { $port_string: + service_enable => true, + service_ensure => 'running', + port => $port, + bind => $facts['networking']['ip'], + dbfilename => "${port}-dump.rdb", + appendfilename => "${port}-appendonly.aof", + appendfsync => 'always', + require => Class['Redis'], + } + } + PUPPET + end end - describe package(redis_name) do - it { is_expected.to be_installed } - end + specify { expect(package(redis_name)).to be_installed } - describe service(redis_name) do - it { is_expected.not_to be_enabled } - it { is_expected.not_to be_running } + specify do + expect(service(redis_name)).not_to be_enabled + expect(service(redis_name)).not_to be_running end instances.each do |instance| - describe file("/etc/systemd/system/redis-server-#{instance}.service") do - its(:content) { is_expected.to match %r{redis-server-#{instance}.conf} } - end - - describe service("redis-server-#{instance}") do - it { is_expected.to be_enabled } + specify do + expect(file("/etc/systemd/system/redis-server-#{instance}.service")). + to be_file. + and have_attributes(content: %r{redis-server-#{instance}.conf}) end - describe service("redis-server-#{instance}") do - it { is_expected.to be_running } - end + specify { expect(service("redis-server-#{instance}")).to be_enabled.and be_running } - describe file("#{config_path}/redis-server-#{instance}.conf") do - its(:content) { is_expected.to match %r{port #{instance}} } + specify do + expect(file("#{config_path}/redis-server-#{instance}.conf")). + to be_file. + and have_attributes(content: %r{port #{instance}}) end - context "redis instance #{instance} should respond to ping command" do - describe command("redis-cli -h #{fact('networking.ip')} -p #{instance} ping") do - its(:stdout) { is_expected.to match %r{PONG} } - end + specify "redis instance #{instance} should respond to ping command" do + expect(command("redis-cli -h #{fact('networking.ip')} -p #{instance} ping")). + to have_attributes(stdout: %r{PONG}) end end end diff --git a/spec/acceptance/suites/default/redis_sentinel_one_node_spec.rb b/spec/acceptance/suites/default/redis_sentinel_one_node_spec.rb index 277242f5..9e8831c1 100644 --- a/spec/acceptance/suites/default/redis_sentinel_one_node_spec.rb +++ b/spec/acceptance/suites/default/redis_sentinel_one_node_spec.rb @@ -1,54 +1,37 @@ require 'spec_helper_acceptance' describe 'redis::sentinel' do - redis_name = case fact('osfamily') - when 'Debian' - 'redis-server' - else - 'redis' - end - - it 'runs successfully' do - pp = <<-EOS - class { 'redis::sentinel': - master_name => 'mymaster', - redis_host => '127.0.0.1', - failover_timeout => 10000, - } - EOS - - apply_manifest(pp, catch_failures: true) - apply_manifest(pp, catch_changes: true) - end - - describe package(redis_name) do - it { is_expected.to be_installed } - end - - describe service(redis_name) do - it { is_expected.to be_running } + redis_name = fact('osfamily') == 'Debian' ? 'redis-server' : 'redis' + + include_examples 'an idempotent resource' do + let(:manifest) do + <<-PUPPET + class { 'redis::sentinel': + master_name => 'mymaster', + redis_host => '127.0.0.1', + failover_timeout => 10000, + } + PUPPET + end end - describe service('redis-sentinel') do - it { is_expected.to be_running } - end + specify { expect(package(redis_name)).to be_installed } + specify { expect(service(redis_name)).to be_running } - case fact('osfamily') - when 'Debian' - describe package('redis-sentinel') do - it { is_expected.to be_installed } - end + specify 'redis should respond to ping command' do + expect(command('redis-cli ping')). + to have_attributes(stdout: %r{PONG}) end - context 'redis should respond to ping command' do - describe command('redis-cli ping') do - its(:stdout) { is_expected.to match %r{PONG} } - end + specify { expect(service('redis-sentinel')).to be_running } + if redis_name == 'redis-server' + specify { expect(package('redis-sentinel')).to be_installed } + else + specify { expect(package('redis-sentinel')).not_to be_installed } end - context 'redis-sentinel should return correct sentinel master' do - describe command('redis-cli -p 26379 SENTINEL masters') do - its(:stdout) { is_expected.to match %r{^mymaster} } - end + specify 'redis-sentinel should return correct sentinel master' do + expect(command('redis-cli -p 26379 SENTINEL masters')). + to have_attributes(stdout: %r{^mymaster}) end end diff --git a/spec/acceptance/suites/default/redis_spec.rb b/spec/acceptance/suites/default/redis_spec.rb index 50269733..572b741a 100644 --- a/spec/acceptance/suites/default/redis_spec.rb +++ b/spec/acceptance/suites/default/redis_spec.rb @@ -1,21 +1,10 @@ require 'spec_helper_acceptance' describe 'redis' do - redis_name = case fact('osfamily') - when 'Debian' - 'redis-server' - else - 'redis' - end + redis_name = fact('osfamily') == 'Debian' ? 'redis-server' : 'redis' - it 'runs successfully' do - pp = <<-EOS - include redis - EOS - - # Apply twice to ensure no errors the second time. - apply_manifest(pp, catch_failures: true) - apply_manifest(pp, catch_changes: true) + include_examples 'an idempotent resource' do + let(:manifest) { 'include redis' } end it 'returns a fact' do @@ -29,17 +18,11 @@ end end - describe package(redis_name) do - it { is_expected.to be_installed } - end + specify { expect(package(redis_name)).to be_installed } + specify { expect(service(redis_name)).to be_running } - describe service(redis_name) do - it { is_expected.to be_running } - end - - context 'redis should respond to ping command' do - describe command('redis-cli ping') do - its(:stdout) { is_expected.to match %r{PONG} } - end + specify 'redis should respond to ping command' do + expect(command('redis-cli ping')). + to have_attributes(stdout: %r{PONG}) end end diff --git a/spec/acceptance/suites/default/redisget_spec.rb b/spec/acceptance/suites/default/redisget_spec.rb index 5d9391cb..c643ad81 100644 --- a/spec/acceptance/suites/default/redisget_spec.rb +++ b/spec/acceptance/suites/default/redisget_spec.rb @@ -1,28 +1,28 @@ require 'spec_helper_acceptance' describe 'redis::get() function' do - it 'runs successfully' do - pp = <<-EOS - include redis - - package { 'redis-rubygem' : - ensure => '3.3.3', - name => 'redis', - provider => 'puppet_gem', - } - EOS - - # Apply twice to ensure no errors the second time. - apply_manifest(pp, catch_failures: true) - apply_manifest(pp, catch_changes: true) + include_examples 'an idempotent resource' do + let(:manifest) do + <<-PUPPET + include redis + + package { 'redis-rubygem' : + ensure => '3.3.3', + name => 'redis', + provider => 'puppet_gem', + } + PUPPET + end end - describe command('redis-cli SET mykey "Hello"') do - its(:stdout) { is_expected.to match(%r{OK}) } + specify do + expect(command('redis-cli SET mykey "Hello"')). + to have_attributes(stdout: %r{OK}) end - describe command('redis-cli GET mykey') do - its(:stdout) { is_expected.to match('Hello') } + specify do + expect(command('redis-cli GET mykey')). + to have_attributes(stdout: %r{Hello}) end context 'with mykey set to Hello' do diff --git a/spec/acceptance/suites/scl/redis5_spec.rb b/spec/acceptance/suites/scl/redis5_spec.rb index 4a19041e..bc0c3cd2 100644 --- a/spec/acceptance/suites/scl/redis5_spec.rb +++ b/spec/acceptance/suites/scl/redis5_spec.rb @@ -9,33 +9,24 @@ on hosts, puppet_resource('service', 'rh-redis5-redis', 'ensure=stopped', 'enable=false') end - it 'runs successfully' do - pp = <<-PUPPET + include_examples 'an idempotent resource' do + let(:manifest) do + <<-PUPPET class { 'redis::globals': scl => 'rh-redis5', } class { 'redis': manage_repo => true, } - PUPPET - - # Apply twice to ensure no errors the second time. - apply_manifest(pp, catch_failures: true) - apply_manifest(pp, catch_changes: true) - end - - describe package('rh-redis5-redis') do - it { is_expected.to be_installed } + PUPPET + end end - describe service('rh-redis5-redis') do - it { is_expected.to be_running } - it { is_expected.to be_enabled } - end + specify { expect(package('rh-redis5-redis')).to be_installed } + specify { expect(service('rh-redis5-redis')).to be_running.and be_enabled } - context 'redis should respond to ping command' do - describe command('scl enable rh-redis5 -- redis-cli ping') do - its(:stdout) { is_expected.to match %r{PONG} } - end + it 'redis should respond to ping command' do + expect(command('scl enable rh-redis5 -- redis-cli ping')). + to have_attributes(stdout: %r{PONG}) end end