fix: Support single-key configuration panels correctly

dploeger · dploeger · commit 8dc87d013d72 · 2022-10-17T16:47:10.000+02:00
diff --git a/lib/api/Configuration.ts b/lib/api/Configuration.ts
@@ -117,7 +117,7 @@ export class Configuration {
         if (lowerCase) {
           configurationKey = configurationKey.toLowerCase()
         }
-        rowObject[configurationKey] = $(values[index + i * 2]).text()
+        rowObject[configurationKey] = $(values[index + i * keys.length]).text()
       })
       returnObject.push(rowObject)
     }
diff --git a/test/ConfigurationTest.ts b/test/ConfigurationTest.ts
@@ -22,7 +22,7 @@ describe('The Configuration API', (): void => {
     chai.expect(configuration.maintainer).to.have.lengthOf(1)
 
     chai.expect(configuration.exceptions).to.have.lengthOf(1)
-    chai.expect(configuration.excludedLabels).to.have.lengthOf(1)
+    chai.expect(configuration.excludedLabels).to.have.lengthOf(2)
 
     chai.expect(configuration.notificationBodyTemplate).to.eq(MockServer.NOTIFICATION_BODY)
     chai.expect(configuration.notificationSubjectTemplate).to.eq(MockServer.NOTIFICATION_SUBJECT)
diff --git a/test/MockServer.ts b/test/MockServer.ts
@@ -155,6 +155,9 @@ export class MockServer {
             </tr>
             <tr>
                 <td>NOT</td>
+            </tr>       
+            <tr>
+                <td>NOT2</td>
             </tr>
             </tbody>
         </table>

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ export class Configuration {`
`117`	`117`	`if (lowerCase) {`
`118`	`118`	`configurationKey = configurationKey.toLowerCase()`
`119`	`119`	`}`
`120`		`- rowObject[configurationKey] = $(values[index + i * 2]).text()`
	`120`	`+ rowObject[configurationKey] = $(values[index + i * keys.length]).text()`
`121`	`121`	`})`
`122`	`122`	`returnObject.push(rowObject)`
`123`	`123`	`}`