ARVM destroy doesn't convert errors to deserialization errors

[chrisandreae]

https://github.com/iknow/iknow_view_models/blob/master/lib/view_model/active_record.rb#L307

https://github.com/iknow/iknow_view_models/blob/master/lib/view_model/active_record/update_context.rb#L162

So database errors such as foreign key violations will bubble out unwrapped. I think we don't want this.

[thefloweringash]

Probably needs an adaptation of this

iknow_view_models/lib/view_model/active_record/update_operation.rb

Lines 209 to 211 in a094031

	rescue ::ActiveRecord::StatementInvalid, ::ActiveRecord::InvalidForeignKey, ::ActiveRecord::RecordNotSaved => ex
	raise ViewModel::DeserializationError::DatabaseConstraint.new(ex.message, blame_reference)
	end