TimeSeries: assume values are all numbers (#18540)

* assume number for TimeSeries types

* use const

(cherry picked from commit 0ba07720df)
This commit is contained in:
Ryan McKinley
2019-08-15 11:08:12 +02:00
committed by Marcus Efraimsson
parent 39f00259f3
commit 9d97f48374
2 changed files with 10 additions and 0 deletions
@@ -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' }],
@@ -29,6 +29,7 @@ function convertTimeSeriesToDataFrame(timeSeries: TimeSeries): DataFrame {
fields: [
{
name: timeSeries.target || 'Value',
type: FieldType.number,
unit: timeSeries.unit,
},
{