actions/create-github-app-token
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

improve installation match; refactor test per copilot review

Browse Source
This commit is contained in:
Stefan Petrushevski
2025-07-08 17:52:25 +02:00
parent 3c69395e16
commit 46f9f788b8
3 changed files with 8 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
dist/main.cjs
Vendored
+2 -3
View File
@@ -42667,12 +42667,11 @@ async function getTokenFromEnterprise(request2, auth5, enterprise2, permissions2
}
});
const enterpriseInstallation = response.data.find(
(installation) => installation.target_type === "Enterprise"
(installation) => installation.target_type === "Enterprise" && installation.account?.slug === enterprise2
);
if (!enterpriseInstallation) {
throw new Error(`No enterprise installation found. Available installations: ${response.data.map((i) => `${i.target_type}:${i.account?.login || "N/A"}`).join(", ")}`);
throw new Error(`No enterprise installation found matching the name ${enterprise2}. Available installations: ${response.data.map((i) => `${i.target_type}:${i.account?.login || "N/A"}`).join(", ")}`);
}
console.info(`### Found enterprise installation: ${JSON.stringify(enterpriseInstallation, null, 2)}`);
const authentication = await auth5({
type: "installation",
installationId: enterpriseInstallation.id,
lib/main.js
+3 -4
View File
@@ -221,15 +221,14 @@ async function getTokenFromEnterprise(request, auth, enterprise, permissions) {
// Find the enterprise installation
const enterpriseInstallation = response.data.find(
installation => installation.target_type === "Enterprise"
installation => installation.target_type === "Enterprise" &&
installation.account?.slug === enterprise
);
if (!enterpriseInstallation) {
throw new Error(`No enterprise installation found. Available installations: ${response.data.map(i => `${i.target_type}:${i.account?.login || 'N/A'}`).join(', ')}`);
throw new Error(`No enterprise installation found matching the name ${enterprise}. Available installations: ${response.data.map(i => `${i.target_type}:${i.account?.login || 'N/A'}`).join(', ')}`);
}
console.info(`### Found enterprise installation: ${JSON.stringify(enterpriseInstallation, null, 2)}`);
// Get token for the enterprise installation
const authentication = await auth({
type: "installation",
tests/main-enterprise-installation-not-found.test.js
+3 -2
View File
@@ -1,9 +1,10 @@
import { test } from "./main.js";
delete process.env.INPUT_OWNER;
delete process.env.INPUT_REPOSITORIES;
// Verify `main` handles when no enterprise installation is found.
await test((mockPool) => {
delete process.env.INPUT_OWNER;
delete process.env.INPUT_REPOSITORIES;
process.env.INPUT_ENTERPRISE = "test-enterprise";