AOSSIE-Org / AOSSIE-Org/OpenVerifiableLLM

[BUG]: verify_preprocessing() stores resolved path instead of original in VerificationReport

Open
#74 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
18
Forks
31
Avg merge
1m
Merged PRs (30d)
2

Description

### Bug Description

## Description
`verify_preprocessing()` calls `Path.resolve()` on the user-supplied
`input_dump` before storing it in `VerificationReport.input_dump`.
On Windows, `resolve()` can alter path casing via junction point
expansion, causing `report.input_dump != str(original_path)`.

## Failing Test
`tests/test_verify.py::TestHappyPath::test_report_stores_input_dump_path`

## Expected Behaviour
`report.input_dump` should equal the original path string passed by the caller.

## Actual Behaviour
On Windows, the stored path may differ in casing from the original.

### Steps to Reproduce

1. Clone the repository on Windows
2. Run `pytest tests/test_verify.py::TestHappyPath::test_report_stores_input_dump_path -v`
3. Observe the test fails

### Logs and Screenshots

image

### Environment Details

-OS : Windows 11

### Impact

Medium - Feature works but has issues

### Code of Conduct

- [x] I have joined the [Discord server](https://discord.gg/hjUhu33uAn) and will post updates there
- [x] I have searched existing issues to avoid duplicates

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.