Prevent ID being set on list-based components temporarily

Author: alexluckett

src/server/plugins/engine/components/ListFormComponent.ts

@@ -125,6 +125,10 @@ export class ListFormComponent extends FormComponent {
       const selected = values.includes(item.value)
       const itemModel: ListItem = { ...item, selected }
 
+      if ('id' in itemModel) {
+        delete itemModel.id
+      }
+
       if (item.description) {
         itemModel.hint = {
           text: item.description

src/server/plugins/engine/pageControllers/QuestionPageController.test.ts

@@ -443,15 +443,13 @@ describe('QuestionPageController', () => {
       expect(filtered[1].model.label?.text).toBe('Select from the list')
       expect(filtered[1].model.items).toEqual([
         {
-          id: expect.any(String),
           checked: false,
           condition: 'isBarnOwl',
           selected: false,
           text: 'Option 1',
           value: '1'
         },
         {
-          id: expect.any(String),
           checked: false,
           condition: 'isBarnOwl',
           selected: false,
@@ -501,15 +499,13 @@ describe('QuestionPageController', () => {
       expect(filtered[1].model.label?.text).toBe('Select from the list')
       expect(filtered[1].model.items).toEqual([
         {
-          id: expect.any(String),
           checked: false,
           condition: 'notBarnOwl',
           selected: false,
           text: 'Option 3',
           value: '3'
         },
         {
-          id: expect.any(String),
           checked: false,
           condition: 'notBarnOwl',
           selected: false,