Provide return annotations to all providers.JobV1.result() methods (#…

…11531)

* Make providers.JobV1 generic on result() return

Makes `JobV1` a `Generic[T]` class where `T` is the return type of the
`run()` method. The intended value is to allow tools such as the primitives
to be more explicit about what the jobs they return will contain.

* Remove future import

* alphebetize imports

* switch to covariant TypeVar

* remove generic and just use Result

* avoid circular import

* fix circular imports attempt 2

qiskit/providers/job.py

@@ -12,15 +12,20 @@
 
 """Job abstract interface."""
 
+from __future__ import annotations
+
 import time
 from abc import ABC, abstractmethod
-from typing import Callable, Optional
+from typing import Callable, Optional, TYPE_CHECKING
 
 from qiskit.exceptions import QiskitError
 from qiskit.providers.backend import Backend
 from qiskit.providers.exceptions import JobTimeoutError
 from qiskit.providers.jobstatus import JOB_FINAL_STATES, JobStatus
 
+if TYPE_CHECKING:
+    from qiskit.result import Result
+
 
 class Job:
     """Base common type for all versioned Job abstract classes.
@@ -128,7 +133,7 @@ def submit(self):
         pass
 
     @abstractmethod
-    def result(self):
+    def result(self) -> Result:
         """Return the results of the job."""
         pass
 
@@ -137,6 +142,6 @@ def cancel(self):
         raise NotImplementedError
 
     @abstractmethod
-    def status(self):
+    def status(self) -> JobStatus:
         """Return the status of the job, among the values of ``JobStatus``."""
         pass