[PR] Add test coverage for MetricsRecordConverter [polaris]

[via GitHub]

hemasrishalini opened a new pull request, #4636:
URL: https://github.com/apache/polaris/pull/4636

   ## Summary
   
   Adds direct unit test coverage for `MetricsRecordConverter`.
   
   ### Changes
   
   * Add `testScanReportConversion()` to verify conversion from `ScanReport` to 
`ScanMetricsRecord`
   * Add `testCommitReportConversion()` to verify conversion from 
`CommitReport` to `CommitMetricsRecord`
   * Verify key mapped fields including:
   
     * catalog ID
     * table ID
     * snapshot ID
     * schema ID
     * filter expression
     * sequence number
     * operation type
   
   ### Testing
   
   Ran:
   
   ```bash
   ./gradlew :polaris-core:test --tests "*MetricsRecordConverterTest"
   ```
   
   and verified the tests pass successfully.
   
   ## Checklist
   
   * [x] 🧪 Added/updated tests with good coverage, or manually tested (and 
explained how)
   * [ ] 🧾 Updated CHANGELOG.md (not needed - test-only change)
   * [ ] 📚 Updated documentation (not needed - test-only change)
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org