mirror of
https://github.com/mattermost/mattermost-mobile.git
synced 2026-11-12 22:20:14 +00:00
* Add AI-generated sparkle indicator to post headers for mobile/web parity. Users on mobile could not tell when a post was AI-generated; this surfaces the same sparkle marker as web using post props and keeps headers visible for consecutive AI posts. Co-authored-by: Cursor <cursoragent@cursor.com> * Make AI-generated indicator accessible to screen readers. CodeRabbit flagged that accessibilityLabel alone is not announced on a standard View; set accessible and accessibilityRole so VoiceOver/TalkBack can surface the AI attribution label. Co-authored-by: Cursor <cursoragent@cursor.com> * ci: retrigger docs-impact-review workflow. Prior runs failed due to Claude Code SDK initialization errors unrelated to PR changes; retry to clear the flaky check. Co-authored-by: Cursor <cursoragent@cursor.com> * Address review nits for AI-generated post indicator. Compute isAiGenerated once and only check previousPost in areConsecutivePosts. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Cursor <cursoragent@cursor.com> Co-authored-by: Mattermost Build <build@mattermost.com>
171 lines
5.2 KiB
TypeScript
171 lines
5.2 KiB
TypeScript
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
import {screen} from '@testing-library/react-native';
|
|
import React from 'react';
|
|
|
|
import CompassIcon from '@components/compass_icon';
|
|
import ExpiryCountdown from '@components/post_list/post/header/expiry_timer';
|
|
import {Screens} from '@constants';
|
|
import {PostTypes} from '@constants/post';
|
|
import {renderWithIntlAndTheme} from '@test/intl-test-helper';
|
|
import TestHelper from '@test/test_helper';
|
|
|
|
import Header from './header';
|
|
|
|
jest.mock('@components/compass_icon', () => ({
|
|
__esModule: true,
|
|
default: jest.fn(),
|
|
}));
|
|
jest.mocked(CompassIcon).mockImplementation(
|
|
(props) => React.createElement('CompassIcon', {testID: `compass-icon${props.name ? '-' + props.name : ''}`, ...props}) as any,
|
|
);
|
|
|
|
jest.mock('@components/post_list/post/header/expiry_timer', () => ({
|
|
__esModule: true,
|
|
default: jest.fn(),
|
|
}));
|
|
jest.mocked(ExpiryCountdown).mockImplementation(
|
|
(props) => React.createElement('ExpiryCountdown', {testID: 'expiry-timer', ...props}) as any,
|
|
);
|
|
|
|
describe('Header', () => {
|
|
const currentUser = TestHelper.fakeUserModel();
|
|
|
|
const defaultProps = {
|
|
commentCount: 0,
|
|
enablePostUsernameOverride: false,
|
|
isAiGenerated: false,
|
|
isAutoResponse: false,
|
|
isCustomStatusEnabled: false,
|
|
isEphemeral: false,
|
|
isMilitaryTime: false,
|
|
isPendingOrFailed: false,
|
|
isSystemPost: false,
|
|
isWebHook: false,
|
|
location: Screens.ABOUT,
|
|
showPostPriority: false,
|
|
teammateNameDisplay: '',
|
|
hideGuestTags: false,
|
|
currentUser,
|
|
isChannelAutotranslated: false,
|
|
};
|
|
|
|
it('Should show BoR icon for own BoR post', () => {
|
|
const ownBoRPost = TestHelper.fakePostModel({
|
|
type: PostTypes.BURN_ON_READ,
|
|
userId: currentUser.id,
|
|
metadata: {
|
|
|
|
// missing expire_at key indicates unrevealed BoR post
|
|
},
|
|
});
|
|
|
|
renderWithIntlAndTheme(
|
|
<Header
|
|
{...defaultProps}
|
|
post={ownBoRPost}
|
|
/>,
|
|
);
|
|
|
|
expect(screen.queryByTestId('compass-icon-fire')).toBeVisible();
|
|
expect(screen.queryByTestId('expiry-timer')).not.toBeVisible();
|
|
});
|
|
|
|
it('Should show BoR countdown for revealed BoR post', () => {
|
|
const ownBoRPost = TestHelper.fakePostModel({
|
|
type: PostTypes.BURN_ON_READ,
|
|
metadata: {
|
|
expire_at: Date.now() + 60000,
|
|
},
|
|
});
|
|
|
|
renderWithIntlAndTheme(
|
|
<Header
|
|
{...defaultProps}
|
|
post={ownBoRPost}
|
|
/>,
|
|
);
|
|
|
|
expect(screen.queryByTestId('expiry-timer')).toBeVisible();
|
|
});
|
|
|
|
it('should render AI-generated indicator when both props are present', () => {
|
|
const aiPost = TestHelper.fakePostModel({
|
|
userId: currentUser.id,
|
|
props: {
|
|
ai_generated_by: currentUser.id,
|
|
ai_generated_by_username: 'ai-bot',
|
|
},
|
|
});
|
|
|
|
renderWithIntlAndTheme(
|
|
<Header
|
|
{...defaultProps}
|
|
isAiGenerated={true}
|
|
post={aiPost}
|
|
/>,
|
|
);
|
|
|
|
expect(screen.queryByTestId('compass-icon-creation-outline')).toBeVisible();
|
|
expect(screen.queryByTestId('post_header.ai_generated_indicator')).toBeVisible();
|
|
});
|
|
|
|
it('should not render AI-generated indicator when props are missing', () => {
|
|
const post = TestHelper.fakePostModel({
|
|
userId: currentUser.id,
|
|
props: {},
|
|
});
|
|
|
|
renderWithIntlAndTheme(
|
|
<Header
|
|
{...defaultProps}
|
|
post={post}
|
|
/>,
|
|
);
|
|
|
|
expect(screen.queryByTestId('compass-icon-creation-outline')).toBeNull();
|
|
expect(screen.queryByTestId('post_header.ai_generated_indicator')).toBeNull();
|
|
});
|
|
|
|
it('should use AI-generated accessibility label when author generated the post', () => {
|
|
const aiPost = TestHelper.fakePostModel({
|
|
userId: currentUser.id,
|
|
props: {
|
|
ai_generated_by: currentUser.id,
|
|
ai_generated_by_username: 'ai-bot',
|
|
},
|
|
});
|
|
|
|
renderWithIntlAndTheme(
|
|
<Header
|
|
{...defaultProps}
|
|
isAiGenerated={true}
|
|
post={aiPost}
|
|
/>,
|
|
);
|
|
|
|
expect(screen.getByTestId('post_header.ai_generated_indicator')).toHaveProp('accessibilityLabel', 'AI-generated');
|
|
});
|
|
|
|
it('should use bot username in accessibility label when different from author', () => {
|
|
const aiPost = TestHelper.fakePostModel({
|
|
userId: currentUser.id,
|
|
props: {
|
|
ai_generated_by: 'other-bot-id',
|
|
ai_generated_by_username: 'other-bot',
|
|
},
|
|
});
|
|
|
|
renderWithIntlAndTheme(
|
|
<Header
|
|
{...defaultProps}
|
|
isAiGenerated={true}
|
|
post={aiPost}
|
|
/>,
|
|
);
|
|
|
|
expect(screen.getByTestId('post_header.ai_generated_indicator')).toHaveProp('accessibilityLabel', 'Message posted by @other-bot');
|
|
});
|
|
});
|