From 06f1d16ca11301472b18043e485cd288a1045ced Mon Sep 17 00:00:00 2001
From: Nico Rehwaldt <git_nikku@nixis.de>
Date: Wed, 14 Dec 2022 22:59:17 +0100
Subject: [PATCH] fix: coherce undefined `<Select />` value to empty string

Explicitly set select value to empty rather than undefined.

Undefined values cause the browser to render the first selection.
---
 src/components/entries/Select.js    |  2 +-
 test/spec/components/Select.spec.js | 43 +++++++++++++++++++++++++++++
 2 files changed, 44 insertions(+), 1 deletion(-)

diff --git a/src/components/entries/Select.js b/src/components/entries/Select.js
index 65a423a9..d6de04d1 100644
--- a/src/components/entries/Select.js
+++ b/src/components/entries/Select.js
@@ -36,7 +36,7 @@ function Select(props) {
     label,
     onChange,
     options = [],
-    value,
+    value = '',
     disabled,
     onFocus,
     onBlur
diff --git a/test/spec/components/Select.spec.js b/test/spec/components/Select.spec.js
index 3f1f522e..1f0075d2 100644
--- a/test/spec/components/Select.spec.js
+++ b/test/spec/components/Select.spec.js
@@ -69,6 +69,49 @@ describe('<Select>', function() {
   });
 
 
+  describe('should select', function() {
+
+    const getOptions = () => [
+      {
+        label: 'A',
+        value: 'A'
+      },
+      {
+        label: 'B',
+        value: 'B'
+      }
+    ];
+
+
+    it('none (undefined value)', function() {
+
+      // when
+      const result = createSelect({ container, getOptions, getValue: () => undefined });
+
+      const selectInput = domQuery('.bio-properties-panel-input', result.container);
+
+      // then
+      expect(selectInput.value).to.equal('');
+    });
+
+
+    it('active entry', function() {
+
+      // when
+      const result = createSelect({ container, getOptions, getValue: () => 'A' });
+
+      const selectInput = domQuery('.bio-properties-panel-input', result.container);
+
+      const optionA = domQuery('option[value="A"]', selectInput);
+
+      // then
+      expect(selectInput.value).to.equal('A');
+      expect(optionA.selected).to.be.true;
+    });
+
+  });
+
+
   it('should render disabled', function() {
 
     // given