Use literal identifier for ImportJobStoreFactory

[paul-m]

Moves some table logic from JobStoreFactory to DeprecatedJobStoreFactoryTrait. This trait contains all the behavior we want to leave behind, for both deprecated table names, and deriving the table name from an identifier.

Adds AbstractJobStoreFactory, which allows subclasses to dictate the table name. Also uses DeprecatedJobStoreFactoryTrait as a fallback to performs all the logic for determining the table name.

Adds ImportJobStoreFactory, which is a subclass of AbstractJobStoreFactory. This hard-codes the table name as jobstore_2613055649_importjob for historical reasons.

Adds dkan.datastore.import_job_store_factory service, which is ImportJobStoreFactory.

Deprecates JobStoreFactory, which now basically only uses DeprecatedJobStoreFactoryTrait.

Finds all code occurrences which uses the old calling system like this: $this->jobStoreFactory->getInstance(ImportJob::class), and replaces them with $this->importJobStoreFactory->getInstance(). This includes replacing or adding the import job store factory service.

Updates tests to pass.

Adds tests for AbstractJobStoreFactory.