Skip to content

Commit cd077d1

Browse files
committed
fix(KFLUXBUGS-96): make the secret forms more clear
1 parent ed4ee3e commit cd077d1

File tree

4 files changed

+7
-8
lines changed

4 files changed

+7
-8
lines changed

src/components/Secrets/SecretForm.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ const SecretForm: React.FC<React.PropsWithChildren<SecretFormProps>> = ({ existi
7171
required
7272
key={values.type}
7373
name="secretName"
74-
label="Select or enter name"
74+
label="Select or enter secret name"
7575
helpText="Unique name of the new secret."
7676
isCreatable
7777
isInputValuePersisted

src/components/Secrets/SecretsForm/SecretTypeSubForm.tsx

+2-3
Original file line numberDiff line numberDiff line change
@@ -116,11 +116,10 @@ export const SecretTypeSubForm: React.FC<React.PropsWithChildren<unknown>> = ()
116116
{isPartnerTaskAvailable(currentTypeRef.current) ? (
117117
<SelectInputField
118118
name="name"
119-
label="Secret name"
120-
toggleAriaLabel="Secret name"
119+
label="Select or enter secret name"
120+
toggleAriaLabel="Select or enter secret name"
121121
helpText="Unique name of the new secret"
122122
toggleId="secret-name-toggle"
123-
placeholderText="Enter name"
124123
variant={SelectVariant.typeahead}
125124
options={options}
126125
isCreatable

src/components/Secrets/__tests___/AddSecretForm.spec.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ describe('AddSecretForm', () => {
4949

5050
await waitFor(() => {
5151
screen.getByText('Secret type');
52-
screen.getByText('Secret name');
52+
screen.getByText('Select or enter secret name');
5353
screen.getByText('Labels');
5454
});
5555
});

src/components/Secrets/__tests___/SecretTypeSubForm.spec.tsx

+3-3
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ describe('SecretTypeSubForm', () => {
4343

4444
it('should render Secret type sub form and fields', () => {
4545
expect(screen.getByText('Secret type')).toBeVisible();
46-
expect(screen.getByText('Secret name')).toBeVisible();
46+
expect(screen.getByText('Select or enter secret name')).toBeVisible();
4747
});
4848

4949
it('should render subforms correctly for specified targets', async () => {
@@ -57,8 +57,8 @@ describe('SecretTypeSubForm', () => {
5757
});
5858

5959
it('should render correct variant of name field', () => {
60-
expect(screen.getByRole('button', { name: 'Secret name' })).toBeVisible();
61-
fireEvent.click(screen.getByRole('button', { name: 'Secret name' }));
60+
expect(screen.getByRole('button', { name: 'Select or enter secret name' })).toBeVisible();
61+
fireEvent.click(screen.getByRole('button', { name: 'Select or enter secret name' }));
6262
expect(screen.getByText('snyk-secret')).toBeVisible();
6363
});
6464
});

0 commit comments

Comments
 (0)