From 0ba07720df9ee053c850dbb708af3a9f019d106d Mon Sep 17 00:00:00 2001 From: Ryan McKinley Date: Tue, 13 Aug 2019 23:07:21 -0700 Subject: [PATCH] TimeSeries: assume values are all numbers (#18540) * assume number for TimeSeries types * use const --- packages/grafana-data/src/utils/processDataFrame.test.ts | 9 +++++++++ packages/grafana-data/src/utils/processDataFrame.ts | 1 + 2 files changed, 10 insertions(+) diff --git a/packages/grafana-data/src/utils/processDataFrame.test.ts b/packages/grafana-data/src/utils/processDataFrame.test.ts index d4885daa8fb..68925e036ff 100644 --- a/packages/grafana-data/src/utils/processDataFrame.test.ts +++ b/packages/grafana-data/src/utils/processDataFrame.test.ts @@ -29,6 +29,15 @@ describe('toDataFrame', () => { expect(series.fields[0].name).toEqual('Value'); }); + it('assumes TimeSeries values are numbers', () => { + const input1 = { + target: 'time', + datapoints: [[100, 1], [200, 2]], + }; + const data = toDataFrame(input1); + expect(data.fields[0].type).toBe(FieldType.number); + }); + it('keeps dataFrame unchanged', () => { const input = { fields: [{ text: 'A' }, { text: 'B' }, { text: 'C' }], diff --git a/packages/grafana-data/src/utils/processDataFrame.ts b/packages/grafana-data/src/utils/processDataFrame.ts index 5c9a68cee9f..7e189503a92 100644 --- a/packages/grafana-data/src/utils/processDataFrame.ts +++ b/packages/grafana-data/src/utils/processDataFrame.ts @@ -29,6 +29,7 @@ function convertTimeSeriesToDataFrame(timeSeries: TimeSeries): DataFrame { fields: [ { name: timeSeries.target || 'Value', + type: FieldType.number, unit: timeSeries.unit, }, {