feat: move CalculateArguments to MatchArgumentCalculator

[olegKoshmeliuk]

🤔 What's changed?

As suggested in #238 (review)
Moved CalculateArguments to MatchArgumentCalculator.

⚡️ What's your motivation?

Allow override of how Reqnroll calculates parameters with old/legacy behavior.

🏷️ What kind of change is this?

• 🏦 Refactoring/debt/DX (improvement to code design, tooling, etc. without changing behavior)

♻️ Anything particular you want feedback on?

I don't see useful writing tests for this new class or testing StepDefinitionMatchService with mock MatchArgumentCalculator.

An alternative approach can be done by just changing CalculateArguments to protected virtual that will not require updating tests

📋 Checklist:

• I've changed the behavior of the code
  • I have added/updated tests to cover my changes.
• My change requires a change to the documentation.
  • I have updated the documentation accordingly.
• Users should know about my change
  • I have added an entry to the "[vNext]" section of the CHANGELOG, linking to this pull request & included my GitHub handle to the release contributors list.

---

This text was originally taken from the template of the Cucumber project, then edited by hand. You can modify the template here.

[gasparnagy]

This is great. See my comments about the global container below.
Besides that, please add a line for it to the changelog (like "added option to override regex group matching behavior") and list yourself to the contributors of this release.

[gasparnagy]

Thx!

[gasparnagy]

@olegKoshmeliuk Thank you for the contribution! According to our guidelines I have invited you to the Reqnroll contributors team. Congrats! 🎉 If you accept it, you will be able to make pull requests easier in the future.

You are also welcome on our discord server: https://go.reqnroll.net/discord-invite