We shouldn't do it like this. It's hiding any exception which will likely hide other bugs from us to.

also I don't understand it ends up passing `product[billing_address]` here, could give us more details on how to reproduce? ideally via a spec