Check empty source when adding claim

fix the rspec cases and add checking for the source when adding claim
Wow, you're putting a lot of effort here, and I appreciate that.

I think the current code is fine for #1 and #2.  I think it's fragile for #3, and I would prefer to remove it.

Here is my main question: *can you explain a case where we actually do want the fetch :trait?  Is there something that breaks without it but works with it?*

As I understand things, it would only cause an error if the claim data is *already* bad, in which case we'd be punishing the current editor for a previous editor's mistakes.

I also do not understand at all why the fetched card would be a search card.  Do you?  It seems like you're using this as a proxy to test whether we're just renaming.  That's not a good test!