puppet-retrospec 0.12.1 → 0.12.2

data/spec/fixtures/modules/concat/spec/acceptance/deprecation_warnings_spec.rb

@@ -1,238 +0,0 @@
-require 'spec_helper_acceptance'
-
-describe 'deprecation warnings' do
-  basedir = default.tmpdir('concat')
-
-  shared_examples 'has_warning' do |pp, w|
-    it 'applies the manifest twice with a stderr regex' do
-      expect(apply_manifest(pp, :catch_failures => true).stderr).to match(/#{Regexp.escape(w)}/m)
-      expect(apply_manifest(pp, :catch_changes => true).stderr).to match(/#{Regexp.escape(w)}/m)
-    end
-  end
-
-  context 'concat gnu parameter' do
-    pp = <<-EOS
-      concat { '#{basedir}/file':
-        gnu => 'foo',
-      }
-      concat::fragment { 'foo':
-        target  => '#{basedir}/file',
-        content => 'bar',
-      }
-    EOS
-    w = 'The $gnu parameter to concat is deprecated and has no effect'
-
-    it_behaves_like 'has_warning', pp, w
-  end
-
-  context 'concat warn parameter =>' do
-    ['true', 'yes', 'on'].each do |warn|
-      context warn do
-        pp = <<-EOS
-          concat { '#{basedir}/file':
-            warn => '#{warn}',
-          }
-          concat::fragment { 'foo':
-            target  => '#{basedir}/file',
-            content => 'bar',
-          }
-        EOS
-        w = 'Using stringified boolean values (\'true\', \'yes\', \'on\', \'false\', \'no\', \'off\') to represent boolean true/false as the $warn parameter to concat is deprecated and will be treated as the warning message in a future release'
-
-        it_behaves_like 'has_warning', pp, w
-
-        describe file("#{basedir}/file") do
-          it { should be_file }
-          its(:content) {
-            should match '# This file is managed by Puppet. DO NOT EDIT.'
-            should match 'bar'
-          }
-        end
-      end
-    end
-
-    ['false', 'no', 'off'].each do |warn|
-      context warn do
-        pp = <<-EOS
-          concat { '#{basedir}/file':
-            warn => '#{warn}',
-          }
-          concat::fragment { 'foo':
-            target  => '#{basedir}/file',
-            content => 'bar',
-          }
-        EOS
-        w = 'Using stringified boolean values (\'true\', \'yes\', \'on\', \'false\', \'no\', \'off\') to represent boolean true/false as the $warn parameter to concat is deprecated and will be treated as the warning message in a future release'
-
-        it_behaves_like 'has_warning', pp, w
-
-        describe file("#{basedir}/file") do
-          it { should be_file }
-          its(:content) {
-            should_not match '# This file is managed by Puppet. DO NOT EDIT.'
-            should match 'bar'
-          }
-        end
-      end
-    end
-  end
-
-  context 'concat::fragment ensure parameter', :unless => fact('osfamily') == 'windows' do
-    context 'target file exists' do
-      before(:all) do
-        pp = <<-EOS
-          file { '#{basedir}':
-            ensure => directory,
-          }
-          file { '#{basedir}/file1':
-            content => "file1 contents\n",
-          }
-        EOS
-        apply_manifest(pp)
-      end
-
-      pp = <<-EOS
-        concat { '#{basedir}/file': }
-        concat::fragment { 'foo':
-          target => '#{basedir}/file',
-          ensure => '#{basedir}/file1',
-        }
-      EOS
-      w = 'Passing a value other than \'present\' or \'absent\' as the $ensure parameter to concat::fragment is deprecated.  If you want to use the content of a file as a fragment please use the $source parameter.'
-
-      it_behaves_like 'has_warning', pp, w
-
-      describe file("#{basedir}/file") do
-        it { should be_file }
-        its(:content) { should match 'file1 contents' }
-      end
-
-      describe 'the fragment can be changed from a symlink to a plain file', :unless => (fact("osfamily") == "windows") do
-        pp = <<-EOS
-          concat { '#{basedir}/file': }
-          concat::fragment { 'foo':
-            target  => '#{basedir}/file',
-            content => 'new content',
-          }
-        EOS
-
-        it 'applies the manifest twice with no stderr' do
-          apply_manifest(pp, :catch_failures => true)
-          apply_manifest(pp, :catch_changes => true)
-        end
-
-        describe file("#{basedir}/file") do
-          it { should be_file }
-          its(:content) {
-            should match 'new content'
-            should_not match 'file1 contents'
-          }
-        end
-      end
-    end # target file exists
-
-    context 'target does not exist', :unless => fact('osfamily') == 'windows' do
-      pp = <<-EOS
-        concat { '#{basedir}/file': }
-        concat::fragment { 'foo':
-          target => '#{basedir}/file',
-          ensure => '#{basedir}/file1',
-        }
-      EOS
-      w = 'Passing a value other than \'present\' or \'absent\' as the $ensure parameter to concat::fragment is deprecated.  If you want to use the content of a file as a fragment please use the $source parameter.'
-
-      it_behaves_like 'has_warning', pp, w
-
-      describe file("#{basedir}/file") do
-        it { should be_file }
-      end
-
-      describe 'the fragment can be changed from a symlink to a plain file', :unless => (fact('osfamily') == 'windows') do
-        pp = <<-EOS
-          concat { '#{basedir}/file': }
-          concat::fragment { 'foo':
-            target  => '#{basedir}/file',
-            content => 'new content',
-          }
-        EOS
-
-        it 'applies the manifest twice with no stderr' do
-          apply_manifest(pp, :catch_failures => true)
-          apply_manifest(pp, :catch_changes => true)
-        end
-
-        describe file("#{basedir}/file") do
-          it { should be_file }
-          its(:content) { should match 'new content' }
-        end
-      end
-    end # target file exists
-
-  end # concat::fragment ensure parameter
-
-  context 'concat::fragment mode parameter' do
-    pp = <<-EOS
-      concat { '#{basedir}/file': }
-      concat::fragment { 'foo':
-        target  => '#{basedir}/file',
-        content => 'bar',
-        mode    => 'bar',
-      }
-    EOS
-    w = 'The $mode parameter to concat::fragment is deprecated and has no effect'
-
-    it_behaves_like 'has_warning', pp, w
-  end
-
-  context 'concat::fragment owner parameter' do
-    pp = <<-EOS
-      concat { '#{basedir}/file': }
-      concat::fragment { 'foo':
-        target  => '#{basedir}/file',
-        content => 'bar',
-        owner   => 'bar',
-      }
-    EOS
-    w = 'The $owner parameter to concat::fragment is deprecated and has no effect'
-
-    it_behaves_like 'has_warning', pp, w
-  end
-
-  context 'concat::fragment group parameter' do
-    pp = <<-EOS
-      concat { '#{basedir}/file': }
-      concat::fragment { 'foo':
-        target  => '#{basedir}/file',
-        content => 'bar',
-        group   => 'bar',
-      }
-    EOS
-    w = 'The $group parameter to concat::fragment is deprecated and has no effect'
-
-    it_behaves_like 'has_warning', pp, w
-  end
-
-  context 'concat::fragment backup parameter' do
-    pp = <<-EOS
-      concat { '#{basedir}/file': }
-      concat::fragment { 'foo':
-        target  => '#{basedir}/file',
-        content => 'bar',
-        backup  => 'bar',
-      }
-    EOS
-    w = 'The $backup parameter to concat::fragment is deprecated and has no effect'
-
-    it_behaves_like 'has_warning', pp, w
-  end
-
-  context 'include concat::setup' do
-    pp = <<-EOS
-      include concat::setup
-    EOS
-    w = 'concat::setup is deprecated as a public API of the concat module and should no longer be directly included in the manifest.'
-
-    it_behaves_like 'has_warning', pp, w
-  end
-
-end

data/spec/fixtures/modules/concat/spec/acceptance/empty_spec.rb

@@ -1,23 +0,0 @@
-require 'spec_helper_acceptance'
-
-describe 'concat force empty parameter' do
-  basedir = default.tmpdir('concat')
-  context 'should run successfully' do
-    pp = <<-EOS
-      concat { '#{basedir}/file':
-        mode  => '0644',
-        force => true,
-      }
-    EOS
-
-    it 'applies the manifest twice with no stderr' do
-      apply_manifest(pp, :catch_failures => true)
-      apply_manifest(pp, :catch_changes => true)
-    end
-
-    describe file("#{basedir}/file") do
-      it { should be_file }
-      its(:content) { should_not match /1\n2/ }
-    end
-  end
-end

data/spec/fixtures/modules/concat/spec/acceptance/fragment_source_spec.rb

@@ -1,157 +0,0 @@
-require 'spec_helper_acceptance'
-
-case fact('osfamily')
-  when 'AIX'
-    username = 'root'
-    groupname = 'system'
-  when 'Darwin'
-    username = 'root'
-    groupname = 'wheel'
-  when 'windows'
-    username = 'Administrator'
-    groupname = 'Administrators'
-  else
-    username = 'root'
-    groupname = 'root'
-end
-
-describe 'concat::fragment source' do
-  basedir = default.tmpdir('concat')
-  context 'should read file fragments from local system' do
-    pp = <<-EOS
-      file { '#{basedir}/file1':
-        content => "file1 contents\n"
-      }
-      file { '#{basedir}/file2':
-        content => "file2 contents\n"
-      }
-      concat { '#{basedir}/foo': }
-
-      concat::fragment { '1':
-        target  => '#{basedir}/foo',
-        source  => '#{basedir}/file1',
-        require => File['#{basedir}/file1'],
-      }
-      concat::fragment { '2':
-        target  => '#{basedir}/foo',
-        content => 'string1 contents',
-      }
-      concat::fragment { '3':
-        target  => '#{basedir}/foo',
-        source  => '#{basedir}/file2',
-        require => File['#{basedir}/file2'],
-      }
-    EOS
-
-    it 'applies the manifest twice with no stderr' do
-      apply_manifest(pp, :catch_failures => true)
-      apply_manifest(pp, :catch_changes => true)
-    end
-
-    describe file("#{basedir}/foo") do
-      it { should be_file }
-      its(:content) {
-        should match 'file1 contents'
-        should match 'string1 contents'
-        should match 'file2 contents'
-      }
-    end
-  end # should read file fragments from local system
-
-  context 'should create files containing first match only.' do
-    pp = <<-EOS
-      file { '#{basedir}/file1':
-        content => "file1 contents\n"
-      }
-      file { '#{basedir}/file2':
-        content => "file2 contents\n"
-      }
-      concat { '#{basedir}/result_file1':
-        owner   => '#{username}',
-        group   => '#{groupname}',
-        mode    => '0644',
-      }
-      concat { '#{basedir}/result_file2':
-        owner   => '#{username}',
-        group   => '#{groupname}',
-        mode    => '0644',
-      }
-      concat { '#{basedir}/result_file3':
-        owner   => '#{username}',
-        group   => '#{groupname}',
-        mode    => '0644',
-      }
-
-      concat::fragment { '1':
-        target  => '#{basedir}/result_file1',
-        source  => [ '#{basedir}/file1', '#{basedir}/file2' ],
-        require => [ File['#{basedir}/file1'], File['#{basedir}/file2'] ],
-        order   => '01',
-      }
-      concat::fragment { '2':
-        target  => '#{basedir}/result_file2',
-        source  => [ '#{basedir}/file2', '#{basedir}/file1' ],
-        require => [ File['#{basedir}/file1'], File['#{basedir}/file2'] ],
-        order   => '01',
-      }
-      concat::fragment { '3':
-        target  => '#{basedir}/result_file3',
-        source  => [ '#{basedir}/file1', '#{basedir}/file2' ],
-        require => [ File['#{basedir}/file1'], File['#{basedir}/file2'] ],
-        order   => '01',
-      }
-    EOS
-
-    it 'applies the manifest twice with no stderr' do
-      apply_manifest(pp, :catch_failures => true)
-      apply_manifest(pp, :catch_changes => true)
-    end
-    describe file("#{basedir}/result_file1") do
-      it { should be_file }
-      its(:content) {
-        should match 'file1 contents'
-        should_not match 'file2 contents'
-      }
-    end
-    describe file("#{basedir}/result_file2") do
-      it { should be_file }
-      its(:content) {
-        should match 'file2 contents'
-        should_not match 'file1 contents'
-      }
-    end
-    describe file("#{basedir}/result_file3") do
-      it { should be_file }
-      its(:content) {
-        should match 'file1 contents'
-        should_not match 'file2 contents'
-      }
-    end
-  end
-
-  context 'should fail if no match on source.' do
-    pp = <<-EOS
-      concat { '#{basedir}/fail_no_source':
-        owner   => '#{username}',
-        group   => '#{groupname}',
-        mode    => '0644',
-      }
-
-      concat::fragment { '1':
-        target  => '#{basedir}/fail_no_source',
-        source => [ '#{basedir}/nofilehere', '#{basedir}/nothereeither' ],
-        order   => '01',
-      }
-    EOS
-
-    it 'applies the manifest with resource failures' do
-      apply_manifest(pp, :expect_failures => true)
-    end
-    describe file("#{basedir}/fail_no_source") do
-      #FIXME: Serverspec::Type::File doesn't support exists? for some reason. so... hack.
-      it { should_not be_file }
-      it { should_not be_directory }
-    end
-  end
-end
-

data/spec/fixtures/modules/concat/spec/acceptance/fragments_are_always_replaced_spec.rb

@@ -1,139 +0,0 @@
-require 'spec_helper_acceptance'
-
-describe 'concat::fragment replace' do
-  basedir = default.tmpdir('concat')
-
-  context 'should create fragment files' do
-    before(:all) do
-      pp = <<-EOS
-        file { '#{basedir}':
-          ensure => directory,
-        }
-      EOS
-      apply_manifest(pp)
-    end
-
-    pp1 = <<-EOS
-      concat { '#{basedir}/foo': }
-
-      concat::fragment { '1':
-        target  => '#{basedir}/foo',
-        content => 'caller has replace unset run 1',
-      }
-    EOS
-    pp2 = <<-EOS
-      concat { '#{basedir}/foo': }
-
-      concat::fragment { '1':
-        target  => '#{basedir}/foo',
-        content => 'caller has replace unset run 2',
-      }
-    EOS
-
-    it 'applies the manifest twice with no stderr' do
-      apply_manifest(pp1, :catch_failures => true)
-      apply_manifest(pp1, :catch_changes => true)
-      apply_manifest(pp2, :catch_failures => true)
-      apply_manifest(pp2, :catch_changes => true)
-    end
-
-    describe file("#{basedir}/foo") do
-      it { should be_file }
-      its(:content) {
-        should_not match 'caller has replace unset run 1'
-        should match 'caller has replace unset run 2'
-      }
-    end
-  end # should create fragment files
-
-  context 'should replace its own fragment files when caller has File { replace=>true } set' do
-    before(:all) do
-      pp = <<-EOS
-        file { '#{basedir}':
-          ensure => directory,
-        }
-      EOS
-      apply_manifest(pp)
-    end
-
-    pp1 = <<-EOS
-      File { replace=>true }
-      concat { '#{basedir}/foo': }
-
-      concat::fragment { '1':
-        target  => '#{basedir}/foo',
-        content => 'caller has replace true set run 1',
-      }
-    EOS
-    pp2 = <<-EOS
-      File { replace=>true }
-      concat { '#{basedir}/foo': }
-
-      concat::fragment { '1':
-        target  => '#{basedir}/foo',
-        content => 'caller has replace true set run 2',
-      }
-    EOS
-
-    it 'applies the manifest twice with no stderr' do
-      apply_manifest(pp1, :catch_failures => true)
-      apply_manifest(pp1, :catch_changes => true)
-      apply_manifest(pp2, :catch_failures => true)
-      apply_manifest(pp2, :catch_changes => true)
-    end
-
-    describe file("#{basedir}/foo") do
-      it { should be_file }
-      its(:content) {
-        should_not match 'caller has replace true set run 1'
-        should match 'caller has replace true set run 2'
-      }
-    end
-  end # should replace its own fragment files when caller has File(replace=>true) set
-
-  context 'should replace its own fragment files even when caller has File { replace=>false } set' do
-    before(:all) do
-      pp = <<-EOS
-        file { '#{basedir}':
-          ensure => directory,
-        }
-      EOS
-      apply_manifest(pp)
-    end
-
-    pp1 = <<-EOS
-      File { replace=>false }
-      concat { '#{basedir}/foo': }
-
-      concat::fragment { '1':
-        target  => '#{basedir}/foo',
-        content => 'caller has replace false set run 1',
-      }
-    EOS
-    pp2 = <<-EOS
-      File { replace=>false }
-      concat { '#{basedir}/foo': }
-
-      concat::fragment { '1':
-        target  => '#{basedir}/foo',
-        content => 'caller has replace false set run 2',
-      }
-    EOS
-
-    it 'applies the manifest twice with no stderr' do
-      apply_manifest(pp1, :catch_failures => true)
-      apply_manifest(pp1, :catch_changes => true)
-      apply_manifest(pp2, :catch_failures => true)
-      apply_manifest(pp2, :catch_changes => true)
-    end
-
-    describe file("#{basedir}/foo") do
-      it { should be_file }
-      its(:content) {
-        should_not match 'caller has replace false set run 1'
-        should match 'caller has replace false set run 2'
-      }
-    end
-  end # should replace its own fragment files even when caller has File(replace=>false) set
-
-end