fix multi asset check selection (#16877)

I didn't test selecting inside a multi asset over the graphql layer, and lo and behold it wasn't being passed properly. Yet more fodder for a combo selection object. --------- Co-authored-by: Rex Ledesma <[email protected]>
dagster-io · Oct 3, 2023 · df4fdae · df4fdae
1 parent 22e0e1e
commit df4fdae
Show file tree

Hide file tree

Showing 13 changed files with 12,016 additions and 10,208 deletions.
diff --git a/python_modules/dagster-graphql/dagster_graphql/implementation/execution/run_lifecycle.py b/python_modules/dagster-graphql/dagster_graphql/implementation/execution/run_lifecycle.py
@@ -93,7 +93,7 @@ def create_valid_pipeline_run(
         ),
         asset_check_selection=(
             frozenset(execution_params.selector.asset_check_selection)
-            if execution_params.selector.asset_check_selection
+            if execution_params.selector.asset_check_selection is not None
             else None
         ),
         op_selection=execution_params.selector.op_selection,