Skip to content

Commit

Permalink
addressing PR feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
corva-alex-s committed Jan 5, 2024
1 parent e7d2440 commit f1489d4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/corva/models/merge/raw.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ class RawPartialMergeEventData(pydantic.BaseModel):
rerun_asset_id: int
app_stream_id: int
rerun_app_stream_id: int
version: Optional[int] = pydantic.Field(
version: int = pydantic.Field(
..., le=1, ge=1
) # Currently handler supports only 1-st version of this event.
app_id: Optional[int]
Expand Down
2 changes: 1 addition & 1 deletion tests/unit/test_partial_rerun_merge_app.py
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ def partial_rerun_merge_app(event, api, asset_cache, rerun_asset_cache):
assert partial_merge_app_response is True


def test_merge_events_parameter_ignored_for_partial_rerun_merge(context):
def test_merge_events_does_not_fail_for_partial_rerun_merge_events(context, mocker):
"""
merge_events parameter is ignored when incoming aws_event is not a list
"""
Expand Down

0 comments on commit f1489d4

Please sign in to comment.