Skip to content

Commit 413cbbc

Browse files
author
Tim Roes
committed
Fix PR review
1 parent e39ef03 commit 413cbbc

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

x-pack/test/functional/apps/visualize/precalculated_histogram.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,7 @@ export default function({ getService, getPageObjects }: FtrProviderContext) {
3939
await PageObjects.visEditor.clickMetricEditor();
4040
});
4141

42-
const renderTableForAggregation = async (
43-
aggregation: string
44-
): Promise<string[][] | string[][][]> => {
42+
const renderTableForAggregation = async (aggregation: string) => {
4543
await PageObjects.visEditor.selectAggregation(aggregation, 'metrics');
4644
await PageObjects.visEditor.selectField('histogram-content', 'metrics');
4745
await PageObjects.visEditor.clickGo();
@@ -68,8 +66,8 @@ export default function({ getService, getPageObjects }: FtrProviderContext) {
6866
expect(data).to.eql([['2.8510720308359434']]);
6967
});
7068

71-
it('with mean aggregation', async () => {
72-
// Percentile values (which are used by media behind the scenes) are not deterministic,
69+
it('with median aggregation', async () => {
70+
// Percentile values (which are used by median behind the scenes) are not deterministic,
7371
// so we can't check for the exact values here, but just check they are all within the given range
7472
// see https://github.com/elastic/elasticsearch/issues/49225
7573
const data = await renderTableForAggregation('Median');

0 commit comments

Comments
 (0)