🧪 Add minitest for erb.rb malformed data error handling - #39
Conversation
Adds a minitest script `test/test_erb.rb` to ensure that malformed weapon data encountered during the categorization logic doesn't crash `scripts/erb.rb`. The test creates a sandbox using `mktmpdir` to prevent repository pollution, simulates a bad YAML file, and asserts the process exits 0 after catching the exception.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
🎯 What: The testing gap addressed: Missing error test for the second data processing block in
erb.rb.📊 Coverage: What scenarios are now tested: Injecting malformed weapon data (
tags: 123instead of an array) to trigger and assert therescue => eerror handling when processing tags and language categorizations.✨ Result: The improvement in test coverage: We have an automated minitest suite
test_erb.rbthat simulates execution in a safe temp directory to confirm robustness without mutating the git tree.PR created automatically by Jules for task 1008594445152255818 started by @hahwul