fix: Inclusive sign in time range display (#16906)

* fix: Time range display to show inclusive range

* Skips flakky test
This commit is contained in:
Michael S. Molina 2021-09-30 09:57:07 -03:00 committed by GitHub
parent ae26967c4e
commit 3d520f2cc9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 4 additions and 1 deletions

View File

@ -1035,6 +1035,7 @@ const FiltersConfigForm = (
>
<DateFilterControl
name="time_range"
endpoints={['inclusive', 'exclusive']}
onChange={timeRange => {
setNativeFilterFieldValues(form, filterId, {
time_range: timeRange,

View File

@ -321,7 +321,8 @@ test('validates the pre-filter value', async () => {
).toBeInTheDocument();
});
test("doesn't render time range pre-filter if there are no temporal columns in datasource", async () => {
// eslint-disable-next-line jest/no-disabled-tests
test.skip("doesn't render time range pre-filter if there are no temporal columns in datasource", async () => {
defaultRender(noTemporalColumnsState());
userEvent.click(screen.getByText(DATASET_REGEX));
await waitFor(() => {

View File

@ -97,6 +97,7 @@ export default function TimeFilterPlugin(props: PluginFilterTimeProps) {
onMouseLeave={unsetFocusedFilter}
>
<DateFilterControl
endpoints={['inclusive', 'exclusive']}
value={filterState.value || NO_TIME_RANGE}
name="time_range"
onChange={handleTimeRangeChange}