Skip to content

Conversation

google-labs-jules[bot]
Copy link
Contributor

This change updates the test_loris2reproschema.py test to use a real data file (HBCD_LORIS.csv) instead of hardcoded dummy data. This ensures the test suite runs against more realistic data, which should help resolve a data-related issue that was causing a sorting problem in a downstream UI.

…rdcoded dummy CSV file. This has been updated to use a real data file, `HBCD_LORIS.csv`.

The new data file was downloaded from an external source as per user instruction to ensure the tests run against realistic data. This is intended to address a data-related issue that was causing problems in a downstream application.
@yibeichan yibeichan marked this pull request as ready for review August 15, 2025 03:58
@yibeichan yibeichan self-requested a review August 15, 2025 03:58
yibeichan and others added 4 commits August 17, 2025 10:22
…roschema

- Add path validation to prevent directory traversal patterns
- Add test case for missing configuration file handling
- Keep full 5.3MB test data file for comprehensive real-world testing
The command was accidentally duplicated at lines 200 and 375 in cli.py.
This removes the second duplicate definition.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
…tion

This script helps users easily run reproschema commands within the
micromamba environment without manual activation.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
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