[MNT] Update setup list test assertion#1652
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the assertion in test_setup_list_filter_flow to replace a placeholder check (assert len(setups) > 0) with a specific count check (assert len(setups) == 2) based on current production server data.
Changes:
- Changed the setup list assertion from
> 0to== 2to reflect the actual number of setups for flow ID 5873 on the production server
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1652 +/- ##
==========================================
- Coverage 53.07% 52.72% -0.35%
==========================================
Files 37 37
Lines 4381 4381
==========================================
- Hits 2325 2310 -15
- Misses 2056 2071 +15 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|

Fixes #1645
Current number of setups is 2.
cc @geetu040 @PGijsbers so should we use the lower bound as 1?