rename to branchRulesetChecks()

github · Dec 12, 2024 · 6a65a3b · 6a65a3b
1 parent d8ab95b
commit 6a65a3b
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 18 deletions.
diff --git a/__tests__/functions/branch-protection-checks.test.js b/__tests__/functions/branch-protection-checks.test.js
@@ -1,4 +1,4 @@
-import {branchProtectionChecks} from '../../src/functions/branch-protection-checks'
+import {branchRulesetChecks} from '../../src/functions/branch-ruleset-checks'
 import * as core from '@actions/core'
 import {COLORS} from '../../src/functions/colors'
 import {SUGGESTED_RULESETS} from '../../src/functions/suggested-rulesets'
@@ -85,7 +85,7 @@ test('finds that no branch protections or rulesets are defined', async () => {
       }
     }
   }
-  expect(await branchProtectionChecks(context, octokit, data)).toStrictEqual({
+  expect(await branchRulesetChecks(context, octokit, data)).toStrictEqual({
     success: false,
     failed_checks: ['missing_branch_rulesets']
   })
@@ -96,7 +96,7 @@ test('finds that no branch protections or rulesets are defined', async () => {
 
 test('exits early if the user has disabled security warnings', async () => {
   data.use_security_warnings = false
-  expect(await branchProtectionChecks(context, octokit, data)).toStrictEqual({
+  expect(await branchRulesetChecks(context, octokit, data)).toStrictEqual({
     success: true
   })
   expect(warningMock).not.toHaveBeenCalled()
@@ -116,7 +116,7 @@ test('finds that the branch ruleset is missing the deletion rule', async () => {
     }
   }
 
-  expect(await branchProtectionChecks(context, octokit, data)).toStrictEqual({
+  expect(await branchRulesetChecks(context, octokit, data)).toStrictEqual({
     success: false,
     failed_checks: ['missing_deletion']
   })
@@ -147,7 +147,7 @@ test('finds that the branch ruleset is missing the dismiss_stale_reviews_on_push
     }
   }
 
-  expect(await branchProtectionChecks(context, octokit, data)).toStrictEqual({
+  expect(await branchRulesetChecks(context, octokit, data)).toStrictEqual({
     success: false,
     failed_checks: ['mismatch_pull_request_dismiss_stale_reviews_on_push']
   })
@@ -172,7 +172,7 @@ test('finds that all suggested branch rulesets are defined', async () => {
     }
   }
 
-  expect(await branchProtectionChecks(context, octokit, data)).toStrictEqual({
+  expect(await branchRulesetChecks(context, octokit, data)).toStrictEqual({
     success: true,
     failed_checks: []
   })
@@ -211,7 +211,7 @@ test('finds that all suggested branch rulesets are defined but required reviews
     }
   }
 
-  expect(await branchProtectionChecks(context, octokit, data)).toStrictEqual({
+  expect(await branchRulesetChecks(context, octokit, data)).toStrictEqual({
     success: false,
     failed_checks: ['mismatch_pull_request_required_approving_review_count']
   })

diff --git a/__tests__/main.test.js b/__tests__/main.test.js
@@ -2,7 +2,7 @@ import {run} from '../src/main'
 import * as reactEmote from '../src/functions/react-emote'
 import * as contextCheck from '../src/functions/context-check'
 import * as prechecks from '../src/functions/prechecks'
-import * as branchProtectionChecks from '../src/functions/branch-protection-checks'
+import * as branchRulesetChecks from '../src/functions/branch-ruleset-checks'
 import * as help from '../src/functions/help'
 import * as validPermissions from '../src/functions/valid-permissions'
 import * as identicalCommitCheck from '../src/functions/identical-commit-check'
@@ -166,7 +166,7 @@ beforeEach(() => {
     }
   })
   jest
-    .spyOn(branchProtectionChecks, 'branchProtectionChecks')
+    .spyOn(branchRulesetChecks, 'branchRulesetChecks')
     .mockImplementation(() => {
       return undefined
     })

diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/functions/branch-protection-checks.js → src/functions/branch-ruleset-checks.js b/src/functions/branch-protection-checks.js → src/functions/branch-ruleset-checks.js
@@ -3,7 +3,7 @@ import {COLORS} from './colors'
 import {API_HEADERS} from './api-headers'
 import {SUGGESTED_RULESETS} from './suggested-rulesets'
 
-export async function branchProtectionChecks(context, octokit, data) {
+export async function branchRulesetChecks(context, octokit, data) {
   const branch = data.branch
   const use_security_warnings = data?.use_security_warnings !== false
 

diff --git a/src/main.js b/src/main.js
@@ -14,7 +14,7 @@ import {actionStatus} from './functions/action-status'
 import {createDeploymentStatus} from './functions/deployment'
 import {isDeprecated} from './functions/deprecated-checks'
 import {prechecks} from './functions/prechecks'
-import {branchProtectionChecks} from './functions/branch-protection-checks'
+import {branchRulesetChecks} from './functions/branch-ruleset-checks'
 import {validPermissions} from './functions/valid-permissions'
 import {lock} from './functions/lock'
 import {unlock} from './functions/unlock'
@@ -450,8 +450,8 @@ export async function run() {
       return 'failure'
     }
 
-    // run branch protection checks
-    await branchProtectionChecks(context, octokit, {
+    // run branch ruleset checks
+    await branchRulesetChecks(context, octokit, {
       branch: inputs.stable_branch,
       use_security_warnings: inputs.use_security_warnings
     })