Update CODEOWNERS

[puppet-modules/puppetlabs-apt.git] / spec / classes / apt_update_spec.rb

diff --git a/spec/classes/apt_update_spec.rb b/spec/classes/apt_update_spec.rb
index 0db9a470b20394edea2b7e8cb9f78dd7e88877e0..c46771a52d8b226a22101dd005e3c98a96970c96 100644 (file)
--- a/spec/classes/apt_update_spec.rb
+++ b/spec/classes/apt_update_spec.rb
@@ -5,11 +5,11 @@ 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: {
@@ -17,14 +17,14 @@ describe 'apt::update', type: :class do
               name: 'Debian',
               release: {
                 major: '9',
-                full: '9.0',
+                full: '9.0'
               },
               distro: {
                 codename: 'stretch',
-                id: 'Debian',
-              },
+                id: 'Debian'
+              }
             },
-            apt_update_last_success: factval,
+            apt_update_last_success: factval
           }
         end
         let(:pre_condition) do
@@ -33,11 +33,11 @@ describe 'apt::update', type: :class 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: {
@@ -45,23 +45,24 @@ describe 'apt::update', type: :class do
             name: 'Debian',
             release: {
               major: '9',
-              full: '9.0',
+              full: '9.0'
             },
             distro: {
               codename: 'stretch',
-              id: 'Debian',
-            },
-          },
+              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 'and Exec[apt_update] refreshonly is overridden to true and has recent run' do
+
+    context 'when Exec[apt_update] refreshonly is overridden to true and has recent run' do
       let(:facts) do
         {
           os: {
@@ -69,14 +70,14 @@ describe 'apt::update', type: :class do
             name: 'Debian',
             release: {
               major: '9',
-              full: '9.0',
+              full: '9.0'
             },
             distro: {
               codename: 'stretch',
-              id: 'Debian',
-            },
+              id: 'Debian'
+            }
           },
-          apt_update_last_success: Time.now.to_i,
+          apt_update_last_success: Time.now.to_i
         }
       end
       let(:pre_condition) do
@@ -88,17 +89,18 @@ describe 'apt::update', type: :class do
 
       it 'skips 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' => true)
+        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: {
@@ -106,25 +108,25 @@ describe 'apt::update', type: :class do
               name: 'Debian',
               release: {
                 major: '9',
-                full: '9.0',
+                full: '9.0'
               },
               distro: {
                 codename: 'stretch',
-                id: 'Debian',
-              },
+                id: 'Debian'
+              }
             },
-            apt_update_last_success: factval,
+            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: {
@@ -132,27 +134,29 @@ describe 'apt::update', type: :class do
             name: 'Debian',
             release: {
               major: '9',
-              full: '9.0',
+              full: '9.0'
             },
             distro: {
               codename: 'stretch',
-              id: 'Debian',
-            },
-          },
+              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
+
   ['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: {
@@ -160,25 +164,25 @@ describe 'apt::update', type: :class do
                 name: 'Debian',
                 release: {
                   major: '9',
-                  full: '9.0',
+                  full: '9.0'
                 },
                 distro: {
                   codename: 'stretch',
-                  id: 'Debian',
-                },
+                  id: 'Debian'
+                }
               },
-              apt_update_last_success: factval,
+              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: {
@@ -186,24 +190,25 @@ describe 'apt::update', type: :class do
               name: 'Debian',
               release: {
                 major: '9',
-                full: '9.0',
+                full: '9.0'
               },
               distro: {
                 codename: 'stretch',
-                id: 'Debian',
-              },
+                id: 'Debian'
+              }
             },
-            apt_update_last_success: Time.now.to_i,
+            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: {
@@ -211,21 +216,21 @@ describe 'apt::update', type: :class do
               name: 'Debian',
               release: {
                 major: '9',
-                full: '9.0',
+                full: '9.0'
               },
               distro: {
                 codename: 'stretch',
-                id: 'Debian',
-              },
+                id: 'Debian'
+              }
             },
-            apt_update_last_success: nil,
+            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