fix(comment content): hide detailed publish errors

This commit is contained in:
Tommaso Casaburi
2026-05-01 15:18:16 +07:00
parent 0cbae87aef
commit f4f14583c6
2 changed files with 6 additions and 4 deletions
@@ -129,15 +129,16 @@ vi.mock('../../loading-ellipsis', () => ({
}));
vi.mock('../../error-display/error-display', () => ({
default: ({ error, inline, showImmediately }: { error?: Error; inline?: boolean; showImmediately?: boolean }) =>
default: ({ displayMessage, error, inline, showImmediately }: { displayMessage?: string; error?: Error; inline?: boolean; showImmediately?: boolean }) =>
createElement(
'button',
{
'data-display-message': displayMessage,
'data-testid': 'error-display',
'data-inline': String(Boolean(inline)),
'data-show-immediately': String(Boolean(showImmediately)),
},
error?.message || String(error),
displayMessage || error?.message || String(error),
),
}));
@@ -420,7 +421,8 @@ describe('CommentContent', () => {
});
const errorDisplay = container.querySelector('[data-testid="error-display"]');
expect(errorDisplay?.textContent).toBe('spam blocker server error');
expect(errorDisplay?.textContent).toBe('Error');
expect(errorDisplay?.getAttribute('data-display-message')).toBe('Error');
expect(errorDisplay?.getAttribute('data-inline')).toBe('true');
expect(errorDisplay?.getAttribute('data-show-immediately')).toBe('true');
expect(container.querySelector('[data-testid="loading-ellipsis"]')).toBeNull();
@@ -136,7 +136,7 @@ const CommentContent = ({ comment: post, prependContent }: { comment: Comment; p
const loadingString = (
<div className={styles.stateString}>
{failedError ? (
<ErrorDisplay error={failedError} inline={true} showImmediately={true} />
<ErrorDisplay error={failedError} displayMessage={capitalize(t('error'))} inline={true} showImmediately={true} />
) : !hasFailedState ? (
<LoadingEllipsis string={stateString || t('loading')} />
) : (