Skip to content

Commit

Permalink
Fixed message for date rounding error and updated tests to use GMT si…
Browse files Browse the repository at this point in the history
…nce that timezone actually works with the API.
  • Loading branch information
John Dorlus committed Jun 29, 2021
1 parent eef613d commit 92a720a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -54,10 +54,10 @@ describe('Processor: Date Index', () => {
// Click submit button with only the type defined
await saveNewProcessor();

// Expect form error as "field" and "value" are required parameters
// Expect form error as "field" and "date rounding" are required parameters
expect(form.getErrorsMessages()).toEqual([
'A field value is required.',
'A field value is required.',
'A date rounding value is required.',
]);
});

Expand Down Expand Up @@ -104,7 +104,7 @@ describe('Processor: Date Index', () => {
});
component.update();

form.setInputValue('timezoneField.input', 'EST');
form.setInputValue('timezoneField.input', 'GMT');

form.setInputValue('localeField.input', 'SPANISH');
// Save the field with new changes
Expand All @@ -118,7 +118,7 @@ describe('Processor: Date Index', () => {
index_name_prefix: 'prefix',
date_formats: ['ISO8601'],
locale: 'SPANISH',
timezone: 'EST',
timezone: 'GMT',
});
});
});
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ const fieldsConfig: FieldsConfig = {
i18n.translate(
'xpack.ingestPipelines.pipelineEditor.dateIndexNameForm.dateRoundingRequiredError',
{
defaultMessage: 'A field value is required.',
defaultMessage: 'A date rounding value is required.',
}
)
),
Expand Down

0 comments on commit 92a720a

Please sign in to comment.