Fix collectEntries params being required and unique

Author: rubensworks

lib/serialize/ComponentConstructor.ts

@@ -358,6 +358,9 @@ export class ComponentConstructor {
         parameters: subParameters,
       },
     };
+    // Params for collected entries are never required, and can have more than one entry.
+    parameterData.unique = false;
+    parameterData.required = false;
     this.populateOptionalParameterFields(parameter, parameterData);
     parameters.push(parameter);
 

test/serialize/ComponentConstructor.test.ts

@@ -710,8 +710,6 @@ describe('ComponentConstructor', () => {
               },
             ],
           },
-          required: true,
-          unique: true,
         },
       ]);
     });
@@ -1063,8 +1061,6 @@ describe('ComponentConstructor', () => {
         {
           '@id': 'mp:a/b/file-param#MyClass_fieldA',
           comment: 'Hi',
-          required: true,
-          unique: true,
           range: {
             '@type': 'mp:a/b/file-param#MyClass_fieldA_range',
             parameters: [
@@ -1130,8 +1126,6 @@ describe('ComponentConstructor', () => {
         {
           '@id': 'mp:a/b/file-param#MyClass_fieldA',
           comment: 'Hi',
-          required: true,
-          unique: true,
           range: {
             '@type': 'mp:a/b/file-param#MyClass_fieldA_range',
             parameters: [
@@ -1315,7 +1309,6 @@ describe('ComponentConstructor', () => {
         {
           '@id': 'mp:a/b/file-param#MyClass_fieldA',
           comment: 'Hi',
-          required: true,
           range: {
             '@type': 'mp:a/b/file-param#MyClass_fieldA_range',
             parameters: [