Get client when processing nameIdentifiers and skip pushes if DOI is in a raidRegistry

[codycooperross]

Purpose

This PR is a fix for #226

The previous PR only skipped NameIdentifier processing when using NameIdentifier.import_one, which is, as far as I now know, an unused method accessed primarily via rake task. This PR targets the Importable.parse_record flow, wherein a DOI gets created or updated in lupo and pushes a SQS message to levriero for processing.

For NameIdentifier.import and NameIdentifier.import_by_month, client data would not be available, and thus NameIdentifier would create event data and ORCID claims even if a DOI were in a raidRegistry. However, these methods are, again, to my knowledge, unused and accessed primarily via rake task.

closes: Add github issue that originated this PR

Approach

Open Questions and Pre-Merge TODOs

Learning

Types of changes

• Bug fix (non-breaking change which fixes an issue)

• New feature (non-breaking change which adds functionality)

• Breaking change (fix or feature that would cause existing functionality to change)

Reviewer, please remember our guidelines:

• Be humble in the language and feedback you give, ask don't tell.
• Consider using positive language as opposed to neutral when offering feedback. This is to avoid the negative bias that can occur with neutral language appearing negative.
• Offer suggestions on how to improve code e.g. simplification or expanding clarity.
• Ensure you give reasons for the changes you are proposing.