Merge pull request #122 from github/case-bug

ensure all handles are treated the same (as lowercase)

__tests__/privileged-requester.test.js

@@ -31,6 +31,31 @@ requesters:
   });
 });
 
+test("We receive the expected config content with values converted to lowercase", async () => {
+  let configContent = `---
+requesters:
+  roBot:
+    labels:
+      - testing
+  RoBot_TwO:
+    labels:
+      - dependencies
+      - github_actions    
+    `;
+  let provider = new GitHubProvider("token");
+  jest
+    .spyOn(provider, "getConfigContent")
+    .mockImplementation(() => configContent);
+  expect(provider.getConfigContent()).toBe(configContent);
+
+  let privilegedRequester = new PrivilegedRequester(provider);
+  let requesters = await privilegedRequester.getRequesters();
+  expect(requesters).toStrictEqual({
+    robot: { labels: ["testing"] },
+    robot_two: { labels: ["dependencies", "github_actions"] },
+  });
+});
+
 test("We do not receive the expected config content", async () => {
   let provider = new GitHubProvider("token");
   jest.spyOn(provider, "getConfigContent").mockImplementation(() => false);

dist/index.js

@@ -37086,6 +37086,14 @@ class PrivilegedRequester {
 
         this.configContents = yaml.load(config);
         this.requesters = this.configContents["requesters"];
+
+        // set the key (which is the requester name) to lowercase
+        if (this.requesters) {
+          this.requesters = Object.keys(this.requesters).reduce((acc, key) => {
+            acc[key.toLowerCase()] = this.requesters[key];
+            return acc;
+          }, {});
+        }
       } catch (err) {
         lib_core.error(
           `There was a problem with the privileged requester configuration.\n${err}\n${err.stack}`,

src/privileged-requester.js

@@ -18,6 +18,14 @@ class PrivilegedRequester {
 
         this.configContents = yaml.load(config);
         this.requesters = this.configContents["requesters"];
+
+        // set the key (which is the requester name) to lowercase
+        if (this.requesters) {
+          this.requesters = Object.keys(this.requesters).reduce((acc, key) => {
+            acc[key.toLowerCase()] = this.requesters[key];
+            return acc;
+          }, {});
+        }
       } catch (err) {
         core.error(
           `There was a problem with the privileged requester configuration.\n${err}\n${err.stack}`,