Refactor code structure for better readability and maintainability

[danhellem]

This pull request updates the tests for the configureWorkTools function to reflect recent changes in tool registration and handler invocation. The main focus is on consolidating tool names and ensuring the correct action parameters are passed in test cases.

Test suite alignment with new tool registration:

• Updated all test cases to look for the "work" tool instead of the previous "work_list_team_iterations" and "work_list_iterations" tool names, ensuring consistency with the refactored tool registration logic. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17]

Parameterization improvements:

• Added the appropriate action field (e.g., "list_team_iterations" or "list_iterations") to the parameters passed to tool handlers in all relevant test cases, ensuring that the handler receives the correct context for each action. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]

These changes keep the test suite in sync with the updated implementation and improve clarity and maintainability.

GitHub issue number

N/A

Associated Risks

Changing signatures of tools and consolidating. Starting with smaller toolsets to make sure this goes well.

✅ PR Checklist

• I have read the contribution guidelines
• I have read the code of conduct guidelines
• Title of the pull request is clear and informative.
• 👌 Code hygiene
• 🔭 Telemetry added, updated, or N/A
• 📄 Documentation added, updated, or N/A
• 🛡️ Automated tests added, or N/A

🧪 How did you test it?

Ran several manual tests. Updated the automated tests

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[krid-583]

NIT: Would be better if we could implement a singular function for required arguments that don't elicit. Need to check on the feasibility of this.