fix(datalad): Separate validator process construction from wait, allowing error logging

[effigies]

We were having an UnboundLocalError show up in the logs:

This resulted from trying to read from process.stdout and process.stderr during a timeout.

This refactors subprocess.run() so that we can access the stdout/stderr in the event of a timeout. We then use the same code for both validators.

[codecov]

Codecov Report

Attention: Patch coverage is 76.92308% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 43.44%. Comparing base (cd5c1e0) to head (ce8ee30).
Report is 2 commits behind head on master.

Files	Patch %	Lines
...ervices/datalad/datalad_service/tasks/validator.py	76.92%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3048      +/-   ##
==========================================
+ Coverage   42.48%   43.44%   +0.96%     
==========================================
  Files         601      601              
  Lines       37971    37970       -1     
  Branches     1113     1147      +34     
==========================================
+ Hits        16131    16496     +365     
+ Misses      21624    21267     -357     
+ Partials      216      207       -9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.