github
/
privileged-requester
зеркало из https://github.com/github/privileged-requester.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

lint and format

This commit is contained in:
GrantBirki 2023-10-17 09:33:09 -06:00
Родитель 4af16e68f9
Коммит edab05ebfa
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 9029BBCCC15C6F8C
8 изменённых файлов: 38 добавлений и 66 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

4
__tests__/privileged-requester.test.js
Просмотреть файл

@ -33,9 +33,7 @@ requesters:
test("We do not receive the expected config content", async () => {
let provider = new GitHubProvider("token");
jest
.spyOn(provider, "getConfigContent")
.mockImplementation(() => false);
jest.spyOn(provider, "getConfigContent").mockImplementation(() => false);
expect(provider.getConfigContent()).toBe(false);
let privilegedRequester = new PrivilegedRequester(provider);

8
__tests__/pull-request.test.js
Просмотреть файл

@ -26,9 +26,7 @@ test("We can list commits", async () => {
{ author: { login: "danhoerst" } },
];
let provider = new GitHubProvider("token");
jest
.spyOn(provider, "listPRCommits")
.mockImplementation(() => prCommits);
jest.spyOn(provider, "listPRCommits").mockImplementation(() => prCommits);
expect(provider.listPRCommits()).toBe(prCommits);
let pullRequest = new PullRequest(provider);
@ -39,9 +37,7 @@ test("We can list commits", async () => {
test("We can list labels", async () => {
let prLabels = [{ name: "bug" }, { name: "feature-request" }];
let provider = new GitHubProvider("token");
jest
.spyOn(provider, "listLabelsOnPR")
.mockImplementation(() => prLabels);
jest.spyOn(provider, "listLabelsOnPR").mockImplementation(() => prLabels);
expect(provider.listLabelsOnPR()).toBe(prLabels);
let pullRequest = new PullRequest(provider);

60
__tests__/runner.test.js
Просмотреть файл

@ -25,9 +25,7 @@ afterEach(() => {
describe("processCommits", () => {
test("We process commits successfully", async () => {
let prCommits = [{ author: { login: "robot" } }];
jest
.spyOn(pullRequest, "listCommits")
.mockImplementation(() => prCommits);
jest.spyOn(pullRequest, "listCommits").mockImplementation(() => prCommits);
expect(pullRequest.listCommits()).toBe(prCommits);
let commits = await runner.processCommits("robot");
@ -39,9 +37,7 @@ describe("processCommits", () => {
{ author: { login: "robot" } },
{ author: { login: "danhoerst" } },
];
jest
.spyOn(pullRequest, "listCommits")
.mockImplementation(() => prCommits);
jest.spyOn(pullRequest, "listCommits").mockImplementation(() => prCommits);
expect(pullRequest.listCommits()).toBe(prCommits);
let commits = await runner.processCommits("robot");
@ -63,9 +59,7 @@ index 2f4e8d9..93c2072 100644
if: blah
with:
name: dist`;
jest
.spyOn(pullRequest, "getDiff")
.mockImplementation(() => prDiff);
jest.spyOn(pullRequest, "getDiff").mockImplementation(() => prDiff);
expect(pullRequest.getDiff()).toBe(prDiff);
let diff = await runner.processDiff();
@ -86,9 +80,7 @@ index 2f4e8d9..93c2072 100644
if: blah
with:
name: dist`;
jest
.spyOn(pullRequest, "getDiff")
.mockImplementation(() => prDiff);
jest.spyOn(pullRequest, "getDiff").mockImplementation(() => prDiff);
expect(pullRequest.getDiff()).toBe(prDiff);
let diff = await runner.processDiff();
@ -117,9 +109,7 @@ describe("labelsEqual", () => {
describe("processLabels", () => {
test("We process labels successfully", async () => {
let prLabels = [{ name: "bug" }, { name: "feature-request" }];
jest
.spyOn(pullRequest, "listLabels")
.mockImplementation(() => prLabels);
jest.spyOn(pullRequest, "listLabels").mockImplementation(() => prLabels);
expect(pullRequest.listLabels()).toBe(prLabels);
let commits = await runner.processLabels({
@ -130,9 +120,7 @@ describe("processLabels", () => {
test("We process labels unsuccessfully", async () => {
let prLabels = [{ name: "bug" }, { name: "feature-request" }];
jest
.spyOn(pullRequest, "listLabels")
.mockImplementation(() => prLabels);
jest.spyOn(pullRequest, "listLabels").mockImplementation(() => prLabels);
expect(pullRequest.listLabels()).toBe(prLabels);
let commits = await runner.processLabels({
@ -148,15 +136,11 @@ describe("processPrivilegedReviewer", () => {
process.env["INPUT_CHECKLABELS"] = "true";
process.env["INPUT_CHECKDIFF"] = "true";
let prLabels = [{ name: "bug" }, { name: "feature-request" }];
jest
.spyOn(pullRequest, "listLabels")
.mockImplementation(() => prLabels);
jest.spyOn(pullRequest, "listLabels").mockImplementation(() => prLabels);
expect(pullRequest.listLabels()).toBe(prLabels);
let prCommits = [{ author: { login: "robot" } }];
jest
.spyOn(pullRequest, "listCommits")
.mockImplementation(() => prCommits);
jest.spyOn(pullRequest, "listCommits").mockImplementation(() => prCommits);
expect(pullRequest.listCommits()).toBe(prCommits);
let prDiff = `| diff --git a/.github/workflows/check-dist.yml b/.github/workflows/check-dist.yml
@ -171,9 +155,7 @@ index 2f4e8d9..93c2072 100644
if: blah
with:
name: dist`;
jest
.spyOn(pullRequest, "getDiff")
.mockImplementation(() => prDiff);
jest.spyOn(pullRequest, "getDiff").mockImplementation(() => prDiff);
expect(pullRequest.getDiff()).toBe(prDiff);
let processed = await runner.processPrivilegedReviewer("robot", {
@ -189,9 +171,7 @@ index 2f4e8d9..93c2072 100644
{ author: { login: "robot" } },
{ author: { login: "malicious" } },
];
jest
.spyOn(pullRequest, "listCommits")
.mockImplementation(() => prCommits);
jest.spyOn(pullRequest, "listCommits").mockImplementation(() => prCommits);
expect(pullRequest.listCommits()).toBe(prCommits);
let processed = await runner.processPrivilegedReviewer("robot", {
@ -205,9 +185,7 @@ index 2f4e8d9..93c2072 100644
{ author: { login: "robot" } },
{ author: { login: "malicious" } },
];
jest
.spyOn(pullRequest, "listCommits")
.mockImplementation(() => prCommits);
jest.spyOn(pullRequest, "listCommits").mockImplementation(() => prCommits);
expect(pullRequest.listCommits()).toBe(prCommits);
let processed = await runner.processPrivilegedReviewer("robot", {
@ -219,9 +197,7 @@ index 2f4e8d9..93c2072 100644
test("We process labels unsuccessfully with the option enabled", async () => {
process.env["INPUT_CHECKLABELS"] = "true";
let prLabels = [{ name: "bug" }, { name: "feature-request" }];
jest
.spyOn(pullRequest, "listLabels")
.mockImplementation(() => prLabels);
jest.spyOn(pullRequest, "listLabels").mockImplementation(() => prLabels);
expect(pullRequest.listLabels()).toBe(prLabels);
let processed = await runner.processPrivilegedReviewer("robot", {
@ -232,9 +208,7 @@ index 2f4e8d9..93c2072 100644
test("We allow bad labels when the option to check them is not set", async () => {
let prLabels = [{ name: "bug" }, { name: "feature-request" }];
jest
.spyOn(pullRequest, "listLabels")
.mockImplementation(() => prLabels);
jest.spyOn(pullRequest, "listLabels").mockImplementation(() => prLabels);
expect(pullRequest.listLabels()).toBe(prLabels);
let processed = await runner.processPrivilegedReviewer("robot", {
@ -258,9 +232,7 @@ index 2f4e8d9..93c2072 100644
if: blah
with:
name: dist`;
jest
.spyOn(pullRequest, "getDiff")
.mockImplementation(() => prDiff);
jest.spyOn(pullRequest, "getDiff").mockImplementation(() => prDiff);
expect(pullRequest.getDiff()).toBe(prDiff);
let processed = await runner.processPrivilegedReviewer("robot", {
@ -283,9 +255,7 @@ index 2f4e8d9..93c2072 100644
if: blah
with:
name: dist`;
jest
.spyOn(pullRequest, "getDiff")
.mockImplementation(() => prDiff);
jest.spyOn(pullRequest, "getDiff").mockImplementation(() => prDiff);
expect(pullRequest.getDiff()).toBe(prDiff);
let processed = await runner.processPrivilegedReviewer("robot", {

14
dist/index.js сгенерированный поставляемый
Просмотреть файл

@ -36930,8 +36930,8 @@ class GitHubProvider {
}
async createReview(prNumber, reviewEvent) {
core.debug(`prNumber: ${prNumber}`)
core.debug(`reviewEvent: ${reviewEvent}`)
core.debug(`prNumber: ${prNumber}`);
core.debug(`reviewEvent: ${reviewEvent}`);
await this.octokit.rest.pulls.createReview({
owner: github.context.repo.owner,
@ -36942,8 +36942,8 @@ class GitHubProvider {
}
async getConfigContent() {
const path = core.getInput("path", { required: true })
core.info(`config path: ${path}`)
const path = core.getInput("path", { required: true });
core.info(`config path: ${path}`);
// getContent defaults to the main branch
const { data: configContent } = await this.octokit.rest.repos.getContent({
@ -37245,7 +37245,11 @@ class Runner {
const index_core = __nccwpck_require__(2186);
let myToken = index_core.getInput("myToken");
const robotUserToken = index_core.getInput("robotUserToken");
if (robotUserToken !== "" || robotUserToken !== undefined || robotUserToken !== null) {
if (
robotUserToken !== "" ||
robotUserToken !== undefined ||
robotUserToken !== null
) {
index_core.info("Robot User configured. I will use that PAT instead.");
myToken = robotUserToken;
}

2
dist/index.js.map сгенерированный поставляемый
Просмотреть файл

Различия файлов скрыты, потому что одна или несколько строк слишком длинны

6
index.js
Просмотреть файл

@ -6,7 +6,11 @@ import { Runner } from "./src/runner";
const core = require("@actions/core");
let myToken = core.getInput("myToken");
const robotUserToken = core.getInput("robotUserToken");
if (robotUserToken !== "" || robotUserToken !== undefined || robotUserToken !== null) {
if (
robotUserToken !== "" ||
robotUserToken !== undefined ||
robotUserToken !== null
) {
core.info("Robot User configured. I will use that PAT instead.");
myToken = robotUserToken;
}

2
package.json
Просмотреть файл

@ -11,7 +11,7 @@
"prepare": "ncc build index.js -o dist --source-map --license licenses.txt",
"test": "jest",
"bundle": "npm run format && npm run package",
"all": "npm run lint && npm run prepare && npm run test"
"all": "npm run lint && npm run format && npm run prepare && npm run test"
},
"repository": {
"type": "git",

8
src/github-provider.js
Просмотреть файл

@ -11,8 +11,8 @@ class GitHubProvider {
}
async createReview(prNumber, reviewEvent) {
core.debug(`prNumber: ${prNumber}`)
core.debug(`reviewEvent: ${reviewEvent}`)
core.debug(`prNumber: ${prNumber}`);
core.debug(`reviewEvent: ${reviewEvent}`);
await this.octokit.rest.pulls.createReview({
owner: github.context.repo.owner,
@ -23,8 +23,8 @@ class GitHubProvider {
}
async getConfigContent() {
const path = core.getInput("path", { required: true })
core.info(`config path: ${path}`)
const path = core.getInput("path", { required: true });
core.info(`config path: ${path}`);
// getContent defaults to the main branch
const { data: configContent } = await this.octokit.rest.repos.getContent({