- update a few broken unit tests

Fixes QueueHandlerTest to works with changes to QueueHandler

- Add exception handling inside enqueue method

Issues #2223673 #2593901 #1969034 #2554163 #2602918 - drush pull / import commands and a bunch of refactoring changes to support reusability

Converting Queue and Delete Handlers into services, eliminating ::create methods, and updated tests.

All known logger calls have been converted to event dispatcher calls which dispatch
Salesforce events that perform the logging.

- Adds salesforce logger channel
- Adds DIC on logger channel to event subscriber
- Template for eliminating log messages from core functionality

PullBase’s logging not converted due to impending sweeping changes in
mads-30 branch to same file.

All references to salesforce_encrypt.client service removed
salesforce_encrypt.client service definition removed
Unused use statements removed from EncryptedRestClientInterface andRestCientInterface
Constructors removed from EncryptedRestClientInterface and RestClientInterface
Salesforce_encrypt\RestClient constructor cleaned up
Adds Json mock to salesforce_encrypt\RestClientTest
Fixes service name error in PullBase
Fixes property visibility settings in RestClient so child salesforce_encrypt\RestClient may
inherit them.

Normalizes RestClientInterface injection in DeleteHandler and QueueHandler
All unit tests passing

and SaleforceControllerTest so all units tests pass.

Makes linter pass on MappedObject
Adds missing SalesforcePushParamsEvent use statement to MappedObject

Completes intelligent Drupal entity ID upsert code in PullBase

If the SF API cannot be reached while trying to save the entity ID in SF, the queue processing is
halted and the current queue item and remaining queue items are kept to process on a future pass.
A check has been added on PullBase::updateEntity() to send the Entity ID to SF if none exists and
the ID is configured to sync.
Adds PullException to facilitate throwing expected Exception up to Cron to tell Cron to suspend
queue processing but keep current and remaining queue items.

Add check on results return in QueueHandler to avoid error of calling $results->records on non-
existant object.
Updates RestClient::apiCall to throw RestExceptions in all error cases
Updates PullBase to detect RestExceptions from RestClient.

Includes test code in PullBase to force exceptions to test new logic. SuspendQueueException may
not be the proper Queue Exception to throw - it seems to delete/‘release’ the problematic item when
it should be kept so it can be processed fully.

Adds new state value to force an update in PullBase::updateEntity

Adds first pass at making pushing upset IDs to SF optional/configurable. It only pushes if the
mapping has a defined upsert key.

- tweak to DeleteHandler timeframes to cover a gap in coverage, and to cover earlier start dates for first cron run
- update sf push to respect salesforce_pull flag
- convert Rest push queue processor events to "OpEvent"s, rather than ParamsEvents