X-Git-Url: https://review.fuel-infra.org/gitweb?a=blobdiff_plain;f=spec%2Fdefines%2Fppa_spec.rb;h=b8e734d09ff294b918a5a8f25a45c8551671cca1;hb=9f27ea9da08cd693153b1d69cc2edcd9e336a4b3;hp=b5aadff31a5e211994c6a3eea770989b6d870357;hpb=32eec3baf6171d4f4aeb513f4e481f82a8a92338;p=puppet-modules%2Fpuppetlabs-apt.git diff --git a/spec/defines/ppa_spec.rb b/spec/defines/ppa_spec.rb index b5aadff..b8e734d 100644 --- a/spec/defines/ppa_spec.rb +++ b/spec/defines/ppa_spec.rb @@ -1,6 +1,15 @@ # frozen_string_literal: true require 'spec_helper' + +def ppa_exec_params(user, repo, distro = 'trusty', environment = []) + [ + environment: environment, + command: "/opt/puppetlabs/puppet/cache/add-apt-repository-#{user}-ubuntu-#{repo}-#{distro}.sh", + logoutput: 'on_failure', + ] +end + describe 'apt::ppa' do let :pre_condition do 'class { "apt": }' @@ -21,21 +30,83 @@ describe 'apt::ppa' do id: 'Ubuntu', }, }, + puppet_vardir: '/opt/puppetlabs/puppet/cache' } end - let(:title) { 'ppa:needs/such.substitution/wow+type' } + let(:title) { 'ppa:needs/substitution' } it { is_expected.not_to contain_package('python-software-properties') } it { - is_expected.to contain_exec('add-apt-repository-ppa:needs/such.substitution/wow+type').that_notifies('Class[Apt::Update]').with(environment: [], - command: '/usr/bin/add-apt-repository -y ppa:needs/such.substitution/wow+type || (rm /etc/apt/sources.list.d/needs-ubuntu-such_substitution-wow_type-trusty.list && false)', # rubocop:disable Layout/LineLength - unless: '/usr/bin/test -f /etc/apt/sources.list.d/needs-ubuntu-such_substitution-wow_type-trusty.list && /usr/bin/test -f /etc/apt/trusted.gpg.d/needs_ubuntu_such_substitution-wow_type.gpg', # rubocop:disable Layout/LineLength - user: 'root', - logoutput: 'on_failure') + is_expected.to contain_exec('add-apt-repository-ppa:needs/substitution') + .that_notifies('Class[Apt::Update]') + .with(*ppa_exec_params('needs', 'substitution')) } end + [ + 'ppa:foo/bar', + 'ppa:foo/bar1.0', + 'ppa:foo10/bar10', + 'ppa:foo-/bar_', + ].each do |value| + describe 'valid resource names' do + let :facts do + { + os: { + family: 'Debian', + name: 'Ubuntu', + release: { + major: '18', + full: '18.04', + }, + distro: { + codename: 'trusty', + id: 'Ubuntu', + }, + }, + } + end + + let(:title) { value } + + it { is_expected.not_to raise_error } + it { is_expected.to contain_exec("add-apt-repository-#{value}") } + end + end + + [ + 'ppa:foo!/bar', + 'ppa:foo/bar!', + 'ppa:foo1.0/bar', + 'ppa:foo/bar/foobar', + '|| ls -la ||', + '|| touch /tmp/foo.txt ||', + ].each do |value| + describe 'invalid resource names' do + let :facts do + { + os: { + family: 'Debian', + name: 'Ubuntu', + release: { + major: '18', + full: '18.04', + }, + distro: { + codename: 'trusty', + id: 'Ubuntu', + }, + }, + } + end + + let(:title) { value } + + it { is_expected.to raise_error(Puppet::PreformattedError, %r{Invalid PPA name: #{value}}) } + end + end + describe 'Ubuntu 15.10 sources.list filename' do let :facts do { @@ -51,17 +122,16 @@ describe 'apt::ppa' do id: 'Ubuntu', }, }, + puppet_vardir: '/opt/puppetlabs/puppet/cache', } end let(:title) { 'ppa:user/foo' } it { - is_expected.to contain_exec('add-apt-repository-ppa:user/foo').that_notifies('Class[Apt::Update]').with(environment: [], - command: '/usr/bin/add-apt-repository -y ppa:user/foo || (rm /etc/apt/sources.list.d/user-ubuntu-foo-wily.list && false)', # rubocop:disable Layout/LineLength - unless: '/usr/bin/test -f /etc/apt/sources.list.d/user-ubuntu-foo-wily.list && /usr/bin/test -f /etc/apt/trusted.gpg.d/user_ubuntu_foo.gpg', # rubocop:disable Layout/LineLength - user: 'root', - logoutput: 'on_failure') + is_expected.to contain_exec('add-apt-repository-ppa:user/foo') + .that_notifies('Class[Apt::Update]') + .with(*ppa_exec_params('user', 'foo', 'wily')) } end @@ -69,12 +139,14 @@ describe 'apt::ppa' do let :pre_condition do 'class { "apt": }' end + let :params do { package_name: 'software-properties-common', package_manage: true, } end + let :facts do { os: { @@ -89,22 +161,17 @@ describe 'apt::ppa' do id: 'Ubuntu', }, }, + puppet_vardir: '/opt/puppetlabs/puppet/cache', } end - let(:title) { 'ppa:needs/such.substitution/wow' } + let(:title) { 'ppa:needs/substitution' } it { is_expected.to contain_package('software-properties-common') } it { - is_expected.to contain_exec('add-apt-repository-ppa:needs/such.substitution/wow').that_notifies('Class[Apt::Update]').with('environment' => [], - 'command' => '/usr/bin/add-apt-repository -y ppa:needs/such.substitution/wow || (rm /etc/apt/sources.list.d/needs-ubuntu-such_substitution-wow-trusty.list && false)', # rubocop:disable Layout/LineLength - 'unless' => '/usr/bin/test -f /etc/apt/sources.list.d/needs-ubuntu-such_substitution-wow-trusty.list && /usr/bin/test -f /etc/apt/trusted.gpg.d/needs_ubuntu_such_substitution-wow.gpg', # rubocop:disable Layout/LineLength - 'user' => 'root', - 'logoutput' => 'on_failure') - } - - it { - is_expected.to contain_file('/etc/apt/sources.list.d/needs-ubuntu-such_substitution-wow-trusty.list').that_requires('Exec[add-apt-repository-ppa:needs/such.substitution/wow]').with('ensure' => 'file') # rubocop:disable Layout/LineLength + is_expected.to contain_exec('add-apt-repository-ppa:needs/substitution') + .that_notifies('Class[Apt::Update]') + .with(*ppa_exec_params('needs', 'substitution')) } end @@ -112,6 +179,7 @@ describe 'apt::ppa' do let :pre_condition do 'class { "apt": }' end + let :facts do { os: { @@ -126,27 +194,23 @@ describe 'apt::ppa' do id: 'Ubuntu', }, }, + puppet_vardir: '/opt/puppetlabs/puppet/cache', } end + let :params do { package_manage: false, } end - let(:title) { 'ppa:needs/such.substitution/wow' } + let(:title) { 'ppa:needs/substitution' } it { is_expected.not_to contain_package('python-software-properties') } it { - is_expected.to contain_exec('add-apt-repository-ppa:needs/such.substitution/wow').that_notifies('Class[Apt::Update]').with('environment' => [], - 'command' => '/usr/bin/add-apt-repository -y ppa:needs/such.substitution/wow || (rm /etc/apt/sources.list.d/needs-ubuntu-such_substitution-wow-trusty.list && false)', # rubocop:disable Layout/LineLength - 'unless' => '/usr/bin/test -f /etc/apt/sources.list.d/needs-ubuntu-such_substitution-wow-trusty.list && /usr/bin/test -f /etc/apt/trusted.gpg.d/needs_ubuntu_such_substitution-wow.gpg', # rubocop:disable Layout/LineLength - 'user' => 'root', - 'logoutput' => 'on_failure') - } - - it { - is_expected.to contain_file('/etc/apt/sources.list.d/needs-ubuntu-such_substitution-wow-trusty.list').that_requires('Exec[add-apt-repository-ppa:needs/such.substitution/wow]').with('ensure' => 'file') # rubocop:disable Layout/LineLength + is_expected.to contain_exec('add-apt-repository-ppa:needs/substitution') + .that_notifies('Class[Apt::Update]') + .with(*ppa_exec_params('needs', 'substitution')) } end @@ -156,6 +220,7 @@ describe 'apt::ppa' do apt::ppa { "ppa:user/foo2": } ' end + let :facts do { os: { @@ -170,25 +235,25 @@ describe 'apt::ppa' do id: 'Ubuntu', }, }, + puppet_vardir: '/opt/puppetlabs/puppet/cache', } end + let :params do { - options: '', package_manage: true, require: 'Apt::Ppa[ppa:user/foo2]', } end + let(:title) { 'ppa:user/foo' } it { is_expected.to compile.with_all_deps } it { is_expected.to contain_package('software-properties-common') } it { - is_expected.to contain_exec('add-apt-repository-ppa:user/foo').that_notifies('Class[Apt::Update]').with(environment: [], - command: '/usr/bin/add-apt-repository ppa:user/foo || (rm /etc/apt/sources.list.d/user-ubuntu-foo-trusty.list && false)', # rubocop:disable Layout/LineLength - unless: '/usr/bin/test -f /etc/apt/sources.list.d/user-ubuntu-foo-trusty.list && /usr/bin/test -f /etc/apt/trusted.gpg.d/user_ubuntu_foo.gpg', # rubocop:disable Layout/LineLength - user: 'root', - logoutput: 'on_failure') + is_expected.to contain_exec('add-apt-repository-ppa:user/foo') + .that_notifies('Class[Apt::Update]') + .with(*ppa_exec_params('user', 'foo')) } end @@ -198,6 +263,7 @@ describe 'apt::ppa' do proxy => { "host" => "localhost" }, }' end + let :facts do { os: { @@ -212,23 +278,23 @@ describe 'apt::ppa' do id: 'Ubuntu', }, }, + puppet_vardir: '/opt/puppetlabs/puppet/cache', } end + let :params do { - 'options' => '', 'package_manage' => true, } end + let(:title) { 'ppa:user/foo' } it { is_expected.to contain_package('software-properties-common') } it { - is_expected.to contain_exec('add-apt-repository-ppa:user/foo').that_notifies('Class[Apt::Update]').with(environment: ['http_proxy=http://localhost:8080'], - command: '/usr/bin/add-apt-repository ppa:user/foo || (rm /etc/apt/sources.list.d/user-ubuntu-foo-trusty.list && false)', # rubocop:disable Layout/LineLength - unless: '/usr/bin/test -f /etc/apt/sources.list.d/user-ubuntu-foo-trusty.list && /usr/bin/test -f /etc/apt/trusted.gpg.d/user_ubuntu_foo.gpg', # rubocop:disable Layout/LineLength - user: 'root', - logoutput: 'on_failure') + is_expected.to contain_exec('add-apt-repository-ppa:user/foo') + .that_notifies('Class[Apt::Update]') + .with(*ppa_exec_params('user', 'foo', 'trusty', ['http_proxy=http://localhost:8080'])) } end @@ -238,6 +304,7 @@ describe 'apt::ppa' do proxy => { "host" => "localhost", "port" => 8180 }, }' end + let :facts do { os: { @@ -252,23 +319,23 @@ describe 'apt::ppa' do id: 'Ubuntu', }, }, + puppet_vardir: '/opt/puppetlabs/puppet/cache', } end + let :params do { - options: '', package_manage: true, } end + let(:title) { 'ppa:user/foo' } it { is_expected.to contain_package('software-properties-common') } it { - is_expected.to contain_exec('add-apt-repository-ppa:user/foo').that_notifies('Class[Apt::Update]').with(environment: ['http_proxy=http://localhost:8180'], - command: '/usr/bin/add-apt-repository ppa:user/foo || (rm /etc/apt/sources.list.d/user-ubuntu-foo-trusty.list && false)', # rubocop:disable Layout/LineLength - unless: '/usr/bin/test -f /etc/apt/sources.list.d/user-ubuntu-foo-trusty.list && /usr/bin/test -f /etc/apt/trusted.gpg.d/user_ubuntu_foo.gpg', # rubocop:disable Layout/LineLength - user: 'root', - logoutput: 'on_failure') + is_expected.to contain_exec('add-apt-repository-ppa:user/foo') + .that_notifies('Class[Apt::Update]') + .with(*ppa_exec_params('user', 'foo', 'trusty', ['http_proxy=http://localhost:8180'])) } end @@ -278,6 +345,7 @@ describe 'apt::ppa' do proxy => { "host" => "localhost", "port" => 8180, "https" => true }, }' end + let :facts do { os: { @@ -292,23 +360,23 @@ describe 'apt::ppa' do id: 'Ubuntu', }, }, + puppet_vardir: '/opt/puppetlabs/puppet/cache', } end + let :params do { - options: '', package_manage: true, } end + let(:title) { 'ppa:user/foo' } it { is_expected.to contain_package('software-properties-common') } it { - is_expected.to contain_exec('add-apt-repository-ppa:user/foo').that_notifies('Class[Apt::Update]').with(environment: ['http_proxy=http://localhost:8180', 'https_proxy=https://localhost:8180'], - command: '/usr/bin/add-apt-repository ppa:user/foo || (rm /etc/apt/sources.list.d/user-ubuntu-foo-trusty.list && false)', # rubocop:disable Layout/LineLength - unless: '/usr/bin/test -f /etc/apt/sources.list.d/user-ubuntu-foo-trusty.list && /usr/bin/test -f /etc/apt/trusted.gpg.d/user_ubuntu_foo.gpg', # rubocop:disable Layout/LineLength - user: 'root', - logoutput: 'on_failure') + is_expected.to contain_exec('add-apt-repository-ppa:user/foo') + .that_notifies('Class[Apt::Update]') + .with(*ppa_exec_params('user', 'foo', 'trusty', ['http_proxy=http://localhost:8180', 'https_proxy=https://localhost:8180'])) } end @@ -316,6 +384,7 @@ describe 'apt::ppa' do let :pre_condition do 'class { "apt": }' end + let :facts do { os: { @@ -330,9 +399,12 @@ describe 'apt::ppa' do id: 'Ubuntu', }, }, + puppet_vardir: '/opt/puppetlabs/puppet/cache', } end + let(:title) { 'ppa:user/foo' } + let :params do { ensure: 'absent', @@ -340,7 +412,12 @@ describe 'apt::ppa' do end it { - is_expected.to contain_file('/etc/apt/sources.list.d/user-ubuntu-foo-trusty.list').that_notifies('Class[Apt::Update]').with(ensure: 'absent') + is_expected.to contain_tidy("remove-apt-repository-script-#{title}") + .with('path' => '/opt/puppetlabs/puppet/cache/add-apt-repository-user-ubuntu-foo-trusty.sh') + + is_expected.to contain_tidy("remove-apt-repository-#{title}") + .with('path' => '/etc/apt/sources.list.d/user-ubuntu-foo-trusty.list') + .that_notifies('Class[Apt::Update]') } end @@ -362,6 +439,7 @@ describe 'apt::ppa' do }, } end + let(:title) { 'ppa:user/foo' } it do @@ -386,6 +464,7 @@ describe 'apt::ppa' do }, } end + let(:title) { 'ppa:user/foo' } it do