Skip to content

Commit b91b6e1

Browse files
fix(prevent): Remove testgen from the static onboarding page (#103114)
Per product request, remove testgen from the places in the static onboarding page Screenshots (new) <img width="1326" height="1057" alt="Screenshot 2025-11-10 at 3 57 13 PM" src="https://github.com/user-attachments/assets/051b8b2a-c10e-402c-84ea-bd7d485d72f4" /> <img width="1350" height="1155" alt="Screenshot 2025-11-10 at 3 57 45 PM" src="https://github.com/user-attachments/assets/a2bc7e7d-4207-431c-a3e2-cf1c5b185b01" /> <img width="1123" height="742" alt="Screenshot 2025-11-10 at 3 54 59 PM" src="https://github.com/user-attachments/assets/5ac134b3-c73a-4e5a-9e47-0c4ea062bc4b" /> Existing (red arrows for things changed/removed) <img width="1349" height="1078" alt="Screenshot 2025-11-10 at 1 58 45 PM" src="https://github.com/user-attachments/assets/649dbb57-a618-4845-bd5d-eebaef6bfe28" /> <img width="1039" height="803" alt="Screenshot 2025-11-10 at 3 55 22 PM" src="https://github.com/user-attachments/assets/03f373a9-259e-49cb-a647-ce105fb0a1c4" />
1 parent d6e169c commit b91b6e1

File tree

6 files changed

+91
-156
lines changed

6 files changed

+91
-156
lines changed

static/app/views/prevent/preventAI/onboarding.spec.tsx

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ describe('PreventAIOnboarding', () => {
4949

5050
expect(
5151
screen.getByRole('heading', {
52-
name: 'Ship Code That Breaks Less With Code Reviews And Tests',
52+
name: 'Ship Code That Breaks Less With Code Reviews',
5353
})
5454
).toBeInTheDocument();
5555

@@ -148,17 +148,13 @@ describe('PreventAIOnboarding', () => {
148148
'It reviews your pull requests, predicting errors and suggesting code fixes.'
149149
)
150150
).toBeInTheDocument();
151-
152-
expect(
153-
screen.getByText('It generates unit tests for untested code in your PR.')
154-
).toBeInTheDocument();
155151
});
156152

157153
it('renders how to use feature descriptions', () => {
158154
render(<PreventAIOnboarding />, {organization});
159155

160156
expect(
161-
screen.getByText('AI Code Review helps you ship better code with three features:')
157+
screen.getByText('AI Code Review helps you ship better code with new features:')
162158
).toBeInTheDocument();
163159

164160
expect(
@@ -176,14 +172,6 @@ describe('PreventAIOnboarding', () => {
176172
)
177173
)
178174
).toBeInTheDocument();
179-
180-
expect(
181-
screen.getByText(
182-
textWithMarkupMatcher(
183-
'It generates unit tests for your PR when you prompt @sentry generate-test.'
184-
)
185-
)
186-
).toBeInTheDocument();
187175
});
188176

189177
it('renders images with correct alt text', () => {

static/app/views/prevent/preventAI/onboarding.tsx

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ export function FeatureOverview() {
4646
{t('How to use AI Code Review')}
4747
</Text>
4848
<Text variant="muted" size="md">
49-
{t('AI Code Review helps you ship better code with three features:')}
49+
{t('AI Code Review helps you ship better code with new features:')}
5050
</Text>
5151
<Container as="ul" style={{margin: 0, fontSize: '12px'}}>
5252
<li>
@@ -77,17 +77,6 @@ export function FeatureOverview() {
7777
)}
7878
</Text>
7979
</li>
80-
<li>
81-
<Text variant="muted" size="sm">
82-
{tct('It generates unit tests for your PR when you prompt [sentryCommand].', {
83-
sentryCommand: (
84-
<Text variant="accent" size="sm" bold>
85-
@sentry generate-test
86-
</Text>
87-
),
88-
})}
89-
</Text>
90-
</li>
9180
</Container>
9281
<Text variant="muted" size="xs">
9382
{tct(
@@ -143,7 +132,7 @@ export default function PreventAIOnboarding() {
143132
<StyledImg src={preventHero} alt="AI Code Review Hero" />
144133
<Flex direction="column" gap="md" maxWidth="500px" padding="2xl 0">
145134
<Heading as="h1" style={{maxWidth: '400px'}}>
146-
{t('Ship Code That Breaks Less With Code Reviews And Tests')}
135+
{t('Ship Code That Breaks Less With Code Reviews')}
147136
</Heading>
148137
<Text variant="primary" size="md">
149138
{t('AI Code Review is an AI agent that automates tasks in your PR:')}
@@ -154,9 +143,6 @@ export default function PreventAIOnboarding() {
154143
'It reviews your pull requests, predicting errors and suggesting code fixes.'
155144
)}
156145
</Container>
157-
<Container as="li">
158-
{t('It generates unit tests for untested code in your PR.')}
159-
</Container>
160146
</Container>
161147
</Flex>
162148
</Flex>

static/app/views/settings/organizationGeneralSettings/organizationSettingsForm.spec.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@ describe('OrganizationSettingsForm', () => {
264264
expect(screen.getByText('beta')).toBeInTheDocument();
265265

266266
expect(
267-
screen.getByText('Use AI to review, find bugs, and generate tests in pull requests')
267+
screen.getByText('Use AI to review and find bugs in pull requests')
268268
).toBeInTheDocument();
269269

270270
const learnMoreLink = screen.getByRole('link', {name: 'Learn more'});
@@ -304,7 +304,7 @@ describe('OrganizationSettingsForm', () => {
304304

305305
expect(screen.getByText('Enable AI Code Review')).toBeInTheDocument();
306306
expect(
307-
screen.getByText('Use AI to review, find bugs, and generate tests in pull requests')
307+
screen.getByText('Use AI to review and find bugs in pull requests')
308308
).toBeInTheDocument();
309309
});
310310

static/app/views/settings/organizationGeneralSettings/preventAiSettings.tsx

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -43,14 +43,11 @@ export const makePreventAiField = (organization: Organization): FieldObject => {
4343
)}
4444
</Flex>
4545
),
46-
help: tct(
47-
'Use AI to review, find bugs, and generate tests in pull requests [link:Learn more]',
48-
{
49-
link: (
50-
<ExternalLink href="https://docs.sentry.io/product/ai-in-sentry/ai-code-review/" />
51-
),
52-
}
53-
),
46+
help: tct('Use AI to review and find bugs in pull requests [link:Learn more]', {
47+
link: (
48+
<ExternalLink href="https://docs.sentry.io/product/ai-in-sentry/ai-code-review/" />
49+
),
50+
}),
5451
visible: ({model}) => {
5552
// Show field when AI features are enabled (hideAiFeatures is false)
5653
const hideAiFeatures = model.getValue('hideAiFeatures');

static/images/features/prevent-pr-comments-dark.svg

Lines changed: 40 additions & 58 deletions
Loading

static/images/features/prevent-pr-comments-light.svg

Lines changed: 40 additions & 58 deletions
Loading

0 commit comments

Comments
 (0)