Adapt message arguments passing to process controller

[unkcpz]

Changes required after aiidateam/plumpy#301 and aiidateam/plumpy#291

[codecov]

Codecov Report

Attention: Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 77.92%. Comparing base (c532b34) to head (5f8e4f1).

Files with missing lines	Patch %	Lines
src/aiida/engine/processes/functions.py	50.00%	1 Missing ⚠️
src/aiida/engine/runners.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6668      +/-   ##
==========================================
+ Coverage   77.92%   77.92%   +0.01%     
==========================================
  Files         563      563              
  Lines       41671    41672       +1     
==========================================
+ Hits        32467    32470       +3     
+ Misses       9204     9202       -2     

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

[khsrali]

Thanks a lot @unkcpz , I have just one comment.

[khsrali]

Thanks @unkcpz
in _perform_actions we have:

 future = action(process.pk, **kwargs)

Therefore I would suggest, either put everything inside functools.partial so it would be serve as we suggested and discussed (one action that would be triggered), or probably take the msg_text out.
Right now, it's kinda hard to understand why because the rest of arguments are passed via **kwargs .

[unkcpz]

Yep, makes sense. I I removed the kwargs and add the typing for action argument as well.

[khsrali]

@unkcpz thanks a lot! changes makes sense.
As agreed let's wait for a plumpy release before merging this. and we have to remember to remove the commit hook for plumpy so to pin the to-be-released version.