fix: trailing FQBN part if there are no configs

Signed-off-by: Akos Kitta <[email protected]>

Author: Akos Kitta

Diff for: arduino-ide-extension/src/common/protocol/boards-service.ts

@@ -574,7 +574,8 @@ export namespace ConfigOption {
     const configSuffix = Object.entries(mergedOptions)
       .map(([option, value]) => `${option}=${value}`)
       .join(',');
-    return `${vendor}:${arch}:${id}:${configSuffix}`;
+
+    return `${vendor}:${arch}:${id}${configSuffix ? `:${configSuffix}` : ''}`;
   }
 
   export class ConfigOptionError extends Error {

Diff for: arduino-ide-extension/src/test/common/boards-service.test.ts

@@ -104,6 +104,12 @@ describe('boards-service', () => {
         })
     );
 
+    it('should not append the trailing boards config part to FQBN if configs is empty', () => {
+      const fqbn = 'a:b:c';
+      const actual = ConfigOption.decorate(fqbn, []);
+      expect(actual).to.be.equal(fqbn);
+    });
+
     it('should be noop when config options is empty', () => {
       const fqbn = 'a:b:c:menu1=value1';
       const actual = ConfigOption.decorate(fqbn, []);