
This patch aim to update our specs test in order to work with the rspec-puppet release 2.0.0, in the mean time, we update rspec syntax in order to be prepared for rspec 3.x move. In details: * Use shared_examples "a Puppet::Error" for puppet::error tests * Convert 'should' keyword to 'is_expected.to' (prepare rspec 3.x) * Fix spec tests for rspec-puppet 2.0.0 * Upgrade and pin rspec-puppet from 1.0.1 to 2.0.0 * Clean Gemfile (remove over-specificication of runtime deps of puppetlabs_spec_helper) * Remove un-used puppet-lint (name_containing_dash config) Change-Id: I5488507176d4665895eef65ddb4b6f0fb4eda3e7 Card: https://trello.com/c/eHXc1Ryd/4-investigate-the-necessary-change-to-be-rspec-puppet-2-0-0-compliant
161 lines
8.7 KiB
Ruby
161 lines
8.7 KiB
Ruby
require 'puppet'
|
|
require 'mocha'
|
|
require File.join(File.dirname(__FILE__), '..', '..', '..', '..', 'lib', 'puppet', 'provider', 'swift_ring_builder')
|
|
RSpec.configure do |config|
|
|
config.mock_with :mocha
|
|
end
|
|
provider_class = Puppet::Provider::SwiftRingBuilder
|
|
describe provider_class do
|
|
|
|
let :builder_file_path do
|
|
'/etc/swift/account.builder'
|
|
end
|
|
|
|
it 'should be able to lookup the local ring and build an object 1.8+' do
|
|
File.expects(:exists?).with(builder_file_path).returns(true)
|
|
provider_class.expects(:builder_file_path).twice.returns(builder_file_path)
|
|
# Swift 1.8 output
|
|
provider_class.expects(:swift_ring_builder).returns(
|
|
'/etc/swift/account.builder, build version 3
|
|
262144 partitions, 3 replicas, 3 zones, 3 devices, 0.00 balance
|
|
The minimum number of hours before a partition can be reassigned is 1
|
|
Devices: id region zone ip address port replication ip replication port name weight partitions balance meta
|
|
1 1 1 192.168.101.13 6002 192.168.101.13 6002 1 1.00 262144 0.00
|
|
2 1 2 192.168.101.14 6002 192.168.101.14 6002 1 1.00 262144 200.00 m2
|
|
0 1 3 192.168.101.15 6002 192.168.101.15 6002 1 1.00 262144-100.00 m2
|
|
3 1 1 192.168.101.16 6002 192.168.101.16 6002 1 1.00 262144-100.00
|
|
'
|
|
)
|
|
resources = provider_class.lookup_ring
|
|
expect(resources['192.168.101.13:6002/1']).to_not be_nil
|
|
expect(resources['192.168.101.14:6002/1']).to_not be_nil
|
|
expect(resources['192.168.101.15:6002/1']).to_not be_nil
|
|
expect(resources['192.168.101.16:6002/1']).to_not be_nil
|
|
|
|
expect(resources['192.168.101.13:6002/1'][:id]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:region]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:zone]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:weight]).to eql '1.00'
|
|
expect(resources['192.168.101.13:6002/1'][:partitions]).to eql '262144'
|
|
expect(resources['192.168.101.13:6002/1'][:balance]).to eql '0.00'
|
|
expect(resources['192.168.101.13:6002/1'][:meta]).to eql ''
|
|
|
|
expect(resources['192.168.101.14:6002/1'][:id]).to eql '2'
|
|
expect(resources['192.168.101.14:6002/1'][:region]).to eql '1'
|
|
expect(resources['192.168.101.14:6002/1'][:zone]).to eql '2'
|
|
expect(resources['192.168.101.14:6002/1'][:weight]).to eql '1.00'
|
|
expect(resources['192.168.101.14:6002/1'][:partitions]).to eql '262144'
|
|
expect(resources['192.168.101.14:6002/1'][:balance]).to eql '200.00'
|
|
expect(resources['192.168.101.14:6002/1'][:meta]).to eql 'm2'
|
|
end
|
|
|
|
it 'should be able to lookup the local ring and build an object 1.8.0' do
|
|
File.expects(:exists?).with(builder_file_path).returns(true)
|
|
provider_class.expects(:builder_file_path).twice.returns(builder_file_path)
|
|
# Swift 1.8 output
|
|
provider_class.expects(:swift_ring_builder).returns(
|
|
'/etc/swift/account.builder, build version 3
|
|
262144 partitions, 3 replicas, 3 zones, 3 devices, 0.00 balance
|
|
The minimum number of hours before a partition can be reassigned is 1
|
|
Devices: id region zone ip address port name weight partitions balance meta
|
|
1 1 1 192.168.101.13 6002 1 1.00 262144 0.00
|
|
2 1 2 192.168.101.14 6002 1 1.00 262144 200.00 m2
|
|
0 1 3 192.168.101.15 6002 1 1.00 262144-100.00 m2
|
|
3 1 1 192.168.101.16 6002 1 1.00 262144-100.00
|
|
'
|
|
)
|
|
resources = provider_class.lookup_ring
|
|
expect(resources['192.168.101.13:6002/1']).to_not be_nil
|
|
expect(resources['192.168.101.14:6002/1']).to_not be_nil
|
|
expect(resources['192.168.101.15:6002/1']).to_not be_nil
|
|
expect(resources['192.168.101.16:6002/1']).to_not be_nil
|
|
|
|
expect(resources['192.168.101.13:6002/1'][:id]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:region]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:zone]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:weight]).to eql '1.00'
|
|
expect(resources['192.168.101.13:6002/1'][:partitions]).to eql '262144'
|
|
expect(resources['192.168.101.13:6002/1'][:balance]).to eql '0.00'
|
|
expect(resources['192.168.101.13:6002/1'][:meta]).to eql ''
|
|
|
|
expect(resources['192.168.101.14:6002/1'][:id]).to eql '2'
|
|
expect(resources['192.168.101.14:6002/1'][:region]).to eql '1'
|
|
expect(resources['192.168.101.14:6002/1'][:zone]).to eql '2'
|
|
expect(resources['192.168.101.14:6002/1'][:weight]).to eql '1.00'
|
|
expect(resources['192.168.101.14:6002/1'][:partitions]).to eql '262144'
|
|
expect(resources['192.168.101.14:6002/1'][:balance]).to eql '200.00'
|
|
expect(resources['192.168.101.14:6002/1'][:meta]).to eql 'm2'
|
|
end
|
|
|
|
it 'should be able to lookup the local ring and build an object 1.7' do
|
|
File.expects(:exists?).with(builder_file_path).returns(true)
|
|
provider_class.expects(:builder_file_path).twice.returns(builder_file_path)
|
|
# Swift 1.7 output
|
|
provider_class.expects(:swift_ring_builder).returns(
|
|
'/etc/swift/account.builder, build version 3
|
|
262144 partitions, 3 replicas, 3 zones, 3 devices, 0.00 balance
|
|
The minimum number of hours before a partition can be reassigned is 1
|
|
Devices: id region zone ip address port name weight partitions balance meta
|
|
1 1 1 192.168.101.13 6002 1 1.00 262144 0.00
|
|
2 1 2 192.168.101.14 6002 1 1.00 262144 0.00
|
|
0 1 3 192.168.101.15 6002 1 1.00 262144 0.00
|
|
'
|
|
)
|
|
resources = provider_class.lookup_ring
|
|
expect(resources['192.168.101.13:6002/1']).to_not be_nil
|
|
expect(resources['192.168.101.14:6002/1']).to_not be_nil
|
|
expect(resources['192.168.101.15:6002/1']).to_not be_nil
|
|
|
|
expect(resources['192.168.101.13:6002/1'][:id]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:region]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:zone]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:weight]).to eql '1.00'
|
|
expect(resources['192.168.101.13:6002/1'][:partitions]).to eql '262144'
|
|
expect(resources['192.168.101.13:6002/1'][:balance]).to eql '0.00'
|
|
expect(resources['192.168.101.13:6002/1'][:meta]).to eql ''
|
|
|
|
expect(resources['192.168.101.14:6002/1'][:id]).to eql '2'
|
|
expect(resources['192.168.101.14:6002/1'][:region]).to eql '1'
|
|
expect(resources['192.168.101.14:6002/1'][:zone]).to eql '2'
|
|
expect(resources['192.168.101.14:6002/1'][:weight]).to eql '1.00'
|
|
expect(resources['192.168.101.14:6002/1'][:partitions]).to eql '262144'
|
|
expect(resources['192.168.101.14:6002/1'][:balance]).to eql '0.00'
|
|
expect(resources['192.168.101.14:6002/1'][:meta]).to eql ''
|
|
end
|
|
|
|
it 'should be able to lookup the local ring and build an object legacy' do
|
|
File.expects(:exists?).with(builder_file_path).returns(true)
|
|
provider_class.expects(:builder_file_path).twice.returns(builder_file_path)
|
|
provider_class.expects(:swift_ring_builder).returns(
|
|
'/etc/swift/account.builder, build version 3
|
|
262144 partitions, 3 replicas, 3 zones, 3 devices, 0.00 balance
|
|
The minimum number of hours before a partition can be reassigned is 1
|
|
Devices: id zone ip address port name weight partitions balance meta
|
|
2 2 192.168.101.14 6002 1 1.00 262144 0.00
|
|
0 3 192.168.101.15 6002 1 1.00 262144 0.00
|
|
1 1 192.168.101.13 6002 1 1.00 262144 0.00
|
|
'
|
|
)
|
|
resources = provider_class.lookup_ring
|
|
expect(resources['192.168.101.15:6002/1']).to_not be_nil
|
|
expect(resources['192.168.101.13:6002/1']).to_not be_nil
|
|
expect(resources['192.168.101.14:6002/1']).to_not be_nil
|
|
|
|
expect(resources['192.168.101.13:6002/1'][:id]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:region]).to eql 'none'
|
|
expect(resources['192.168.101.13:6002/1'][:zone]).to eql '1'
|
|
expect(resources['192.168.101.13:6002/1'][:weight]).to eql '1.00'
|
|
expect(resources['192.168.101.13:6002/1'][:partitions]).to eql '262144'
|
|
expect(resources['192.168.101.13:6002/1'][:balance]).to eql '0.00'
|
|
expect(resources['192.168.101.13:6002/1'][:meta]).to eql ''
|
|
|
|
expect(resources['192.168.101.14:6002/1'][:id]).to eql '2'
|
|
expect(resources['192.168.101.14:6002/1'][:region]).to eql 'none'
|
|
expect(resources['192.168.101.14:6002/1'][:zone]).to eql '2'
|
|
expect(resources['192.168.101.14:6002/1'][:weight]).to eql '1.00'
|
|
expect(resources['192.168.101.14:6002/1'][:partitions]).to eql '262144'
|
|
expect(resources['192.168.101.14:6002/1'][:balance]).to eql '0.00'
|
|
expect(resources['192.168.101.14:6002/1'][:meta]).to eql ''
|
|
end
|
|
end
|