Skip to content

Commit 4a2fdcb

Browse files
Dakshclaude
andcommitted
resolve comments: remove hardcoded metric count, simplify tests
- Remove "16 metrics" from javadoc (implementation detail) - Simplify TeamUsageStatsTest for non-multi-tenant app - Restore original client after TeamUsageStatsIntegrationTest Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 5a27792 commit 4a2fdcb

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/main/java/io/getstream/chat/java/models/TeamUsageStats.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@ public static class QueryTeamUsageStatsResponse extends StreamResponseObject {
212212
/**
213213
* Queries team-level usage statistics from the warehouse database.
214214
*
215-
* <p>Returns all 16 metrics grouped by team with cursor-based pagination.
215+
* <p>Returns usage metrics grouped by team with cursor-based pagination.
216216
*
217217
* <p>Date Range Options (mutually exclusive):
218218
*

src/test/java/io/getstream/chat/java/TeamUsageStatsTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,17 @@
11
package io.getstream.chat.java;
22

3-
import io.getstream.chat.java.models.TeamUsageStats.QueryTeamUsageStatsResponse;
43
import io.getstream.chat.java.models.TeamUsageStats;
4+
import io.getstream.chat.java.models.TeamUsageStats.QueryTeamUsageStatsResponse;
55
import java.time.LocalDate;
66
import java.time.format.DateTimeFormatter;
77
import org.junit.jupiter.api.Assertions;
88
import org.junit.jupiter.api.DisplayName;
99
import org.junit.jupiter.api.Test;
1010

1111
/**
12-
* Basic tests for Team Usage Stats API using regular (non-multi-tenant) app credentials.
13-
* Since the regular app doesn't have multi-tenant enabled, teams will always be empty.
14-
* Full data verification is done in TeamUsageStatsIntegrationTest with multi-tenant credentials.
12+
* Basic tests for Team Usage Stats API using regular (non-multi-tenant) app credentials. Since the
13+
* regular app doesn't have multi-tenant enabled, teams will always be empty. Full data verification
14+
* is done in TeamUsageStatsIntegrationTest with multi-tenant credentials.
1515
*/
1616
public class TeamUsageStatsTest {
1717

0 commit comments

Comments
 (0)