fix: update label and help text in connection and table group forms

Author: aarthy-dk

testgen/ui/components/frontend/js/components/alert.js

@@ -15,6 +15,7 @@ const { div } = van.tags;
 const alertTypeColors = {
     info: {backgroundColor: 'rgba(28, 131, 225, 0.1)', color: 'rgb(0, 66, 128)'},
     success: {backgroundColor: 'rgba(33, 195, 84, 0.1)', color: 'rgb(23, 114, 51)'},
+    warn: {backgroundColor: 'rgba(255, 227, 18, 0.2)', color: 'rgb(255, 255, 194)'},
     error: {backgroundColor: 'rgba(255, 43, 43, 0.09)', color: 'rgb(125, 53, 59)'},
 };
 

testgen/ui/components/frontend/js/components/connection_form.js

@@ -217,12 +217,12 @@ const ConnectionForm = (props, saveButton) => {
             div(
                 { class: 'flex-column fx-gap-3', style: 'flex: 2' },
                 Select({
-                    label: 'SQL Flavor',
+                    label: 'Database Type',
                     value: connectionFlavor,
                     options: props.flavors,
                     disabled: props.disableFlavor,
                     height: 38,
-                    help: 'The type of database server that you will connect to. This determines TestGen\'s drivers and SQL dialect.',
+                    help: 'Type of database server to connect to. This determines the database driver and SQL dialect that will be used by TestGen.',
                     testId: 'sql_flavor',
                     onChange: (value) => connectionFlavor.val = value,
                 }),
@@ -231,7 +231,7 @@ const ConnectionForm = (props, saveButton) => {
                     label: 'Connection Name',
                     value: connectionName,
                     height: 38,
-                    help: 'Your name for this connection. Can be any text.',
+                    help: 'Unique name to describe the connection',
                     onChange: (value, state) => {
                         connectionName.val = value;
                         setFieldValidity('connection_name', state.valid);
@@ -272,7 +272,6 @@ const ConnectionForm = (props, saveButton) => {
                     label: 'Database',
                     value: connectionDatabase,
                     height: 38,
-                    help: 'The name of the database defined on your host where your schemas and tables is present.',
                     disabled: connectByUrl,
                     onChange: (value, state) => {
                         connectionDatabase.val = value;
@@ -282,10 +281,9 @@ const ConnectionForm = (props, saveButton) => {
                 }),
                 Input({
                     name: 'db_user',
-                    label: 'User',
+                    label: 'Username',
                     value: connectionUsername,
                     height: 38,
-                    help: 'Username to connect to your database.',
                     onChange: (value, state) => {
                         connectionUsername.val = value;
                         setFieldValidity('db_user', state.valid);
@@ -318,7 +316,7 @@ const ConnectionForm = (props, saveButton) => {
         Checkbox({
             name: 'connect_by_url',
             label: 'URL Override',
-            help: 'If this switch is set to on, the connection string will be driven by the field below. Only user name and password will be passed per the relevant fields above.',
+            help: 'When checked, the connection string will be driven by the field below, along with the username and password from the fields above',
             checked: connectByUrl.val,
             onChange: (checked) => connectByUrl.val = checked,
         }),
@@ -344,7 +342,6 @@ const ConnectionForm = (props, saveButton) => {
                 }),
                 Input({
                     label: 'URL Suffix',
-                    help: `Provide a connection string directly. This will override connection parameters if the 'Connect by URL' switch is set.`,
                     value: connectionStringSuffix,
                     class: 'fx-flex',
                     height: 38,
@@ -392,7 +389,6 @@ const PasswordConnectionForm = (password, onValueChange, useSecretsPlaceholder)
                 value: password,
                 height: 38,
                 type: 'password',
-                help: 'Password to connect to your database.',
                 placeholder: useSecretsPlaceholder ? secretsPlaceholder : '',
                 onChange: onValueChange,
             }),
@@ -429,7 +425,6 @@ const HttpPathConnectionForm = (
                 value: password,
                 height: 38,
                 type: 'password',
-                help: 'Password to connect to your database.',
                 placeholder: useSecretsPlaceholder ? secretsPlaceholder : '',
                 onChange: (value, state) => passwordFieldState.val = {value, valid: state.valid},
             }),
@@ -507,7 +502,7 @@ const KeyPairConnectionForm = (
                         value: privateKeyPhrase,
                         height: 38,
                         type: 'password',
-                        help: 'Passphrase used while creating the private Key (leave empty if not applicable)',
+                        help: 'Passphrase used when creating the private key. Leave empty if the private key is not encrypted.',
                         placeholder: useSecretsPlaceholder ? secretsPlaceholder : '',
                         onChange: (value, state) => privateKeyPhraseFieldState.val = {value, valid: state.valid},
                     }),
@@ -529,7 +524,6 @@ const KeyPairConnectionForm = (
                 value: password,
                 height: 38,
                 type: 'password',
-                help: 'Password to connect to your database.',
                 placeholder: useSecretsPlaceholder ? secretsPlaceholder : '',
                 onChange: (value, state) => passwordFieldState.val = {value, valid: state.valid},
             });

testgen/ui/components/frontend/js/components/table_group_form.js

@@ -173,7 +173,7 @@ const MainForm = (
             label: 'Name',
             value: tableGroupsName,
             height: 38,
-            help: 'A unique name to describe the table group',
+            help: 'Unique name to describe the table group',
             helpPlacement: 'bottom-right',
             onChange: (value, state) => {
                 tableGroupsName.val = value;
@@ -186,7 +186,7 @@ const MainForm = (
             label: 'Tables to Include Mask',
             value: profilingIncludeMask,
             height: 38,
-            help: 'A SQL filter supported by your database\'s LIKE operator for table names to include',
+            help: 'SQL filter supported by your database\'s LIKE operator for table names to include',
             onChange: (value, state) => {
                 profilingIncludeMask.val = value;
                 options.setValidity?.('profiling_include_mask', state.valid);
@@ -197,7 +197,7 @@ const MainForm = (
             label: 'Tables to Exclude Mask',
             value: profilingExcludeMask,
             height: 38,
-            help: 'A SQL filter supported by your database\'s LIKE operator for table names to exclude',
+            help: 'SQL filter supported by your database\'s LIKE operator for table names to exclude',
             onChange: (value, state) => {
                 profilingExcludeMask.val = value;
                 options.setValidity?.('profiling_exclude_mask', state.valid);
@@ -208,7 +208,7 @@ const MainForm = (
             label: 'Explicit Table List',
             value: profilingTableSet,
             height: 38,
-            help: 'A list of specific table names to include, separated by commas',
+            help: 'List of specific table names to include, separated by commas',
             onChange: (value, state) => {
                 profilingTableSet.val = value;
                 options.setValidity?.('profiling_table_set', state.valid);
@@ -225,7 +225,7 @@ const MainForm = (
             label: 'Schema',
             value: tableGroupSchema,
             height: 38,
-            help: 'The database schema containing the tables in the Table Group',
+            help: 'Database schema containing the tables for the Table Group',
             helpPlacement: 'bottom-left',
             onChange: (value, state) => {
                 tableGroupSchema.val = value;
@@ -235,21 +235,21 @@ const MainForm = (
         }),
         Input({
             name: 'profile_id_column_mask',
-            label: 'Profiling ID column mask',
+            label: 'Profiling ID Column Mask',
             value: profileIdColumnMask,
             height: 38,
-            help: 'A SQL filter supported by your database\'s LIKE operator representing ID columns (optional)',
+            help: 'SQL filter supported by your database\'s LIKE operator representing ID columns (optional)',
             onChange: (value, state) => {
                 profileIdColumnMask.val = value;
                 options.setValidity?.('profile_id_column_mask', state.valid);
             },
         }),
         Input({
             name: 'profile_sk_column_mask',
-            label: 'Profiling Surrogate Key column mask',
+            label: 'Profiling Surrogate Key Column Mask',
             value: profileSkColumnMask,
             height: 38,
-            help: 'A SQL filter supported by your database\'s LIKE operator representing surrogate key columns (optional)',
+            help: 'SQL filter supported by your database\'s LIKE operator representing surrogate key columns (optional)',
             onChange: (value, state) => {
                 profileSkColumnMask.val = value
                 options.setValidity?.('profile_sk_column_mask', state.valid);
@@ -258,10 +258,10 @@ const MainForm = (
         Input({
             name: 'profiling_delay_days',
             type: 'number',
-            label: 'Min Profiling Age, Days',
+            label: 'Min Profiling Age (in days)',
             value: profilingDelayDays,
             height: 38,
-            help: 'The number of days to wait before new profiling will be available to generate tests',
+            help: 'Number of days to wait before new profiling will be available to generate tests',
             onChange: (value, state) => {
                 profilingDelayDays.val = value;
                 options.setValidity?.('profiling_delay_days', state.valid);
@@ -288,7 +288,7 @@ const SamplingForm = (
         Checkbox({
             name: 'profile_use_sampling',
             label: 'Use profile sampling',
-            help: 'Toggle on to base profiling on a sample of records instead of the full table',
+            help: 'When checked, profiling will be based on a sample of records instead of the full table',
             checked: profileUseSampling,
             onChange: (value) => profileUseSampling.val = value,
         }),
@@ -303,7 +303,7 @@ const SamplingForm = (
                     label: 'Sample percent',
                     value: profileSamplePercent,
                     height: 38,
-                    help: 'Percent of records to include in the sample, unless the calculated count falls below the specified minimum.',
+                    help: 'Percent of records to include in the sample, unless the calculated count falls below the specified minimum',
                     onChange: (value, state) => {
                         profileSamplePercent.val = value;
                         options.setValidity?.('profile_sample_percent', state.valid);
@@ -316,7 +316,7 @@ const SamplingForm = (
                     label: 'Min Sample Record Count',
                     value: profileSampleMinCount,
                     height: 38,
-                    help: 'The minimum number of records to be included in any sample (if available)',
+                    help: 'Minimum number of records to be included in any sample (if available)',
                     onChange: (value, state) => {
                         profileSampleMinCount.val = value;
                         options.setValidity?.('profile_sample_min_count', state.valid);