Addressing Rubocop Errors

[puppet-modules/puppetlabs-apt.git] / spec / unit / puppet / type / apt_key_spec.rb

diff --git a/spec/unit/puppet/type/apt_key_spec.rb b/spec/unit/puppet/type/apt_key_spec.rb
index a1f24b5ad2f35a4e26c910032c279dd9369779e4..a25d8271e31886ac67c990295ae4189598130ef0 100644 (file)
--- a/spec/unit/puppet/type/apt_key_spec.rb
+++ b/spec/unit/puppet/type/apt_key_spec.rb
@@ -2,7 +2,7 @@ require 'spec_helper'
 require 'puppet'
 
 describe Puppet::Type.type(:apt_key) do
-  context 'only namevar 32bit key id' do
+  context 'with only namevar 32bit key id' do
     let(:resource) do
       Puppet::Type.type(:apt_key).new(
         id: 'EF8D349F',
@@ -129,105 +129,83 @@ describe Puppet::Type.type(:apt_key) do
     end
   end
 
-  context 'validation' do
+  context 'with validation' do
     it 'raises an error if content and source are set' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: 'EF8D349F',
-          source: 'http://apt.puppetlabs.com/pubkey.gpg',
-          content: 'Completely invalid as a GPG key',
-        )
+        Puppet::Type.type(:apt_key).new(id: 'EF8D349F',
+                                        source: 'http://apt.puppetlabs.com/pubkey.gpg',
+                                        content: 'Completely invalid as a GPG key')
       }.to raise_error(%r{content and source are mutually exclusive})
     end
 
     it 'raises an error if a weird length key is used' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: 'FEF8D349F',
-          source: 'http://apt.puppetlabs.com/pubkey.gpg',
-          content: 'Completely invalid as a GPG key',
-        )
+        Puppet::Type.type(:apt_key).new(id: 'FEF8D349F',
+                                        source: 'http://apt.puppetlabs.com/pubkey.gpg',
+                                        content: 'Completely invalid as a GPG key')
       }.to raise_error(%r{Valid values match})
     end
 
     it 'raises an error when an invalid URI scheme is used in source' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: 'EF8D349F',
-          source: 'hkp://pgp.mit.edu',
-        )
+        Puppet::Type.type(:apt_key).new(id: 'EF8D349F',
+                                        source: 'hkp://pgp.mit.edu')
       }.to raise_error(%r{Valid values match})
     end
 
     it 'allows the http URI scheme in source' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: 'EF8D349F',
-          source: 'http://pgp.mit.edu',
-        )
+        Puppet::Type.type(:apt_key).new(id: 'EF8D349F',
+                                        source: 'http://pgp.mit.edu')
       }.not_to raise_error
     end
 
     it 'allows the http URI with username and password' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: '4BD6EC30',
-          source: 'http://testme:Password2@pgp.mit.edu',
-        )
+        Puppet::Type.type(:apt_key).new(id: '4BD6EC30',
+                                        source: 'http://testme:Password2@pgp.mit.edu')
       }.not_to raise_error
     end
 
     it 'allows the https URI scheme in source' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: 'EF8D349F',
-          source: 'https://pgp.mit.edu',
-        )
+        Puppet::Type.type(:apt_key).new(id: 'EF8D349F',
+                                        source: 'https://pgp.mit.edu')
       }.not_to raise_error
     end
 
     it 'allows the https URI with username and password' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: 'EF8D349F',
-          source: 'https://testme:Password2@pgp.mit.edu',
-        )
+        Puppet::Type.type(:apt_key).new(id: 'EF8D349F',
+                                        source: 'https://testme:Password2@pgp.mit.edu')
       }.not_to raise_error
     end
 
     it 'allows the ftp URI scheme in source' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: 'EF8D349F',
-          source: 'ftp://pgp.mit.edu',
-        )
+        Puppet::Type.type(:apt_key).new(id: 'EF8D349F',
+                                        source: 'ftp://pgp.mit.edu')
       }.not_to raise_error
     end
 
     it 'allows an absolute path in source' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: 'EF8D349F',
-          source: '/path/to/a/file',
-        )
+        Puppet::Type.type(:apt_key).new(id: 'EF8D349F',
+                                        source: '/path/to/a/file')
       }.not_to raise_error
     end
 
     it 'allows 5-digit ports' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: 'EF8D349F',
-          source: 'http://pgp.mit.edu:12345/key',
-        )
+        Puppet::Type.type(:apt_key).new(id: 'EF8D349F',
+                                        source: 'http://pgp.mit.edu:12345/key')
       }.not_to raise_error
     end
 
     it 'allows 5-digit ports when using key servers' do
       expect {
-        Puppet::Type.type(:apt_key).new(
-          id: 'EF8D349F',
-          server: 'http://pgp.mit.edu:12345',
-        )
+        Puppet::Type.type(:apt_key).new(id: 'EF8D349F',
+                                        server: 'http://pgp.mit.edu:12345')
       }.not_to raise_error
     end
   end