These two lines have a bug and don't currently work:

Your response is a hash but the keys must be accessed by strings, not symbols, otherwise options[:avs_result] and options[:cvv_result] will always be assigned nil.

```ruby
          options[:avs_result] = { :code => response['avsCode'] }
          options[:cvv_result] = response['cvvCode']
```

Also, I think I would be inclined to remove the if AUTH or SALE conditional around these and always populate them when you construct the options hash, otherwise we're bringing per-action specifics down into a generic method. It looks like ActiveMerchant::Billing::Response should cope with that.