Skip to content
Permalink

Comparing changes

This is a direct comparison between two commits made in this repository or its related repositories. View the default comparison for this range or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: dagster-io/dagster
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: f8913e4e916366108d88c945d2d771d0a6069bd8
Choose a base ref
..
head repository: dagster-io/dagster
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 5191aeca6fb45225652e9091bdfbc4b957d5c1e7
Choose a head ref
Showing with 3 additions and 3 deletions.
  1. +3 −3 ...ries/dagster-components/dagster_components_tests/integration_tests/validation_tests/test_cases.py
6 changes: 3 additions & 3 deletions ...ster-components/dagster_components_tests/integration_tests/validation_tests/test_cases.py
View file
Original file line number Diff line number Diff line change
@@ -37,7 +37,7 @@ def _validate_error_msg(msg: str) -> None:
component_path="validation/basic_component_missing_value",
component_type_filepath=Path(__file__).parent / "basic_components.py",
should_error=True,
validate_error_msg=msg_includes_all_of("component.yaml:4", "params.an_int", "required"),
validate_error_msg=msg_includes_all_of("component.yaml:3", "params.an_int", "required"),
validate_error_msg_additional_cli=msg_includes_all_of(
"Field `an_int` is required but not provided"
),
@@ -67,7 +67,7 @@ def _validate_error_msg(msg: str) -> None:
"component.yaml:7",
"params.nested.foo.an_int",
"Input should be a valid integer",
"component.yaml:12",
"component.yaml:1",
"params.nested.baz.a_string",
"Input should be a valid string",
),
@@ -77,7 +77,7 @@ def _validate_error_msg(msg: str) -> None:
component_type_filepath=Path(__file__).parent / "basic_components.py",
should_error=True,
validate_error_msg=msg_includes_all_of(
"component.yaml:6", "params.nested.foo.an_int", "required"
"component.yaml:5", "params.nested.foo.an_int", "required"
),
validate_error_msg_additional_cli=msg_includes_all_of(
"Field `a_string` is required but not provided"