Allow jobs to be passed in lazily to repository definitions (#7382)

Summary:
This check was crashing when a job was passed in lazily.

It will still crash when a graph is passed in lazily (the assumption that repos consist of jobs not graphs appears to be pretty deep) but this will make it so that users can pass in a lambda that returns a job. The error is a bit odd, but is the same error you would get if you passed in a job function that returned some other random object that wasn't a job.

python_modules/dagster/dagster/core/definitions/repository_definition.py

@@ -1,5 +1,6 @@
 import warnings
 from abc import ABC, abstractmethod
+from inspect import isfunction
 from types import FunctionType
 from typing import (
     TYPE_CHECKING,
@@ -594,7 +595,7 @@ def from_dict(repository_definitions: Dict[str, Dict[str, Any]]) -> "CachingRepo
 
             if isinstance(job, GraphDefinition):
                 repository_definitions["jobs"][key] = job.coerce_to_job()
-            elif not isinstance(job, JobDefinition):
+            elif not isinstance(job, JobDefinition) and not isfunction(job):
                 raise DagsterInvalidDefinitionError(
                     f"Object mapped to {key} is not an instance of JobDefinition or GraphDefinition."
                 )

python_modules/dagster/dagster_tests/core_tests/definitions_tests/test_repository_definition.py

@@ -25,6 +25,7 @@
     sensor,
     solid,
 )
+from dagster.check import CheckError
 from dagster.core.definitions.partition import PartitionedConfig, StaticPartitionsDefinition
 
 
@@ -465,6 +466,51 @@ def jobs():
     assert jobs.get_job("other_graph")
 
 
+def test_lazy_jobs():
+    @graph
+    def my_graph():
+        pass
+
+    @repository
+    def jobs():
+        return {
+            "jobs": {
+                "my_graph": my_graph,
+                "my_job": lambda: my_graph.to_job(name="my_job"),
+                "other_job": lambda: my_graph.to_job(name="other_job"),
+            }
+        }
+
+    assert jobs.get_pipeline("my_graph")
+    assert jobs.get_pipeline("my_job")
+    assert jobs.get_pipeline("other_job")
+
+    assert jobs.has_job("my_graph")
+    assert jobs.get_job("my_job")
+    assert jobs.get_job("other_job")
+
+
+def test_lazy_graph():
+    @graph
+    def my_graph():
+        pass
+
+    @repository
+    def jobs():
+        return {
+            "jobs": {
+                "my_graph": lambda: my_graph,
+            }
+        }
+
+    # Repository with a lazy graph can be constructed, but fails when you try to fetch it
+    with pytest.raises(
+        CheckError,
+        match="Invariant failed. Description: Bad constructor for job my_graph: must return JobDefinition",
+    ):
+        assert jobs.get_pipeline("my_graph")
+
+
 def test_list_dupe_graph():
     @graph
     def foo():