fix(radio-group): change check of value to only null or undefined

Author: mblankenstein

packages/radio-group/src/LionRadioGroup.js

@@ -94,7 +94,7 @@ export class LionRadioGroup extends LionFieldset {
 
   __triggerCheckedValueChanged() {
     const value = this.checkedValue;
-    if (value && value !== this.__previousCheckedValue) {
+    if (value != null && value !== this.__previousCheckedValue) {
       this.dispatchEvent(
         new CustomEvent('checked-value-changed', { bubbles: true, composed: true }),
       );

packages/radio-group/test/lion-radio-group.test.js

@@ -36,6 +36,20 @@ describe('<lion-radio-group>', () => {
     expect(el.checkedValue).to.deep.equal({ some: 'data' });
   });
 
+  it('can handle 0 and empty string as valid values ', async () => {
+    const el = await fixture(html`
+      <lion-radio-group>
+        <lion-radio name="data[]" .choiceValue=${0} .choiceChecked=${true}></lion-radio>
+        <lion-radio name="data[]" .choiceValue=${''}></lion-radio>
+      </lion-radio-group>
+    `);
+    await nextFrame();
+
+    expect(el.checkedValue).to.equal(0);
+    el.formElementsArray[1].choiceChecked = true;
+    expect(el.checkedValue).to.equal('');
+  });
+
   it('still has a full modelValue ', async () => {
     const el = await fixture(html`
       <lion-radio-group>