Repository navigation
Fix JSONDecodeError in E2E tests when stdout is empty or warning-only - #93
Conversation
Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
|
👋 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. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the
|
Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
…E2E tests Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Updated
parse_json_outputintests/test_e2e.pyto return an empty list[]whenbwCLI output is empty or contains only warning messages, preventingjson.decoder.JSONDecodeErrorfailures during E2E test runs. Added corresponding unit tests intests/test_e2e_helpers.py.PR created automatically by Jules for task 8467635074331933391 started by @mvfc


Confidence Score: 5/5
The PR appears safe to merge; no outstanding issues were found.Summary
The PR adds an initial
bw syncto the E2E session setup and tests thatparse_json_outputrejects empty or warning-only output.Reviews (3) · Last reviewed commit: "Ensure bw sync in bw_session fixture to ..." · Reviewed by Greptile