fix: propagate EditableGeoJsonLayer accessor updateTriggers to GeoJsonLayer (#446)

* fix: propagate EditableGeoJsonLayer accessor updateTriggers to GeoJsonLayer

* test: trigger propagation

Author: timnyborg

modules/editable-layers/src/editable-layers/editable-geojson-layer.ts

@@ -332,10 +332,10 @@ export class EditableGeoJsonLayer extends EditableLayer<
       },
 
       updateTriggers: {
-        getLineColor: [this.props.selectedFeatureIndexes, this.props.mode],
-        getFillColor: [this.props.selectedFeatureIndexes, this.props.mode],
-        getPointRadius: [this.props.selectedFeatureIndexes, this.props.mode],
-        getLineWidth: [this.props.selectedFeatureIndexes, this.props.mode]
+        getLineColor: [this.props.updateTriggers.getLineColor, this.props.selectedFeatureIndexes, this.props.mode],
+        getFillColor: [this.props.updateTriggers.getFillColor, this.props.selectedFeatureIndexes, this.props.mode],
+        getPointRadius: [this.props.updateTriggers.getPointRadius, this.props.selectedFeatureIndexes, this.props.mode],
+        getLineWidth: [this.props.updateTriggers.getLineWidth, this.props.selectedFeatureIndexes, this.props.mode]
       }
     });
 

modules/editable-layers/test/editable-layers/editable-geojson-layer.spec.ts

@@ -0,0 +1,28 @@
+import {test, expect} from 'vitest';
+import {EditableGeoJsonLayer} from '../../src/editable-layers/editable-geojson-layer';
+import {DrawPointMode} from '../../src/edit-modes/draw-point-mode'
+
+test('Propagates update triggers to geojson layer', () =>{
+  const editableLayer = new EditableGeoJsonLayer({
+    id: 'test',
+    data: null,
+    mode: DrawPointMode,
+    selectedFeatureIndexes: [],
+    updateTriggers: {
+      getLineColor: ['lineColor'],
+      getFillColor: ['fillColor'],
+      getPointRadius: ['radius'],
+      getLineWidth: ['width'],
+    }
+  });
+  // Avoid the need for deck.gl-initialized state
+  editableLayer.createGuidesLayers = () => [];
+  editableLayer.createTooltipsLayers = () => [];
+
+  const [geoJsonLayer, ..._rest] = editableLayer.renderLayers();
+  const {updateTriggers} = geoJsonLayer.props;
+  expect(updateTriggers.getLineColor.flat()).toContain('lineColor')
+  expect(updateTriggers.getFillColor.flat()).toContain('fillColor')
+  expect(updateTriggers.getPointRadius.flat()).toContain('radius')
+  expect(updateTriggers.getLineWidth.flat()).toContain('width')
+})