🔧 FIX auto_persist decorator typing

[chrisjsewell]

This is used in aiida-core,
to decorate the WorkChain class.
The problem currently is that, because of the decorator typing,
no methods defined on subclasses of Savable
are available for static analysers (such as IDE auto-completions).
This change ensures the output of the decorator is the same as the input.

[codecov]

Codecov Report

Base: 90.46% // Head: 90.46% // Increases project coverage by +0.01% 🎉

Coverage data is based on head (e4efc9e) compared to base (8c7640b).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #239      +/-   ##
==========================================
+ Coverage   90.46%   90.46%   +0.01%     
==========================================
  Files          21       21              
  Lines        2953     2954       +1     
==========================================
+ Hits         2671     2672       +1     
  Misses        282      282              

Impacted Files	Coverage Δ
src/plumpy/persistence.py	93.93% <100.00%> (+0.02%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[chrisjsewell]

and obviously I tested this fix works, in the context of VS Code:

(this does not work currently)

[sphuber]

and obviously I tested this fix works, in the context of VS Code:

How does this show that it is working? The to_context method is not decorated with auto_persist is it?

[chrisjsewell]

The to_context method is not decorated with auto_persist is it?

No, the whole WorkChain class is decorated with auto_persist

How does this show that it is working?

This is it without the change, all it knows is that wc is a Savable