in AEP, use active_test=False

[sbidoul]

This is important to consider all move lines in computations.
Without this, a formula based on deactivated taxes would
return no result, which is wrong since the corresponding
move lines do exist.

[sbidoul]

@jbeficent @benwillig do you have an opinion on this one?

[benwillig]

In my opinion, it's logic to disable the active test. But maybe it should be an option in the MIS report instance (or maybe a checkbox in the instance view). If not the user should be informed that we also get data linked to inactive objects, because if the user check the journal items, he might be confused about the differences between the report and the data shown in the list view.

[sbidoul]

@benwillig I thought about that too, but I could not imagine a realistic situation where this could cause in issue in practice. Do you have one in mind?

[benwillig]

No I have no real use case.

[sbidoul]

Tested in production.