Code Review
/
puppet-modules
/
puppetlabs-apt.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Release prep v9.1.0
[puppet-modules/puppetlabs-apt.git]
/
spec
/
defines
/
pin_spec.rb
diff --git
a/spec/defines/pin_spec.rb
b/spec/defines/pin_spec.rb
index a80ee6fa12387c5b60dbbf930f5b78eea2a3560f..1dd02980afe12a76f511c858fad50a295d2c9e81 100644
(file)
--- a/
spec/defines/pin_spec.rb
+++ b/
spec/defines/pin_spec.rb
@@
-1,3
+1,5
@@
+# frozen_string_literal: true
+
require 'spec_helper'
describe 'apt::pin', type: :define do
let :pre_condition do
require 'spec_helper'
describe 'apt::pin', type: :define do
let :pre_condition do
@@
-5,10
+7,18
@@
describe 'apt::pin', type: :define do
end
let(:facts) do
{
end
let(:facts) do
{
- os: { family: 'Debian', name: 'Debian', release: { major: '8', full: '8.0' } },
- lsbdistid: 'Debian',
- osfamily: 'Debian',
- lsbdistcodename: 'jessie',
+ os: {
+ family: 'Debian',
+ name: 'Debian',
+ release: {
+ major: '9',
+ full: '9.0'
+ },
+ distro: {
+ codename: 'stretch',
+ id: 'Debian'
+ }
+ }
}
end
let(:title) { 'my_pin' }
}
end
let(:title) { 'my_pin' }
@@
-21,7
+31,7
@@
describe 'apt::pin', type: :define do
let :params do
{
'packages' => 'vim',
let :params do
{
'packages' => 'vim',
- 'version'
=> '1',
+ 'version'
=> '1'
}
end
}
end
@@
-32,7
+42,7
@@
describe 'apt::pin', type: :define do
let :params do
{
'packages' => 'vim',
let :params do
{
'packages' => 'vim',
- 'origin'
=> 'test',
+ 'origin'
=> 'test'
}
end
}
end
@@
-42,33
+52,34
@@
describe 'apt::pin', type: :define do
context 'without defaults' do
let :params do
{
context 'without defaults' do
let :params do
{
- 'explanation'
=> 'foo',
- 'order'
=> 99,
- 'release'
=> '1',
- 'codename'
=> 'bar',
+ 'explanation' => 'foo',
+ 'order' => 99,
+ 'release' => '1',
+ 'codename' => 'bar',
'release_version' => '2',
'release_version' => '2',
- 'component'
=> 'baz',
- 'originator'
=> 'foobar',
- 'label'
=> 'foobaz',
- 'priority'
=> 10,
+ 'component' => 'baz',
+ 'originator' => 'foobar',
+ 'label' => 'foobaz',
+ 'priority'
=> 10
}
end
it { is_expected.to contain_apt__setting('pref-my_pin').with_content(%r{Explanation: foo\nPackage: \*\nPin: release a=1, n=bar, v=2, c=baz, o=foobar, l=foobaz\nPin-Priority: 10\n}) }
}
end
it { is_expected.to contain_apt__setting('pref-my_pin').with_content(%r{Explanation: foo\nPackage: \*\nPin: release a=1, n=bar, v=2, c=baz, o=foobar, l=foobaz\nPin-Priority: 10\n}) }
+
it {
it {
-
is_expected
.to contain_apt__setting('pref-my_pin').with('priority' => 99)
+
expect(subject)
.to contain_apt__setting('pref-my_pin').with('priority' => 99)
}
end
context 'with ensure absent' do
let :params do
{
}
end
context 'with ensure absent' do
let :params do
{
- 'ensure' => 'absent'
,
+ 'ensure' => 'absent'
}
end
it {
}
end
it {
-
is_expected
.to contain_apt__setting('pref-my_pin').with('ensure' => 'absent')
+
expect(subject)
.to contain_apt__setting('pref-my_pin').with('ensure' => 'absent')
}
end
}
end
@@
-82,65
+93,65
@@
describe 'apt::pin', type: :define do
context 'with invalid order' do
let :params do
{
context 'with invalid order' do
let :params do
{
- 'order' => 'foo'
,
+ 'order' => 'foo'
}
end
it do
}
end
it do
-
is_expected
.to raise_error(Puppet::Error, %r{expects an Integer value, got String})
+
expect(subject)
.to raise_error(Puppet::Error, %r{expects an Integer value, got String})
end
end
context 'with packages == * and version' do
let :params do
{
end
end
context 'with packages == * and version' do
let :params do
{
- 'version' => '1'
,
+ 'version' => '1'
}
end
it do
}
end
it do
-
is_expected
.to raise_error(Puppet::Error, %r{parameter version cannot be used in general form})
+
expect(subject)
.to raise_error(Puppet::Error, %r{parameter version cannot be used in general form})
end
end
context 'with packages == * and release and origin' do
let :params do
{
end
end
context 'with packages == * and release and origin' do
let :params do
{
- 'origin'
=> 'test',
- 'release' => 'foo'
,
+ 'origin' => 'test',
+ 'release' => 'foo'
}
end
it do
}
end
it do
-
is_expected
.to raise_error(Puppet::Error, %r{parameters release and origin are mutually exclusive})
+
expect(subject)
.to raise_error(Puppet::Error, %r{parameters release and origin are mutually exclusive})
end
end
context 'with specific release and origin' do
let :params do
{
end
end
context 'with specific release and origin' do
let :params do
{
- 'release'
=> 'foo',
- 'origin'
=> 'test',
- 'packages' => 'vim'
,
+ 'release' => 'foo',
+ 'origin' => 'test',
+ 'packages' => 'vim'
}
end
it do
}
end
it do
-
is_expected
.to raise_error(Puppet::Error, %r{parameters release, origin, and version are mutually exclusive})
+
expect(subject)
.to raise_error(Puppet::Error, %r{parameters release, origin, and version are mutually exclusive})
end
end
context 'with specific version and origin' do
let :params do
{
end
end
context 'with specific version and origin' do
let :params do
{
- 'version'
=> '1',
- 'origin'
=> 'test',
- 'packages' => 'vim'
,
+ 'version' => '1',
+ 'origin' => 'test',
+ 'packages' => 'vim'
}
end
it do
}
end
it do
-
is_expected
.to raise_error(Puppet::Error, %r{parameters release, origin, and version are mutually exclusive})
+
expect(subject)
.to raise_error(Puppet::Error, %r{parameters release, origin, and version are mutually exclusive})
end
end
end
end
end
end