Skip to content

Conversation

@shaljam
Copy link
Contributor

@shaljam shaljam commented Apr 1, 2025

Submit a pull request

CLA

  • I have signed the Stream CLA (required).
  • The code changes follow best practices
  • Code changes are tested (add some information if not applicable)

Description of the pull request

@shaljam shaljam changed the title [CHA-0] fix: use the default blocklist for the test [CHA-0] Fix failing tests on CI Apr 1, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request fixes failing tests on CI by updating the test configuration and preparing a clean user state before test execution.

  • Refactors the moderation test to use a variable for the blocklist name rather than a hard-coded string.
  • Adds a new method in BasicTest to clean up users before tests run.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/test/java/io/getstream/chat/java/ModerationTest.java Updates moderation config test to use a variable, ensuring consistency with the configuration.
src/test/java/io/getstream/chat/java/BasicTest.java Introduces cleanUsers() to delete users before tests, supporting a fresh test environment.
Comments suppressed due to low confidence (1)

src/test/java/io/getstream/chat/java/ModerationTest.java:15

  • [nitpick] Typo in the method name: 'Gettting' should be corrected to 'Getting' for clarity.
void whenUpsertingGetttingDeletingModerationConfig_thenNoException() {

@shaljam shaljam merged commit 65ea0c8 into main Apr 1, 2025
1 check passed
@shaljam shaljam deleted the fix-blocklist-test branch April 1, 2025 13:33
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