Skip to content

[CI] Delete unused test artifact#3944

Merged
testableapple merged 1 commit intodevelopfrom
ci/delete-useless-file
Jan 30, 2026
Merged

[CI] Delete unused test artifact#3944
testableapple merged 1 commit intodevelopfrom
ci/delete-useless-file

Conversation

@testableapple
Copy link
Copy Markdown
Contributor

@testableapple testableapple commented Jan 30, 2026

Summary by CodeRabbit

  • Chores
    • Removed an unused test fixture file to streamline test infrastructure.

✏️ Tip: You can customize this high-level summary in your review settings.

@testableapple testableapple requested a review from a team as a code owner January 30, 2026 10:55
@testableapple testableapple added the 🤖 CI/CD Any work related to CI/CD label Jan 30, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

A test fixture file containing a mock chat message payload with a Giphy link attachment and associated user, member, and channel metadata was removed from the test tools directory.

Changes

Cohort / File(s) Summary
Test Fixture Removal
TestTools/StreamChatTestMockServer/Fixtures/JSONs/http_giphy_link.json
Deleted JSON test fixture simulating a complete chat message with video attachment, Giphy link, user profile data, reaction counts, and message state information.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 The Giphy fixture hops away,
A mock message departs today,
No more links in JSONs stored—
Cleanup hopping, neatly poured! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main change: deleting an unused test fixture file. It directly relates to the changeset which removes http_giphy_link.json.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ci/delete-useless-file

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions
Copy link
Copy Markdown

1 Warning
⚠️ Please provide a summary in the Pull Request description

Generated by 🚫 Danger

@testableapple testableapple enabled auto-merge (squash) January 30, 2026 11:53
@testableapple testableapple merged commit d1b17e5 into develop Jan 30, 2026
14 checks passed
@testableapple testableapple deleted the ci/delete-useless-file branch January 30, 2026 13:14
testableapple added a commit that referenced this pull request Feb 12, 2026
* [CI] Convert Xcode groups to buildable folders (#3940)

* [CI] Delete unused test artifact (#3944)

* [CI] Resolve mock server compilation issue (#3946)

* [CI] Resolve target membership issues after migrating to buildable folders (#3949)

* Make the members and messages size in ChannelListQuery optional (#3951)

* Prevent adding back deleted reactions in case of client errors (#3956)

* Fix refreshing channel when setting push notification preference for the first time (#3954)

* Bump 4.97.1

* Update release version to snapshot

---------

Co-authored-by: Martin Mitrevski <martinmitrevski.oh@gmail.com>
Co-authored-by: Toomas Vahter <toomas.vahter@getstream.io>
Co-authored-by: Stream Bot <ci@getstream.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖 CI/CD Any work related to CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants