Skip to content

fix(testmanagement): return the upstream message from updateTestCase failures - #484

Merged
gaurav-singh-9227 merged 4 commits into
browserstack:mainfrom
SavioBS629:fix/update-testcase-error-message
Oct 16, 2026
Merged

gaurav-singh-9227 merged 4 commits into
browserstack:mainfrom
SavioBS629:fix/update-testcase-error-message

Conversation

@SavioBS629

Copy link
Copy Markdown
Collaborator

The fallback branch interpolated the CallToolResult returned by formatAxiosError into a string, so every non-404/403 failure reached the caller as "[object Object]" with a credentials hint. The response now carries the HTTP status and the Test Management message, with the credentials hint only on 401. errorText() serialises object bodies so the helper cannot produce the same output for any caller.

…failures

The fallback branch interpolated the CallToolResult returned by formatAxiosError
into a string, so every non-404/403 failure reached the caller as
"[object Object]" with a credentials hint. The response now carries the HTTP
status and the Test Management message, with the credentials hint only on 401.
errorText() serialises object bodies so the helper cannot produce the same
output for any caller.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Central YAML (base), Organization UI (inherited), Workspace UI (inherited)
  • Review profile: ASSERTIVE
  • Plan: Enterprise
  • Run ID: 90c98aaf-51d9-453d-baa9-1c7c0cf33dbe

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

sarve-shreyas
sarve-shreyas previously approved these changes Oct 14, 2026
SavioBS629 and others added 2 commits October 14, 2026 14:48
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
An empty or whitespace-only body, or a whitespace-only message field, now
falls back to the error's own message instead of surfacing a literal "".
Surfaced text is capped at 500 characters. errorText has direct unit tests
covering field precedence, object serialisation, unknown body shapes, empty
bodies, the cap, and non-axios errors.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@gaurav-singh-9227
gaurav-singh-9227 merged commit f2473f8 into browserstack:main Oct 16, 2026
2 checks passed
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.

3 participants