paulononato/backstage
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Allow setting dismissStaleReviews on github:publish action

Browse Source 
Signed-off-by: Miguel Alexandre <m.alexandrex@gmail.com>
This commit is contained in:
Miguel Alexandre
2022-12-09 16:26:17 +01:00
parent d4543aea7c
commit 0053d07bee
8 changed files with 160 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.changeset/witty-wasps-kiss.md
+5
View File
@@ -0,0 +1,5 @@
---
'@backstage/plugin-scaffolder-backend': minor
---
Update the `github:publish` action to allow passing wether to dismiss stale reviews on the protected default branch.
plugins/scaffolder-backend/src/scaffolder/actions/builtin/github/githubRepoPush.test.ts
+66
View File
@@ -495,4 +495,70 @@ describe('github:repo:push', () => {
expect(enableBranchProtectionOnDefaultRepoBranch).not.toHaveBeenCalled();
});
it('should call enableBranchProtectionOnDefaultRepoBranch with the correct values of dismissStaleReviews', async () => {
mockOctokit.rest.repos.get.mockResolvedValue({
data: {
clone_url: 'https://github.com/clone/url.git',
html_url: 'https://github.com/html/url',
},
});
await action.handler(mockContext);
expect(enableBranchProtectionOnDefaultRepoBranch).toHaveBeenCalledWith({
owner: 'owner',
client: mockOctokit,
repoName: 'repository',
logger: mockContext.logger,
defaultBranch: 'master',
requireCodeOwnerReviews: false,
requiredStatusCheckContexts: [],
requireBranchesToBeUpToDate: true,
enforceAdmins: true,
dismissStaleReviews: false,
});
await action.handler({
...mockContext,
input: {
...mockContext.input,
dismissStaleReviews: true,
},
});
expect(enableBranchProtectionOnDefaultRepoBranch).toHaveBeenCalledWith({
owner: 'owner',
client: mockOctokit,
repoName: 'repository',
logger: mockContext.logger,
defaultBranch: 'master',
requireCodeOwnerReviews: false,
requiredStatusCheckContexts: [],
requireBranchesToBeUpToDate: true,
enforceAdmins: true,
dismissStaleReviews: true,
});
await action.handler({
...mockContext,
input: {
...mockContext.input,
dismissStaleReviews: false,
},
});
expect(enableBranchProtectionOnDefaultRepoBranch).toHaveBeenCalledWith({
owner: 'owner',
client: mockOctokit,
repoName: 'repository',
logger: mockContext.logger,
defaultBranch: 'master',
requireCodeOwnerReviews: false,
requiredStatusCheckContexts: [],
requireBranchesToBeUpToDate: true,
enforceAdmins: true,
dismissStaleReviews: false,
});
});
});
plugins/scaffolder-backend/src/scaffolder/actions/builtin/github/githubRepoPush.ts
+4
View File
@@ -49,6 +49,7 @@ export function createGithubRepoPushAction(options: {
gitAuthorName?: string;
gitAuthorEmail?: string;
requireCodeOwnerReviews?: boolean;
dismissStaleReviews?: boolean;
bypassPullRequestAllowances?:
| {
users?: string[];
@@ -71,6 +72,7 @@ export function createGithubRepoPushAction(options: {
properties: {
repoUrl: inputProps.repoUrl,
requireCodeOwnerReviews: inputProps.requireCodeOwnerReviews,
dismissStaleReviews: inputProps.dismissStaleReviews,
requiredStatusCheckContexts: inputProps.requiredStatusCheckContexts,
bypassPullRequestAllowances: inputProps.bypassPullRequestAllowances,
requireBranchesToBeUpToDate: inputProps.requireBranchesToBeUpToDate,
@@ -102,6 +104,7 @@ export function createGithubRepoPushAction(options: {
gitAuthorName,
gitAuthorEmail,
requireCodeOwnerReviews = false,
dismissStaleReviews = false,
bypassPullRequestAllowances,
requiredStatusCheckContexts = [],
requireBranchesToBeUpToDate = true,
@@ -148,6 +151,7 @@ export function createGithubRepoPushAction(options: {
gitCommitMessage,
gitAuthorName,
gitAuthorEmail,
dismissStaleReviews,
);
ctx.output('remoteUrl', remoteUrl);
plugins/scaffolder-backend/src/scaffolder/actions/builtin/github/helpers.ts
+2
View File
@@ -263,6 +263,7 @@ export async function initRepoPushAndProtect(
gitCommitMessage?: string,
gitAuthorName?: string,
gitAuthorEmail?: string,
dismissStaleReviews?: boolean,
) {
const gitAuthorInfo = {
name: gitAuthorName
@@ -303,6 +304,7 @@ export async function initRepoPushAndProtect(
requiredStatusCheckContexts,
requireBranchesToBeUpToDate,
enforceAdmins: protectEnforceAdmins,
dismissStaleReviews: dismissStaleReviews,
});
} catch (e) {
assertError(e);
plugins/scaffolder-backend/src/scaffolder/actions/builtin/github/inputProperties.ts
+7
View File
@@ -38,6 +38,12 @@ const requireCodeOwnerReviews = {
'Require an approved review in PR including files with a designated Code Owner',
type: 'boolean',
};
const dismissStaleReviews = {
title: 'Dismiss Stale Reviews',
description:
'New reviewable commits pushed to a matching branch will dismiss pull request review approvals.',
type: 'boolean',
};
const requiredStatusCheckContexts = {
title: 'Required Status Check Contexts',
description:
@@ -207,6 +213,7 @@ export { bypassPullRequestAllowances };
export { repoUrl };
export { repoVisibility };
export { requireCodeOwnerReviews };
export { dismissStaleReviews };
export { requiredStatusCheckContexts };
export { requireBranchesToBeUpToDate };
export { sourcePath };
plugins/scaffolder-backend/src/scaffolder/actions/builtin/helpers.ts
+3
View File
@@ -193,6 +193,7 @@ type BranchProtectionOptions = {
requireBranchesToBeUpToDate?: boolean;
defaultBranch?: string;
enforceAdmins?: boolean;
dismissStaleReviews?: boolean;
};
export const enableBranchProtectionOnDefaultRepoBranch = async ({
@@ -206,6 +207,7 @@ export const enableBranchProtectionOnDefaultRepoBranch = async ({
requireBranchesToBeUpToDate = true,
defaultBranch = 'master',
enforceAdmins = true,
dismissStaleReviews = false,
}: BranchProtectionOptions): Promise<void> => {
const tryOnce = async () => {
try {
@@ -233,6 +235,7 @@ export const enableBranchProtectionOnDefaultRepoBranch = async ({
required_approving_review_count: 1,
require_code_owner_reviews: requireCodeOwnerReviews,
bypass_pull_request_allowances: bypassPullRequestAllowances,
dismiss_stale_reviews: dismissStaleReviews,
},
});
} catch (e) {
plugins/scaffolder-backend/src/scaffolder/actions/builtin/publish/github.test.ts
+69
View File
@@ -922,4 +922,73 @@ describe('publish:github', () => {
names: ['node.js'],
});
});
it('should call enableBranchProtectionOnDefaultRepoBranch with the correct values of dismissStaleReviews', async () => {
mockOctokit.rest.users.getByUsername.mockResolvedValue({
data: { type: 'User' },
});
mockOctokit.rest.repos.createForAuthenticatedUser.mockResolvedValue({
data: {
name: 'repo',
},
});
await action.handler(mockContext);
expect(enableBranchProtectionOnDefaultRepoBranch).toHaveBeenCalledWith({
owner: 'owner',
client: mockOctokit,
repoName: 'repo',
logger: mockContext.logger,
defaultBranch: 'master',
requireCodeOwnerReviews: false,
requiredStatusCheckContexts: [],
requireBranchesToBeUpToDate: true,
enforceAdmins: true,
dismissStaleReviews: false,
});
await action.handler({
...mockContext,
input: {
...mockContext.input,
dismissStaleReviews: true,
},
});
expect(enableBranchProtectionOnDefaultRepoBranch).toHaveBeenCalledWith({
owner: 'owner',
client: mockOctokit,
repoName: 'repo',
logger: mockContext.logger,
defaultBranch: 'master',
requireCodeOwnerReviews: false,
requiredStatusCheckContexts: [],
requireBranchesToBeUpToDate: true,
enforceAdmins: true,
dismissStaleReviews: true,
});
await action.handler({
...mockContext,
input: {
...mockContext.input,
dismissStaleReviews: false,
},
});
expect(enableBranchProtectionOnDefaultRepoBranch).toHaveBeenCalledWith({
owner: 'owner',
client: mockOctokit,
repoName: 'repo',
logger: mockContext.logger,
defaultBranch: 'master',
requireCodeOwnerReviews: false,
requiredStatusCheckContexts: [],
requireBranchesToBeUpToDate: true,
enforceAdmins: true,
dismissStaleReviews: false,
});
});
});
plugins/scaffolder-backend/src/scaffolder/actions/builtin/publish/github.ts
+4
View File
@@ -68,6 +68,7 @@ export function createPublishGithubAction(options: {
}
| undefined;
requireCodeOwnerReviews?: boolean;
dismissStaleReviews?: boolean;
requiredStatusCheckContexts?: string[];
requireBranchesToBeUpToDate?: boolean;
repoVisibility?: 'private' | 'internal' | 'public';
@@ -103,6 +104,7 @@ export function createPublishGithubAction(options: {
access: inputProps.access,
bypassPullRequestAllowances: inputProps.bypassPullRequestAllowances,
requireCodeOwnerReviews: inputProps.requireCodeOwnerReviews,
dismissStaleReviews: inputProps.dismissStaleReviews,
requiredStatusCheckContexts: inputProps.requiredStatusCheckContexts,
requireBranchesToBeUpToDate: inputProps.requireBranchesToBeUpToDate,
repoVisibility: inputProps.repoVisibility,
@@ -138,6 +140,7 @@ export function createPublishGithubAction(options: {
homepage,
access,
requireCodeOwnerReviews = false,
dismissStaleReviews = false,
bypassPullRequestAllowances,
requiredStatusCheckContexts = [],
requireBranchesToBeUpToDate = true,
@@ -213,6 +216,7 @@ export function createPublishGithubAction(options: {
gitCommitMessage,
gitAuthorName,
gitAuthorEmail,
dismissStaleReviews,
);
ctx.output('remoteUrl', remoteUrl);