(POOLER-73) Add spec tests for remove_vmpooler_migration_vm

Add spec tests for remove_vmpooler_migration_vm
This commit is contained in:
Glenn Sarti 2017-02-14 08:57:44 -08:00
parent a4c55d5bf4
commit fb6be8e079

View file

@ -1302,6 +1302,25 @@ EOT
end end
end end
describe '#remove_vmpooler_migration_vm' do
before do
expect(subject).not_to be_nil
end
it 'should remove the migration from redis' do
redis.sadd('vmpooler__migration', vm)
expect(redis.sismember('vmpooler__migration',vm)).to be(true)
subject.remove_vmpooler_migration_vm(pool, vm)
expect(redis.sismember('vmpooler__migration',vm)).to be(false)
end
it 'should log a message and swallow an error if one occurs' do
expect(redis).to receive(:srem).and_raise(RuntimeError,'MockError')
expect(logger).to receive(:log).with('s', "[x] [#{pool}] '#{vm}' removal from vmpooler__migration failed with an error: MockError")
subject.remove_vmpooler_migration_vm(pool, vm)
end
end
describe '#_check_pool' do describe '#_check_pool' do
# Default test fixtures will consist of; # Default test fixtures will consist of;
# - Empty Redis dataset # - Empty Redis dataset