Skip to content

FEEDS-1195: Skip user flagging tests when moderation not enabled #635

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
itsmeadi merged 5 commits into from
Feb 11, 2026
Merged

FEEDS-1195: Skip user flagging tests when moderation not enabled #635

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
bcbae0c
test: skip client.flagUser() integration test
itsmeadi Feb 11, 2026
15a2477
test: conditionally run user flagging tests based on APP_ID
itsmeadi Feb 11, 2026
6583f5e
docs: explain why moderation tests are conditional
itsmeadi Feb 11, 2026
ae6f7cb
docs: simplify moderation test comment
itsmeadi Feb 11, 2026
7fb34f6
docs: clarify why moderation tests are conditional
itsmeadi Feb 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 10 additions & 3 deletions test/integration/cloud/user_flag.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,20 @@
import expect from 'expect.js';

import { CloudContext } from './utils';
import config from '../utils/config';

// User flagging requires moderation features to be enabled on the Stream account.
// APP_ID 16792 has moderation enabled. Other test apps (e.g., from StreamAPI CI)
// have moderation disabled, so we skip these tests there.
const shouldRunModerationTests = config.APP_ID === '16792';
const describeOrSkip = shouldRunModerationTests ? describe : describe.skip;

describe('User Flagging', () => {
const ctx = new CloudContext();

ctx.createUsers();

describe('When creating activities to establish users in moderation system', () => {
describeOrSkip('When creating activities to establish users in moderation system', () => {
ctx.requestShouldNotError(async () => {
// Create activities with users as actors to establish them in the moderation system
// Using user1 and user2 which are commonly used across integration tests
Expand All @@ -26,7 +33,7 @@ describe('User Flagging', () => {
});
});

describe('When flagging a user with client.flagUser()', () => {
describeOrSkip('When flagging a user with client.flagUser()', () => {
ctx.requestShouldNotError(async () => {
// Flag user1 (which exists in the moderation system from other tests)
ctx.response = await ctx.serverSideClient.flagUser('user1', {
Expand All @@ -37,7 +44,7 @@ describe('User Flagging', () => {
});
});

describe('When flagging using user.flag()', () => {
describeOrSkip('When flagging using user.flag()', () => {
ctx.requestShouldNotError(async () => {
// Flag using the user object method
ctx.response = await ctx.serverSideClient.user('user1').flag({
Expand Down