Removes needs for TestQueueHandler test class by
- adding state, request, and logger classes to QueueHandler constructor (injected)
- converting watchdog_exception calls to injected logger calls (also removes need to
  use LoggerTrait)
- stubs and overrides the parseUrl methods in a mock QueueHanlder class in the
  test code
Fixes a null object error with a trinary in PullBase
Does not fix all PullBase tests

# Conflicts:
#	modules/salesforce_pull/src/DeleteHandler.php
#	modules/salesforce_pull/src/Plugin/QueueWorker/PullBase.php

PullBaseTest needs work.
TestPullBase removed - test version of class no longer needed.
Removes all calls to watchdog_exception in PullBase, injected logger used
instead.
Adds LoggerFactory dependency into PullBase constructor and create()
method.
Removes wrapper functions now using injected storage classes (already testable!).
Adds full logger mocking to PullBaseTest.
Adds .tags to .gitignore

Adds missing use statements to MappedObject
Adds missing use statements to PullBase
Removes unnecessary EntityManager dependency to PullBase; EntityTypeManager
already exists

Fixes injected service call in PushQueue

Overrides SalesforceMapping’s inherited postLoad method to attach

Adds missing use statements
Updates .gitignore to exclude .tags file
Removes a lingering procedural function call in DeleteHandler

Adds Properties::pull() method

- Update documentation with pointers to new events

Salesforce mapping field plunging manager dependency to entity; removes
same hard dependency from constructor for testability

# Conflicts:
#	modules/salesforce_pull/src/DeleteHandler.php
#	modules/salesforce_pull/src/Plugin/QueueWorker/PullBase.php

Throwing exceptions on load() needs to be optional, otherwise form validation will choke since Entity API assumes an unpredictable result from load() operations

Moves module declaration to Drupal::logger creation in salesforce_pull.module
Adds Drupal::logger, EntityTypeManager, and Drupal::state dep inject to DeleteHandler
Adds logging trait to DeleteHandler
Fleshes out comments in DeleteHandler
Removes remaining procedural function call in favor of testable code in DeleteHandler
Removes EntityManager use from PullBase in favor of existing EntityTypeManager

Changes EntityManager injection in DeleteHandler to EntityTypeManager. EntityManager is due
to be deprecated.
Adds LoggingTrait to DeleteHandler - should probably examine use of watchdog_exception.
If it can be removed, the no longer need for trait.

# Conflicts:
#	modules/salesforce_pull/salesforce_pull.module
#	modules/salesforce_pull/src/DeleteHandler.php

- When "async" is false, attempt real-time push and enqueue async push on failure only
- When "async" is true, do not attempt real-time push; enqueue async push only

* 8.x-3.x:
  stripped out storage-as-service. just use the entity manager service to fetch storage and call it a day.
  Issue #2845064 by jtolj: Salesforce Pull uninstall broken
  override createInstance behavior to provide proper args to constructor
  Fixes syntax and reference errors
  refactor to use MappingConstants and eliminate module constants
  Get rid of salesforce_get_api() convenience wrapper for testability
  Fully replace procedural load operations with storage class operations.
  get entity type id from the proper place
  NOT TESTED AT ALL - end-of-day commit, but the basic class hierarchy is in place. - Create storage classes for mapping and mapped object - Trait for storage classes which throw an exception on load to override EntityStorageBase behavior
  Moves salesforce_mapping.module wrapper functions to traits

Also addresses Issue #2845048

Objectify salesforce_mapping.module wrapper functions for testability
Adds Storage services for mappings and mapped objects to replace convenience methods.

just use the entity manager service to fetch storage and call it a day.

this is ready to merge