SCRUM-327: fix(rewards) register excel parser in DI, adopt Result pattern, and add unit tests for voucher upload

[Moha-sami]

Summary

• Register \IExcelVoucherParserService\ in Dependency Injection to fix runtime \InvalidOperationException\ (500)
• Adopt \Result\ pattern instead of throwing exceptions
• Remove broken \GetCurrentUserId()\ that threw \NotImplementedException\
• Safely parse \BatchId\ with fallback generation for non-numeric batch strings
• Fix CA2024 warning in \ExcelVoucherParser\
• Add 14 comprehensive unit tests in \UploadRewardVouchersCommandTests\ covering all preview, commit, validation, and controller flows (748 tests passing, 0 warnings)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
hr-system	Ready	Preview	Sep 20, 2026 6:11pm UTC

[github-actions]

Automated PR Validation Feedback

Action Needed on this Pull Request:

• Code Formatting Failed: 5 changed C# file(s) do not match formatting rules. Run dotnet format locally on your changed files and commit the result.

Please push a fix to your branch to re-run validation!