Fix: ensure geopicker is loaded for non-nested setgeopoint actions

Author: eyelidlessness

src/widget/geo/geopicker.js

@@ -77,7 +77,7 @@ class Geopicker extends Widget {
      * @type {string}
      */
     static get selector() {
-        return ':is(.question input[data-type-xml="geopoint"], .question input[data-type-xml="geotrace"], .question input[data-type-xml="geoshape"]):not([data-setvalue], [data-setgeopoint])';
+        return '.question [name]:not([name=""], [type="hidden"]):is(input[data-type-xml="geopoint"], input[data-type-xml="geotrace"], input[data-type-xml="geoshape"])';
     }
 
     /**

test/spec/widget.geo.spec.js

@@ -132,4 +132,28 @@ describe('geoshape widget', () => {
 
         expect(geopickers.length).to.equal(0);
     });
+
+    it('loads the widget when a setgeopoint odk-instance-first-load action is defined adjacent to the input', () => {
+        const form = loadForm('setgeopoint.xml');
+
+        form.init();
+
+        const widget = form.view.html.querySelector(
+            '[name="/data/visible_first_load_adjacent_action"] ~ .geopicker'
+        );
+
+        expect(widget).not.to.be.null;
+    });
+
+    it('loads the widget when a setgeopoint odk-instance-first-load action is defined in the model', () => {
+        const form = loadForm('setgeopoint.xml');
+
+        form.init();
+
+        const widget = form.view.html.querySelector(
+            '[name="/data/visible_first_load_model_action"] ~ .geopicker'
+        );
+
+        expect(widget).not.to.be.null;
+    });
 });