+# frozen_string_literal: true
+
require 'spec_helper'
describe 'apt::update', type: :class do
context "when apt::update['frequency']='always'" do
{
- 'a recent run' => Time.now.to_i,
- 'we are due for a run' => 1_406_660_561,
- 'the update-success-stamp file does not exist' => -1,
+ 'a recent run' => Time.now.to_i,
+ 'we are due for a run' => 1_406_660_561,
+ 'the update-success-stamp file does not exist' => -1
}.each_pair do |desc, factval|
- context "when $::apt_update_last_success indicates #{desc}" do
+ context "when $apt_update_last_success indicates #{desc}" do
let(:facts) do
{
- os: { family: 'Debian', name: 'Debian', release: { major: '7', full: '7.0' } },
- lsbdistid: 'Debian',
- osfamily: 'Debian',
- apt_update_last_success: factval,
- lsbdistcodename: 'wheezy',
- puppetversion: Puppet.version,
+ os: {
+ family: 'Debian',
+ name: 'Debian',
+ release: {
+ major: '9',
+ full: '9.0'
+ },
+ distro: {
+ codename: 'stretch',
+ id: 'Debian'
+ }
+ },
+ apt_update_last_success: factval
}
end
let(:pre_condition) do
it 'triggers an apt-get update run' do
# set the apt_update exec's refreshonly attribute to false
- is_expected.to contain_exec('apt_update').with('refreshonly' => false)
+ expect(subject).to contain_exec('apt_update').with('refreshonly' => false)
end
end
end
- context 'when $::apt_update_last_success is nil' do
+ context 'when $apt_update_last_success is nil' do
let(:facts) do
{
- os: { family: 'Debian', name: 'Debian', release: { major: '7', full: '7.0' } },
- lsbdistid: 'Debian',
- osfamily: 'Debian',
- lsbdistcodename: 'wheezy',
- puppetversion: Puppet.version,
+ os: {
+ family: 'Debian',
+ name: 'Debian',
+ release: {
+ major: '9',
+ full: '9.0'
+ },
+ distro: {
+ codename: 'stretch',
+ id: 'Debian'
+ }
+ }
}
end
let(:pre_condition) { "class{ '::apt': update => {'frequency' => 'always' },}" }
it 'triggers an apt-get update run' do
# set the apt_update exec\'s refreshonly attribute to false
- is_expected.to contain_exec('apt_update').with('refreshonly' => false)
+ expect(subject).to contain_exec('apt_update').with('refreshonly' => false)
+ end
+ end
+
+ context 'when Exec[apt_update] refreshonly is overridden to true and has recent run' do
+ let(:facts) do
+ {
+ os: {
+ family: 'Debian',
+ name: 'Debian',
+ release: {
+ major: '9',
+ full: '9.0'
+ },
+ distro: {
+ codename: 'stretch',
+ id: 'Debian'
+ }
+ },
+ apt_update_last_success: Time.now.to_i
+ }
+ end
+ let(:pre_condition) do
+ "
+ class{'::apt': update => {'frequency' => 'always' },}
+ Exec <| title=='apt_update' |> { refreshonly => true }
+ "
+ end
+
+ it 'skips an apt-get update run' do
+ # set the apt_update exec's refreshonly attribute to false
+ expect(subject).to contain_exec('apt_update').with('refreshonly' => true)
end
end
end
+
context "when apt::update['frequency']='reluctantly'" do
{
- 'a recent run' => Time.now.to_i,
- 'we are due for a run' => 1_406_660_561,
- 'the update-success-stamp file does not exist' => -1,
+ 'a recent run' => Time.now.to_i,
+ 'we are due for a run' => 1_406_660_561,
+ 'the update-success-stamp file does not exist' => -1
}.each_pair do |desc, factval|
- context "when $::apt_update_last_success indicates #{desc}" do
+ context "when $apt_update_last_success indicates #{desc}" do
let(:facts) do
{
- os: { family: 'Debian', name: 'Debian', release: { major: '7', full: '7.0' } },
- lsbdistid: 'Debian',
- osfamily: 'Debian',
- apt_update_last_success: factval,
- lsbdistcodename: 'wheezy',
- puppetversion: Puppet.version,
+ os: {
+ family: 'Debian',
+ name: 'Debian',
+ release: {
+ major: '9',
+ full: '9.0'
+ },
+ distro: {
+ codename: 'stretch',
+ id: 'Debian'
+ }
+ },
+ apt_update_last_success: factval
}
end
let(:pre_condition) { "class{ '::apt': update => {'frequency' => 'reluctantly' },}" }
it 'does not trigger an apt-get update run' do
# don't change the apt_update exec's refreshonly attribute. (it should be true)
- is_expected.to contain_exec('apt_update').with('refreshonly' => true)
+ expect(subject).to contain_exec('apt_update').with('refreshonly' => true)
end
end
end
- context 'when $::apt_update_last_success is nil' do
+ context 'when $apt_update_last_success is nil' do
let(:facts) do
{
- os: { family: 'Debian', name: 'Debian', release: { major: '7', full: '7.0' } },
- lsbdistid: 'Debian',
- osfamily: 'Debian',
- lsbdistcodename: 'wheezy',
- puppetversion: Puppet.version,
+ os: {
+ family: 'Debian',
+ name: 'Debian',
+ release: {
+ major: '9',
+ full: '9.0'
+ },
+ distro: {
+ codename: 'stretch',
+ id: 'Debian'
+ }
+ }
}
end
let(:pre_condition) { "class{ '::apt': update => {'frequency' => 'reluctantly' },}" }
it 'does not trigger an apt-get update run' do
# don't change the apt_update exec's refreshonly attribute. (it should be true)
- is_expected.to contain_exec('apt_update').with('refreshonly' => true)
+ expect(subject).to contain_exec('apt_update').with('refreshonly' => true)
end
end
end
- %w[daily weekly].each do |update_frequency|
+
+ ['daily', 'weekly'].each do |update_frequency|
context "when apt::update['frequency'] has the value of #{update_frequency}" do
- { 'we are due for a run' => 1_406_660_561, 'the update-success-stamp file does not exist' => -1 }.each_pair do |desc, factval|
- context "when $::apt_update_last_success indicates #{desc}" do
+ pair = { 'we are due for a run' => 1_406_660_561, 'the update-success-stamp file does not exist' => -1 }
+ pair.each_pair do |desc, factval|
+ context "when $apt_update_last_success indicates #{desc}" do
let(:facts) do
{
- os: { family: 'Debian', name: 'Debian', release: { major: '7', full: '7.0' } },
- lsbdistid: 'Debian',
- osfamily: 'Debian',
- apt_update_last_success: factval,
- lsbdistcodename: 'wheezy',
- puppetversion: Puppet.version,
+ os: {
+ family: 'Debian',
+ name: 'Debian',
+ release: {
+ major: '9',
+ full: '9.0'
+ },
+ distro: {
+ codename: 'stretch',
+ id: 'Debian'
+ }
+ },
+ apt_update_last_success: factval
}
end
let(:pre_condition) { "class{ '::apt': update => {'frequency' => '#{update_frequency}',} }" }
it 'triggers an apt-get update run' do
# set the apt_update exec\'s refreshonly attribute to false
- is_expected.to contain_exec('apt_update').with('refreshonly' => false)
+ expect(subject).to contain_exec('apt_update').with('refreshonly' => false)
end
end
end
- context 'when the $::apt_update_last_success fact has a recent value' do
+ context 'when the $apt_update_last_success fact has a recent value' do
let(:facts) do
{
- os: { family: 'Debian', name: 'Debian', release: { major: '7', full: '7.0' } },
- lsbdistid: 'Debian',
- osfamily: 'Debian',
- lsbdistcodename: 'wheezy',
- apt_update_last_success: Time.now.to_i,
- puppetversion: Puppet.version,
+ os: {
+ family: 'Debian',
+ name: 'Debian',
+ release: {
+ major: '9',
+ full: '9.0'
+ },
+ distro: {
+ codename: 'stretch',
+ id: 'Debian'
+ }
+ },
+ apt_update_last_success: Time.now.to_i
}
end
let(:pre_condition) { "class{ '::apt': update => {'frequency' => '#{update_frequency}',} }" }
it 'does not trigger an apt-get update run' do
# don't change the apt_update exec\'s refreshonly attribute. (it should be true)
- is_expected.to contain_exec('apt_update').with('refreshonly' => true)
+ expect(subject).to contain_exec('apt_update').with('refreshonly' => true)
end
end
- context 'when $::apt_update_last_success is nil' do
+
+ context 'when $apt_update_last_success is nil' do
let(:facts) do
{
- os: { family: 'Debian', name: 'Debian', release: { major: '7', full: '7.0' } },
- lsbdistid: 'Debian',
- osfamily: 'Debian',
- lsbdistcodename: 'wheezy',
- apt_update_last_success: nil,
- puppetversion: Puppet.version,
+ os: {
+ family: 'Debian',
+ name: 'Debian',
+ release: {
+ major: '9',
+ full: '9.0'
+ },
+ distro: {
+ codename: 'stretch',
+ id: 'Debian'
+ }
+ },
+ apt_update_last_success: nil
}
end
let(:pre_condition) { "class{ '::apt': update => {'frequency' => '#{update_frequency}',} }" }
it 'triggers an apt-get update run' do
# set the apt_update exec\'s refreshonly attribute to false
- is_expected.to contain_exec('apt_update').with('refreshonly' => false)
+ expect(subject).to contain_exec('apt_update').with('refreshonly' => false)
end
end
end
Code Review / puppet-modules / puppetlabs-apt.git / blobdiff