Skip to content

Conversation

@stark-cybertech
Copy link
Collaborator

Type of Change

  • πŸ› Bug fix
  • ✨ New feature
  • ♻️ Code refactor
  • πŸ“ Documentation update
  • πŸš€ Performance improvement
  • βœ… Test addition/update
  • 🧹 Chore/maintenance
  • πŸ“ API Spec update
  • Other (please describe):

Changes

  • Adds basic testing structure for matching-microservice

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have updated the API spec to reflect any changes my code makes

Related Issues

Fixes #
Related to #

Additional Notes

Tests currently fail. Will be fixed in next commit
Moving getMatchStatus(UUID groupID) into GroupService allows removing the dependency on MatchService.
@stark-cybertech stark-cybertech self-assigned this Jul 2, 2025
@stark-cybertech stark-cybertech merged commit 40c5c63 into feature/matching-service Jul 2, 2025
9 checks passed
@stark-cybertech stark-cybertech deleted the feature/matching-service-tests branch July 2, 2025 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants