Job.get_status() does not always return JobStatus Enum

[indepndnt]

As it is, the return type for get_status is Union[JobStatus, str, None]. This results in an error if you expect an Enum and call Job.get_status().value.

I believe the return type annotations in line 325 and 332 should be Optional[JobStatus] and line 336 should be updated to self._status = JobStatus(as_text(status)) if status else None.

rq/rq/job.py

Lines 325 to 337 in 2f5fecc

	def get_status(self, refresh: bool = True) -> JobStatus:
	"""Gets the Job Status
	Args:
	refresh (bool, optional): Whether to refresh the Job. Defaults to True.
	Returns:
	status (JobStatus): The Job Status
	"""
	if refresh:
	status = self.connection.hget(self.key, 'status')
	self._status = as_text(status) if status else None
	return self._status

Also line 950 in Job.restore should probably be self._status = JobStatus(as_text(obj.get('status'))) if obj.get('status') else None for consistency and so that it doesn't set Job._status to a string.

rq/rq/job.py

Line 950 in 2f5fecc

self._status = obj.get('status').decode() if obj.get('status') else None

I also noticed that the docstring for the as_text function is inconsistent, it should probably say value (str): The decoded string.

rq/rq/utils.py

Lines 41 to 52 in 2f5fecc

	def as_text(v: Union[bytes, str]) -> str:
	"""Converts a bytes value to a string using `utf-8`.
	Args:
	v (Union[bytes, str]): The value (bytes or string)
	Raises:
	ValueError: If the value is not bytes or string
	Returns:
	value (Optional[str]): Either the decoded string or None
	"""