ensure all handles are treated the same (as lowercase)
This commit is contained in:
Родитель
5f97a67765
Коммит
a3d1539bb3
|
@ -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);
|
||||
|
|
|
@ -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}`,
|
||||
|
|
Различия файлов скрыты, потому что одна или несколько строк слишком длинны
|
@ -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}`,
|
||||
|
|
Загрузка…
Ссылка в новой задаче