refactor: remove traces and attachments from test case summary#258
Merged
Conversation
The summary header's Storage card duplicated data already shown elsewhere: traces and every attachment (screenshots, videos, non-media files) appear in the Failure evidence card on the Diagnosis tab for failing cases, and in the Traces + Attachments cards on the Artifacts tab for passing cases. Drop the block and its now-unused helpers (attFileUrl, isImage, totalStorageSize, fileName, image ext/mime sets, runtime config) plus the traces/attachments props threaded in from the page. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Rdhk8bsdrPdE7AEKTXFVbj
Contributor
Coverage Report for Reporter (./reporter)
File CoverageNo changed files found. |
Contributor
Coverage Report for Application (./application)
File CoverageNo changed files found. |
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
2 participants
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.
What & why
Removes the traces and attachments storage card from the TestCaseSummary component. This includes:
TraceInfoandAttachmentInfotype importstracesandattachmentsprops from the componentattFileUrl,isImage,totalStorageSize,fileName) that were only used for rendering this cardThe storage card display logic is being consolidated or moved elsewhere, simplifying the TestCaseSummary component to focus on core test case metadata.
How was it tested?
No testing needed — this is a straightforward refactor removing unused UI components and their supporting code. The component will continue to render all other cards (SCM info, CI info, browser, links) as before.
Checklist
https://claude.ai/code/session_01Rdhk8bsdrPdE7AEKTXFVbj