fix: prevent stream render if dataType unknown (#448)

awslabs · Jan 9, 2023 · 16999db · 16999db
1 parent ffaa88a
commit 16999db
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/...ges/components/src/components/iot-time-series-connector/iot-time-series-connector.spec.ts b/...ges/components/src/components/iot-time-series-connector/iot-time-series-connector.spec.ts
@@ -152,9 +152,6 @@ it('provides data streams', async () => {
         expect.objectContaining({
           id: DATA_STREAM.id,
         }),
-        expect.objectContaining({
-          id: DATA_STREAM_2.id,
-        }),
       ]),
       viewport,
     })
@@ -295,9 +292,6 @@ it('updates with new queries', async () => {
         expect.objectContaining({
           id: DATA_STREAM.id,
         }),
-        expect.objectContaining({
-          id: DATA_STREAM_2.id,
-        }),
       ]),
       viewport,
     })

diff --git a/packages/components/src/components/iot-time-series-connector/iot-time-series-connector.tsx b/packages/components/src/components/iot-time-series-connector/iot-time-series-connector.tsx
@@ -79,7 +79,8 @@ export class IotTimeSeriesConnector {
       styleSettings: this.styleSettings,
       assignDefaultColors: this.assignDefaultColors || false,
     }).filter((stream) => {
-      if (!stream.dataType || stream.streamType === 'ALARM') return true;
+      if (stream.error || !stream.dataType) return false;
+      if (stream.streamType === 'ALARM') return true;
       return this.supportedDataTypes.includes(stream.dataType);
     });