Merge pull request #422 from cmurphy/lint
authorDaniele Sluijters <daenney@users.noreply.github.com>
Fri, 13 Feb 2015 08:23:53 +0000 (09:23 +0100)
committerDaniele Sluijters <daenney@users.noreply.github.com>
Fri, 13 Feb 2015 08:23:53 +0000 (09:23 +0100)
Ensure linter fails on lint warnings

spec/defines/key_spec.rb

index b09dc5bf505976df567783f8f5c48f6a027e0150..7ac1d2813089a8092d861acf570a1c3c1237784f 100644 (file)
@@ -276,7 +276,7 @@ describe 'apt::key', :type => :define do
     describe 'duplication' do
       context 'two apt::key resources for same key, different titles' do
         let :pre_condition do
-          "apt::key { 'duplicate': key => #{title}, }"
+          "apt::key { 'duplicate': key => '#{title}', }"
         end
 
         it 'contains two apt::key resources' do
@@ -305,7 +305,7 @@ describe 'apt::key', :type => :define do
 
       context 'two apt::key resources, different ensure' do
         let :pre_condition do
-          "apt::key { 'duplicate': key => #{title}, ensure => 'absent', }"
+          "apt::key { 'duplicate': key => '#{title}', ensure => 'absent', }"
         end
         it 'informs the user of the impossibility' do
           expect { subject }.to raise_error(/already ensured as absent/)