Skip to content

Feature/matching service tests#63

Merged
stark-cybertech merged 4 commits into
feature/matching-servicefrom
feature/matching-service-tests
Jul 2, 2025
Merged

Feature/matching service tests#63
stark-cybertech merged 4 commits into
feature/matching-servicefrom
feature/matching-service-tests

Conversation

@stark-cybertech

Copy link
Copy Markdown
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.

1 participant