Making sure ProfileLookupRunner returns response

While the callbacks are helpful, it was returning unexpected values.
Extracting inner class to its own file

Instead of having the inline untested class, I am instead moving it
to an separate file with its own test.
Adding requires_net_connect tag

One of the specs requires net connect, so I'm marking it as such
Adding requires_net_connect tag

One of the specs requires net connect, so I'm marking it as such
Updating Orcid::ProfileConnectionService

To reflect the callback nature, and not wanting to have duplicate
knowledge (Note: removing the duplicate
`self.orcid_profile_candidates=[]`)

I prefer the callback route as it works with the BatchProfile spec
Merge branch 'adding-feature-test-for-profile-connection'

* adding-feature-test-for-profile-connection:
  Updating Orcid::ProfileConnectionService
  Adding requires_net_connect tag
  Extracting inner class to its own file
  Making sure ProfileLookupRunner returns response