We're seeing a failing test on Hudson after the Condor changes were applied. It's
not actually a bug in the code, but that the test isn't properly setting up the
requirements for a new instance, causing it to fail immediately. But in actuality, this
test only cares that the instance state hasn't changed, so I modified the test to
express this.
Extra credit:
I tried to write this as:
expect {
put :update, :instance => {:state => 'running', :name =>
'mockinstance6'}, :id => instance.id
}.to {
change(instance, :name)
!change(instance, :state)
}
but it looks as though this is not legal. If anyone happens to know a cleaner way of
rewriting the attached patch, I'd love to find out more.
-- Matt
Show replies by date