Repository navigation
Align robot catalog tests with vendor modules and terminal width - #878
Merged
Merged
Conversation
[Low impact] This test-only PR appears safe to merge. SummaryThis PR repairs two tests after the robot catalog moved to vendor modules.
Production code and existing assertions are unchanged. No actionable issues were found. Tests were not run during this review. Reviews (1) · Last reviewed commit: "test: align robot catalog checks with ve..." · Reviewed by Greptile |
This was referenced Oct 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Repair two tests after the vendor robot catalog migration in #875. The list-robot label case checks an unwrapped label at an explicit 120-column terminal width; narrow responsive table wrapping is not a label-semantic failure. The UR embodiment mock now targets the canonical
embodichain_robots.universal_robots.ur_robotowner rather than a retired flat module.Only two test files change (+3/-1). Production CLI/robot behavior, assertions, pytest import policy, CI selection and test gates are unchanged; no compatibility module or skip is added.
The same failures were exposed by the full main-based suites for #864, #859 and #858. This focused maintenance commit is carried into those branches so they can be checked by the unchanged main CI while this fix is reviewed.
Validation
c5b1fdfc2, the original two cases fail with the CI errors; after the change both pass.black ., whitespace checks and API coverage 3206/3206 pass.Type of change
Checklist
black ..