#on the first run, but if it's not run in awhile something is likely borked
#with apt and we'd want to know about it.
- if $::apt::_update['always'] == false {
- #if always_apt_update is true there's no point in parsing this logic.
-
- case $::apt::_update['frequency'] {
- 'always': {
- $_kick_apt = true
- }
- 'daily': {
- #compare current date with the apt_update_last_success fact to determine
- #if we should kick apt_update.
- $daily_threshold = (strftime('%s') - 86400)
- if $::apt_update_last_success {
- if $::apt_update_last_success < $daily_threshold {
- $_kick_apt = true
- } else {
- $_kick_apt = false
- }
- } else {
- #if apt-get update has not successfully run, we should kick apt_update
+ case $::apt::_update['frequency'] {
+ 'always': {
+ $_kick_apt = true
+ }
+ 'daily': {
+ #compare current date with the apt_update_last_success fact to determine
+ #if we should kick apt_update.
+ $daily_threshold = (strftime('%s') - 86400)
+ if $::apt_update_last_success {
+ if $::apt_update_last_success < $daily_threshold {
$_kick_apt = true
+ } else {
+ $_kick_apt = false
}
+ } else {
+ #if apt-get update has not successfully run, we should kick apt_update
+ $_kick_apt = true
}
- 'weekly':{
- #compare current date with the apt_update_last_success fact to determine
- #if we should kick apt_update.
- $weekly_threshold = (strftime('%s') - 604800)
- if $::apt_update_last_success {
- if ( $::apt_update_last_success < $weekly_threshold ) {
- $_kick_apt = true
- } else {
- $_kick_apt = false
- }
- } else {
- #if apt-get update has not successfully run, we should kick apt_update
+ }
+ 'weekly':{
+ #compare current date with the apt_update_last_success fact to determine
+ #if we should kick apt_update.
+ $weekly_threshold = (strftime('%s') - 604800)
+ if $::apt_update_last_success {
+ if ( $::apt_update_last_success < $weekly_threshold ) {
$_kick_apt = true
+ } else {
+ $_kick_apt = false
}
- }
- default: {
- #catches 'recluctantly', and any other value (which should not occur).
- #do nothing.
- $_kick_apt = false
+ } else {
+ #if apt-get update has not successfully run, we should kick apt_update
+ $_kick_apt = true
}
}
- } else {
- $_kick_apt = false
+ default: {
+ #catches 'recluctantly', and any other value (which should not occur).
+ #do nothing.
+ $_kick_apt = false
+ }
}
if $_kick_apt {
require 'spec_helper'
describe 'apt::update', :type => :class do
- context "when update['always']=true" do
- #This should completely disable all of this logic. These tests are to guarantee that we don't somehow magically change the behavior.
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy' } }
- let (:pre_condition) { "class{'::apt': update => {'always' => true},}" }
- it 'should trigger 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 })
- end
- ['always','daily','weekly','reluctantly'].each do |update_frequency|
- context "when apt::update['frequency'] has the value of #{update_frequency}" do
- { 'a recent run' => Time.now.to_i, 'we are due for a run' => 1406660561,'the update-success-stamp file does not exist' => -1 }.each_pair do |desc, factval|
- context "and $::apt_update_last_success indicates #{desc}" do
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :apt_update_last_success => factval, :lsbdistcodename => 'wheezy' } }
- let (:pre_condition) { "class{'::apt': update => {'always' => true, 'frequency' => '#{update_frequency}'}, }" }
- it 'should trigger 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})
- end
- end
- context 'when $::apt_update_last_success is nil' do
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy' } }
- let (:pre_condition) { "class{'::apt': update => {'always' => true, 'frequency' => '#{update_frequency}'}, }" }
- it 'should trigger 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})
- end
- end
- end
- end
- end
- end
-
- context "when apt::update['always']=false" do
- context "and apt::update['frequency']='always'" do
- { 'a recent run' => Time.now.to_i, 'we are due for a run' => 1406660561,'the update-success-stamp file does not exist' => -1 }.each_pair do |desc, factval|
- context "and $::apt_update_last_success indicates #{desc}" do
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :apt_update_last_success => factval, :lsbdistcodename => 'wheezy' } }
- let (:pre_condition) { "class{'::apt': update => {'always' => false, 'frequency' => 'always' },}" }
- it 'should trigger 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})
- end
- end
- end
- context 'when $::apt_update_last_success is nil' do
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy' } }
- let (:pre_condition) { "class{ '::apt': update => {'always' => false, 'frequency' => 'always' },}" }
+ context "and apt::update['frequency']='always'" do
+ { 'a recent run' => Time.now.to_i, 'we are due for a run' => 1406660561,'the update-success-stamp file does not exist' => -1 }.each_pair do |desc, factval|
+ context "and $::apt_update_last_success indicates #{desc}" do
+ let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :apt_update_last_success => factval, :lsbdistcodename => 'wheezy' } }
+ let (:pre_condition) { "class{'::apt': update => {'frequency' => 'always' },}" }
it 'should trigger an apt-get update run' do
- #set the apt_update exec\'s refreshonly attribute to false
+ #set the apt_update exec's refreshonly attribute to false
is_expected.to contain_exec('apt_update').with({'refreshonly' => false})
end
end
end
- context "and apt::update['frequency']='reluctantly'" do
- {'a recent run' => Time.now.to_i, 'we are due for a run' => 1406660561,'the update-success-stamp file does not exist' => -1 }.each_pair do |desc, factval|
- context "and $::apt_update_last_success indicates #{desc}" do
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :apt_update_last_success => factval, :lsbdistcodename => 'wheezy'} }
- let (:pre_condition) { "class{ '::apt': update => {'always' => false, 'frequency' => 'reluctantly' },}" }
- it 'should 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})
- end
- end
+ context 'when $::apt_update_last_success is nil' do
+ let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy' } }
+ let (:pre_condition) { "class{ '::apt': update => {'frequency' => 'always' },}" }
+ it 'should trigger 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})
end
- context 'when $::apt_update_last_success is nil' do
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy' } }
- let (:pre_condition) { "class{ '::apt': update => {'always' => false, 'frequency' => 'reluctantly' },}" }
+ end
+ end
+ context "and apt::update['frequency']='reluctantly'" do
+ {'a recent run' => Time.now.to_i, 'we are due for a run' => 1406660561,'the update-success-stamp file does not exist' => -1 }.each_pair do |desc, factval|
+ context "and $::apt_update_last_success indicates #{desc}" do
+ let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :apt_update_last_success => factval, :lsbdistcodename => 'wheezy'} }
+ let (:pre_condition) { "class{ '::apt': update => {'frequency' => 'reluctantly' },}" }
it 'should 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})
end
end
end
- ['daily','weekly'].each do |update_frequency|
- context "and apt::update['frequency'] has the value of #{update_frequency}" do
- { 'we are due for a run' => 1406660561,'the update-success-stamp file does not exist' => -1 }.each_pair do |desc, factval|
- context "and $::apt_update_last_success indicates #{desc}" do
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :apt_update_last_success => factval, :lsbdistcodename => 'wheezy' } }
- let (:pre_condition) { "class{ '::apt': update => {'always' => false, 'frequency' => '#{update_frequency}',} }" }
- it 'should trigger 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})
- end
- end
- end
- context 'when the $::apt_update_last_success fact has a recent value' do
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy', :apt_update_last_success => Time.now.to_i } }
- let (:pre_condition) { "class{ '::apt': update => {'always' => false, 'frequency' => '#{update_frequency}',} }" }
- it 'should 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})
- end
- end
- context 'when $::apt_update_last_success is nil' do
- let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy', :apt_update_last_success => nil } }
- let (:pre_condition) { "class{ '::apt': update => {'always' => false, 'frequency' => '#{update_frequency}',} }" }
+ context 'when $::apt_update_last_success is nil' do
+ let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy' } }
+ let (:pre_condition) { "class{ '::apt': update => {'frequency' => 'reluctantly' },}" }
+ it 'should 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})
+ end
+ end
+ end
+ ['daily','weekly'].each do |update_frequency|
+ context "and apt::update['frequency'] has the value of #{update_frequency}" do
+ { 'we are due for a run' => 1406660561,'the update-success-stamp file does not exist' => -1 }.each_pair do |desc, factval|
+ context "and $::apt_update_last_success indicates #{desc}" do
+ let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :apt_update_last_success => factval, :lsbdistcodename => 'wheezy' } }
+ let (:pre_condition) { "class{ '::apt': update => {'frequency' => '#{update_frequency}',} }" }
it 'should trigger 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})
end
end
end
+ context 'when the $::apt_update_last_success fact has a recent value' do
+ let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy', :apt_update_last_success => Time.now.to_i } }
+ let (:pre_condition) { "class{ '::apt': update => {'frequency' => '#{update_frequency}',} }" }
+ it 'should 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})
+ end
+ end
+ context 'when $::apt_update_last_success is nil' do
+ let(:facts) { { :lsbdistid => 'Debian', :osfamily => 'Debian', :lsbdistcodename => 'wheezy', :apt_update_last_success => nil } }
+ let (:pre_condition) { "class{ '::apt': update => {'frequency' => '#{update_frequency}',} }" }
+ it 'should trigger 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})
+ end
+ end
end
end
end