Details
-
Task
-
Status: Done
-
L3 - Default
-
Resolution: Won't Do
-
None
-
None
-
Not defined
Description
AT:
- duplicated code in the user task duration/frequency commands and integration tests are being removed and only the changing parts are being kept
Hint:
It's already quite complicated to understand how the query is being build. Maybe we should think about a different way of executing the queries instead of adding to the complex structure.
Affected components:
- user task integration tests
- user task commands
- ProcessReportDataCreator
- SingleReportEvaluator
- ProcessReportDataBuilderHelper
Possible refactorings:
- Add report data creation to command itself
- Add report builder class
- Command is build up by a list of aggregation steps. Each step conatins buildAggregation and processAggregation methods