Skip to content

Commit 8b52ddb

Browse files
committed
fix(tests): update test_model_code_change.py for clarity and context
- Add a comment to clarify the reason for skipping tests due to outdated `test_full_model.yaml`. - Remove unnecessary sorting and resetting of index in the test to streamline the DataFrame processing.
1 parent 577b308 commit 8b52ddb

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

tests/context/plan_and_run/test_model_code_change.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ def test_given_model_chain_when_running_with_different_flags_then_behaves_as_exp
111111
plan_options=PlanOptions(
112112
skip_backfill=skip_backfill,
113113
enable_preview=True,
114-
skip_tests=True,
114+
skip_tests=True, # This is because the test_full_model.yaml is not updated to reflect the new model changes
115115
),
116116
)
117117

@@ -153,8 +153,6 @@ def test_given_model_chain_when_running_with_different_flags_then_behaves_as_exp
153153
"num_orders": [5, 1, 1],
154154
}
155155
)
156-
.sort_values(by="item_id")
157-
.reset_index(drop=True)
158156
)
159157

160158
print("full_model_df")

0 commit comments

Comments
 (0)