-
-
Notifications
You must be signed in to change notification settings - Fork 677
General Test Case Refactor #2875
Copy link
Copy link
Closed
Labels
Description
As per discussed in #2848 and #2874:
Interesting suggestion from the bot, do we want this for later tests elsewhere?
It can be useful. We could also add something like
cls.full_range = range(..., ...)and use it in the required places asself.full_range.
I think, we need to pass years range to
setUpClass, and let it createcls.full_range,cls.holidays,cls.holidays_non_observed, ...
cls.no_estimated_holidays- iftest_classinherited fromIslamicHolidays(or evenEasternCalendarHolidays),
cls.subdiv_holidays- iftest_classhave subdivisions...
- Migration to
cls.full_rangein place ofrange(start_year, 2050)that's still in widespread usage - Migration to
years_islamic_no_estimatedparameter fromcls.no_estimated_holidaysforsetUpClass - Migration to
years_subdivparameter fromcls.subdiv_holidaysforsetUpClass - Migration to
year_subdiv_non_observedparameter fromcls.subdiv_holidays_non_observedforsetUpClass
Reactions are currently unavailable