Re-add deprecated `--execution` arg on `benchmark pallet`

[ggwpez]

Changes:

• Re-add the --execution arg on command benchmark pallet that was removed in #14387. It is a No-OP and prints ⚠️ Argument --execution is deprecated. Its value of wasm will be ignore now upon usage.
• Add DeferGuard::new as convenience function.
• Extend benchmark * cmd tests.

[bkchr]

Why bring it back?

I mean yes, I should not have removed it directly, but now this happened. This is also more a dev centric command and not really used by "users".

[ggwpez]

Why bring it back?

I mean yes, I should not have removed it directly, but now this happened. This is also more a dev centric command and not really used by "users".

Its probably used in automation (also by downstream teams). So I think having some grace period is some nicer experience.
But its surely not really necessary, so I can also close again 🤷‍♂️

[bkchr]

Why bring it back? I mean yes, I should not have removed it directly, but now this happened. This is also more a dev centric command and not really used by "users".

Its probably used in automation (also by downstream teams). So I think having some grace period is some nicer experience. But its surely not really necessary, so I can also close again man_shrugging

Yeah, IDK. Then let's keep it, but we don't need this in all the tests.

[ggwpez]

bot merge

[paritytech-processbot[bot]]

Waiting for commit status.