mysql_user: deprecate alias user for name argument

[Andersson007]

SUMMARY

Relates to #669.
It wasn't even documented, the failures are addressed in #668

[Andersson007]

ready for review

[Andersson007]

if we wanna merge this, #668 should go first, then I'll rebase it

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.50%. Comparing base (37a718c) to head (89b9b56).
Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #670      +/-   ##
==========================================
+ Coverage   76.31%   76.50%   +0.19%     
==========================================
  Files          20       20              
  Lines        2651     2652       +1     
  Branches      676      676              
==========================================
+ Hits         2023     2029       +6     
+ Misses        427      422       -5     
  Partials      201      201              

Flag	Coverage Δ
integration	75.86% <100.00%> (+0.19%)	⬆️

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

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

[Andersson007]

ready for review

[samccann]

Not really qualified for an official review, but the changes make sense to me.

[laurent-indermuehle]

I'm not super fan of removing an alias afterward.
But I parsed all my code base and it turns out, I only use 'name'. Not surprised because it's the default for Ansible modules after all.

So for simplification sake, let's merge this! :)
Thanks @Andersson007 for the work!

[Andersson007]

@samccann @laurent-indermuehle thanks for reviewing!