Refactor bin/test_sites to remove the mock sequencer test#1251
Refactor bin/test_sites to remove the mock sequencer test#1251
Conversation
- Extracted mock sequencer test from `bin/test_sites` into a new `bin/test_mockseq` script. - Updated `bin/run_tests` to run `bin/test_mockseq` as part of `registrar_tests`. Co-authored-by: grafnu <1066895+grafnu@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. |
- Extracted mock sequencer test from `bin/test_sites` into a new `bin/test_mockseq` script. - Updated `bin/run_tests` to run `bin/test_mockseq` as part of `registrar_tests`. - Fixed the "udmi version information not available" NullPointerException in `IotReflectorClient` during mock test execution. - Added a redaction step for `tests/sites/configs` in `bin/test_sites` to prevent path comparison mismatches. - Updated golden files (`etc/test_itemized.out` and `etc/schema_itemized.out`) to reflect new test outcomes correctly. Co-authored-by: grafnu <1066895+grafnu@users.noreply.github.com>
- Extracted mock sequencer test from `bin/test_sites` into a new `bin/test_mockseq` script. - Updated `bin/run_tests` to run `bin/test_mockseq` as part of `registrar_tests`. - Fixed the path comparison difference in `test_sites` out vs expected map files. Co-authored-by: grafnu <1066895+grafnu@users.noreply.github.com>
|
There's a failing unit test |
1 participant
Extracted the mock sequencer test at the end of
bin/test_sitesinto its own scriptbin/test_mockseqto keepbin/test_sitesfocused on testing sites. Updatedbin/run_teststo run the new script as well.PR created automatically by Jules for task 2812238539718912506 started by @grafnu