Skip to content

Conversation

@aleksandar-apostolov
Copy link
Collaborator

@aleksandar-apostolov aleksandar-apostolov commented Sep 24, 2025

Goal

Add more tests.

Implementation

  • Add Sort tests
  • Add tests for factory methods and creation logic of the client and various dependencies.

Testing

Tests pass

Checklist

  • Issue linked (if any)
  • Tests/docs updated
  • I have signed the Stream CLA (required for external contributors)

aleksandar-apostolov and others added 10 commits September 19, 2025 16:30
# Conflicts:
#	stream-android-core/src/main/java/io/getstream/android/core/internal/socket/StreamSocketSession.kt
#	stream-android-core/src/test/java/io/getstream/android/core/internal/socket/StreamSocketSessionTest.kt
…i/socket/StreamWebSocket.kt

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Sep 24, 2025

PR checklist ✅

All required conditions are satisfied:

  • Title length is OK (or ignored by label).
  • At least one pr: label exists.
  • Sections ### Goal, ### Implementation, and ### Testing are filled.

🎉 Great job! This PR is ready for review.

@aleksandar-apostolov aleksandar-apostolov added the pr:test Test-only changes label Sep 24, 2025
@VelikovPetar VelikovPetar merged commit ec5eff0 into develop Sep 25, 2025
5 checks passed
@VelikovPetar VelikovPetar deleted the tests branch September 25, 2025 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:test Test-only changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants