Fix TagResultOutputMixin

[adrien-berchet]

No description provided.

[codecov]

Codecov Report

Merging #13 (c5e3b73) into main (0775a96) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #13   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            7         7           
  Lines          711       719    +8     
  Branches       133       134    +1     
=========================================
+ Hits           711       719    +8     

Flag	Coverage Δ
pytest	100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
data_validation_framework/target.py	100.00% <100.00%> (ø)
data_validation_framework/task.py	100.00% <100.00%> (ø)

[arnaudon]

What does it fix?

[adrien-berchet]

If several tasks have this mixin the prefix can change unexpectedly in some specific cases. Now the target of the BaseValidationTask only uses self.result_path (which is computed when the task is instanciated, not when it is executed) instead of using the default prefix of the target class.

[arnaudon]

Ah I see, cool!