Catch edge case where externalIds field is None

[mskarlin]

S2 can evidently provide this field, but its value can be None. This prevents an exception in that case, and adds a test.