deps: update release-please to 17.1.2 (#1137)

The Octokit update requires providing/mocking a fetch implementation
(which is necessary because we use nock for http mocking)

Fixes #1136
This commit is contained in:
Jeff Ching
2025-08-20 11:43:09 -07:00
committed by GitHub
parent 1155c2c8dc
commit f07192c046
6 changed files with 1899 additions and 8722 deletions

9186
dist/index.js vendored

File diff suppressed because one or more lines are too long

2
dist/src/index.d.ts vendored
View File

@@ -1 +1 @@
export declare function main(): Promise<void>;
export declare function main(fetchOverride?: any): Promise<void>;

1386
package-lock.json generated

File diff suppressed because it is too large Load Diff

View File

@@ -29,7 +29,7 @@
"homepage": "https://github.com/googleapis/release-please-action#readme",
"dependencies": {
"@actions/core": "^1.10.0",
"release-please": "^16.18.0"
"release-please": "^17.1.2"
},
"devDependencies": {
"@types/mocha": "^9.0.0",
@@ -38,9 +38,10 @@
"@vercel/ncc": "^0.38.0",
"c8": "^7.5.0",
"gts": "^5.2.0",
"mocha": "^9.1.4",
"mocha": "^11.7.1",
"mocked-env": "^1.3.5",
"nock": "^13.2.1",
"node-fetch": "^2.6.0",
"sinon": "^9.2.4",
"standard": "^16.0.4",
"typescript": "^4.6.4"

View File

@@ -118,10 +118,10 @@ function loadOrBuildManifest(
);
}
export async function main() {
export async function main(fetchOverride?: any) {
core.info(`Running release-please version: ${VERSION}`)
const inputs = parseInputs();
const github = await getGitHubInstance(inputs);
const github = await getGitHubInstance(inputs, fetchOverride);
if (!inputs.skipGitHubRelease) {
const manifest = await loadOrBuildManifest(github, inputs);
@@ -136,7 +136,7 @@ export async function main() {
}
}
function getGitHubInstance(inputs: ActionInputs): Promise<GitHub> {
function getGitHubInstance(inputs: ActionInputs, fetchOverride?: any): Promise<GitHub> {
const [owner, repo] = inputs.repoUrl.split('/');
let proxy: Proxy | undefined = undefined;
if (inputs.proxyServer) {
@@ -155,6 +155,7 @@ function getGitHubInstance(inputs: ActionInputs): Promise<GitHub> {
graphqlUrl: inputs.githubGraphqlUrl,
token: inputs.token,
defaultBranch: inputs.targetBranch,
fetch: fetchOverride,
};
return GitHub.create(githubCreateOpts);
}

View File

@@ -20,6 +20,7 @@ import * as sinon from 'sinon';
import * as nock from 'nock';
import {RestoreFn} from 'mocked-env';
import mockedEnv from 'mocked-env';
const fetch = require('node-fetch');
import {Manifest, GitHub} from 'release-please';
// As defined in action.yml
@@ -106,7 +107,7 @@ describe('release-please-action', () => {
});
fakeManifest.createReleases.resolves([]);
fakeManifest.createPullRequests.resolves([]);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
});
@@ -116,7 +117,7 @@ describe('release-please-action', () => {
'release-type': 'simple',
});
fakeManifest.createPullRequests.resolves([]);
await action.main();
await action.main(fetch);
sinon.assert.notCalled(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
});
@@ -126,7 +127,7 @@ describe('release-please-action', () => {
'release-type': 'simple',
});
fakeManifest.createReleases.resolves([]);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.notCalled(fakeManifest.createPullRequests);
});
@@ -137,7 +138,7 @@ describe('release-please-action', () => {
});
fakeManifest.createReleases.resolves([]);
fakeManifest.createPullRequests.resolves([]);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
@@ -157,7 +158,7 @@ describe('release-please-action', () => {
});
fakeManifest.createReleases.resolves([]);
fakeManifest.createPullRequests.resolves([]);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
@@ -184,7 +185,7 @@ describe('release-please-action', () => {
restoreEnv = mockInputs({});
fakeManifest.createReleases.resolves([]);
fakeManifest.createPullRequests.resolves([]);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
});
@@ -193,7 +194,7 @@ describe('release-please-action', () => {
'skip-github-release': 'true',
});
fakeManifest.createPullRequests.resolves([]);
await action.main();
await action.main(fetch);
sinon.assert.notCalled(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
});
@@ -202,7 +203,7 @@ describe('release-please-action', () => {
'skip-github-pull-request': 'true',
});
fakeManifest.createReleases.resolves([]);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.notCalled(fakeManifest.createPullRequests);
});
@@ -212,7 +213,7 @@ describe('release-please-action', () => {
});
fakeManifest.createReleases.resolves([]);
fakeManifest.createPullRequests.resolves([]);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
@@ -230,7 +231,7 @@ describe('release-please-action', () => {
});
fakeManifest.createReleases.resolves([]);
fakeManifest.createPullRequests.resolves([]);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
@@ -256,7 +257,7 @@ describe('release-please-action', () => {
const fromManifestStub = sandbox
.stub(Manifest, 'fromManifest')
.resolves(fakeManifest);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
@@ -281,7 +282,7 @@ describe('release-please-action', () => {
fakeManifest.createReleases.resolves([]);
fakeManifest.createPullRequests.resolves([]);
sandbox.stub(Manifest, 'fromManifest').resolves(fakeManifest);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
@@ -324,7 +325,7 @@ describe('release-please-action', () => {
]);
fakeManifest.createPullRequests.resolves([]);
sandbox.stub(Manifest, 'fromManifest').resolves(fakeManifest);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
@@ -348,7 +349,7 @@ describe('release-please-action', () => {
fakeManifest.createReleases.resolves([]);
fakeManifest.createPullRequests.resolves([fixturePrs[0]]);
sandbox.stub(Manifest, 'fromManifest').resolves(fakeManifest);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
@@ -396,7 +397,7 @@ describe('release-please-action', () => {
]);
fakeManifest.createPullRequests.resolves([]);
sandbox.stub(Manifest, 'fromManifest').resolves(fakeManifest);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
assert.strictEqual(output['a--id'], 123456);
@@ -432,7 +433,7 @@ describe('release-please-action', () => {
fakeManifest.createReleases.resolves([]);
fakeManifest.createPullRequests.resolves(fixturePrs);
sandbox.stub(Manifest, 'fromManifest').resolves(fakeManifest);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);
@@ -446,7 +447,7 @@ describe('release-please-action', () => {
fakeManifest.createReleases.resolves([]);
fakeManifest.createPullRequests.resolves([]);
sandbox.stub(Manifest, 'fromManifest').resolves(fakeManifest);
await action.main();
await action.main(fetch);
sinon.assert.calledOnce(fakeManifest.createReleases);
sinon.assert.calledOnce(fakeManifest.createPullRequests);