Cdspt 15587 align functionality of totals methods for all extractor subclasses#18

Merged

TiffanyCheng27 merged 8 commits into

mainfrom

CDSPT-15587-align-functionality-of-totals-methods-for-all-extractor-subclasses

Jun 24, 2026

Commits on Jun 19, 2026

feat: added get_row_count and ensured implementation of get_total and get_row_count are the same across Extracto classes
scottbw
committed
test: updated tests to fit new contract whereby asking for a measure not in dataset.measures raises an error rather than silently giving you the row count.
scottbw
committed
fix: removed duplicate logic for missing measures, instead when we get a request for a query with measure None we just return Count(*). Substituting measures happens in the Extractor layer now.
scottbw
committed
test: Added a consistency test across implementations
scottbw
committed
fix: added a deprecation warning when calling get_total with no measure or None
scottbw
committed

Commits on Jun 23, 2026