Are the changes in test-ebirdgeo.R intentional?

Yes. They make sure that getlatlng at least doesn't throw an error when it runs. I wasn't sure how to test the regex fix.

I don't see how these added lines accomplish that?

You're right. It doesn't provide the coverage I wanted.

I think there should be a vcr response for if http://ipinfo.io returns nothing, as that isn't covered in the current tests, I think.

Hi @RichardLitt, where are you at with this? I've tried running these tests locally and they seem to be blocking IP addresses as expected.

I removed the unnecessary lines, and this should be good to go if it works for you. Thank you!