Skip to content

Commit 6583f5e

Browse files
committed
docs: explain why moderation tests are conditional
Add comment explaining that APP_ID 6743 (default test app) can't have moderation enabled as it would break other existing tests. This is why we conditionally run moderation tests only on APP_ID 16792.
1 parent 15a2477 commit 6583f5e

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

test/integration/cloud/user_flag.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@ import config from '../utils/config';
55

66
// User flagging requires moderation features to be enabled on the Stream account.
77
// Only run these tests on accounts with moderation enabled (APP_ID 16792).
8+
//
9+
// Note: We can't enable moderation on the default test app (6743) because it would
10+
// break other existing tests that rely on moderation being disabled. Therefore,
11+
// we conditionally run these tests only on APP_ID 16792 which has moderation enabled.
812
const shouldRunModerationTests = config.APP_ID === '16792';
913
const describeOrSkip = shouldRunModerationTests ? describe : describe.skip;
1014

0 commit comments

Comments
 (0)