Fix use of TimeGraphArrow sourceId and destinationId

The sourceId and destinationId should be the row id of source row and
destination row. But it is used as the index of the source and
destination rows in the full list of rows.

Change to use the sourceId and destinationId properly. This requires
that the TimeGraphChartArrows layer receive the full list of row ids.

Enable unit tests in build pipeline.

Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>

.github/workflows/ci-cd.yml

@@ -39,6 +39,11 @@ jobs:
           NODE_OPTIONS: "--max_old_space_size=4096"
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # https://github.com/microsoft/vscode-ripgrep/issues/9
 
+      - name: Test
+        shell: bash
+        run: |
+          yarn test
+
   publish:
     needs: build
     if: github.ref == 'refs/heads/master' && github.event_name == 'push' && github.repository == 'eclipse-cdt-cloud/timeline-chart'

example/src/index.ts

@@ -175,7 +175,7 @@ timeGraphChart.registerMouseInteractions({
     }
 });
 
-timeGraphChartArrows.addArrows(timeGraph.arrows);
+timeGraphChartArrows.addArrows(timeGraph.arrows, testDataProvider.getRowIds());
 timeGraphChartRangeEvents.addRangeEvents(timeGraph.rangeEvents);
 
 const cursorReset = document.getElementById('cursor-reset');

example/src/test-data-provider.ts

@@ -181,6 +181,14 @@ export class TestDataProvider {
         })
     }
 
+    getRowIds(): number[] {
+        const rowIds: number[] = [];
+        timeGraphEntries.model.entries.forEach(entry => {
+            rowIds.push(entry.id);
+        });
+        return rowIds;
+    }
+
     getData(opts: { range?: TimelineChart.TimeGraphRange, resolution?: number }): TimelineChart.TimeGraphModel {
         const rows: TimelineChart.TimeGraphRowModel[] = [];
         const range = opts.range || { start: BigInt(0), end: this.totalLength };