-
Notifications
You must be signed in to change notification settings - Fork 93
Open
Labels
enhancementNew feature or requestNew feature or request
Description
Problem
The stop-loss/take-profit scenario test in tests/scenarios/test_strategy_with_stop_losses_take_profits.py is currently skipped via @unittest.skip as part of the scenario test optimization effort.
Required changes
-
Reduce execution time — Ensure the test completes within 30 seconds. Use minimal date ranges that still exercise the stop-loss and take-profit logic.
-
Use only offline test data from
tests/resources/test_data/— All data sources must reference CSV files located intests/resources/test_data/(or its subdirectories). The test currently usesCSVOHLCVDataProviderwhich is good, but ensure the referenced CSV file lives undertests/resources/test_data/.
Affected files
tests/scenarios/test_strategy_with_stop_losses_take_profits.py
Acceptance criteria
- Test runs in under 30 seconds
- Test uses data exclusively from
tests/resources/test_data/ - Remove the
@unittest.skipdecorator once optimized - Test passes on both macOS and Ubuntu CI runners
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request