Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(gomod): preserve go/toolchain directives #34779

Open
wants to merge 35 commits into
base: main
Choose a base branch
from
+268 −38
Open

fix(gomod): preserve go/toolchain directives #34779

Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
72f8adf
preserve go mod toolchain directive in any case
trim21 Mar 13, 2025
3629f60
preserve go mod toolchain directive in any case
trim21 Mar 13, 2025
ab9f5b9
add more test
trim21 Mar 13, 2025
be3d457
fix eslint
trim21 Mar 13, 2025
0d94a80
fix regex
trim21 Mar 13, 2025
59a450d
Update lib/modules/manager/gomod/artifacts.ts
trim21 Mar 13, 2025
4b3fd95
Update lib/modules/manager/gomod/artifacts.ts
trim21 Mar 13, 2025
7843e56
preserve -d location
trim21 Mar 13, 2025
3f2750a
better variable name
trim21 Mar 13, 2025
4db59ef
Merge branch 'main' into fix/go-mod-toolchain
trim21 Mar 13, 2025
125e3db
preserve go/toolchain directive
trim21 Mar 13, 2025
de81b17
fix constraints
trim21 Mar 13, 2025
1667c12
fix test
trim21 Mar 13, 2025
ffe38b8
keep old flag order
trim21 Mar 13, 2025
6f2afba
go: control updates of directives when constraint is present
casaqori Apr 2, 2025
9064156
Merge branch 'main' into fix/go-mod-toolchain
trim21 Apr 2, 2025
8f46eb1
add notice when nothing changed
casaqori Apr 2, 2025
f91d392
Merge branch 'main' into fix/go-mod-toolchain
trim21 Apr 3, 2025
2fc88c9
Revert "add notice when nothing changed"
trim21 Apr 5, 2025
875aa50
revert
trim21 Apr 5, 2025
ed66459
fix version
trim21 Apr 5, 2025
a4c4690
fix test
trim21 Apr 5, 2025
e238578
use join instead of string concat
trim21 Apr 5, 2025
0eb2aac
handle major upgrade
trim21 Apr 5, 2025
26c511c
fix lint
trim21 Apr 5, 2025
ffec247
cov
trim21 Apr 5, 2025
5264a45
cov
trim21 Apr 5, 2025
0858735
docs
trim21 Apr 5, 2025
c3bf9f8
fix(gomod): disable toolchain switch
trim21 Apr 7, 2025
dea2c8c
fix(gomod): disable toolchain switch
trim21 Apr 7, 2025
e321b50
pick minial version
trim21 Apr 11, 2025
60bdc84
Merge remote-tracking branch 'origin/fix/go-no-switch-toolchain' into…
trim21 Apr 11, 2025
f72f7ed
fix test
trim21 Apr 11, 2025
6a5b70d
fix test
trim21 Apr 11, 2025
575376f
add comment
trim21 Apr 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
130 changes: 94 additions & 36 deletions lib/modules/manager/gomod/artifacts.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ describe('modules/manager/gomod/artifacts', () => {
).toBeNull();
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: {
cwd: '/tmp/github/some/repo',
env: {
Expand Down Expand Up @@ -169,7 +169,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: {
cwd: '/tmp/github/some/repo',
env: {
Expand Down Expand Up @@ -227,7 +227,7 @@ describe('modules/manager/gomod/artifacts', () => {

expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -278,7 +278,7 @@ describe('modules/manager/gomod/artifacts', () => {

expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -359,7 +359,7 @@ describe('modules/manager/gomod/artifacts', () => {

expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -425,7 +425,7 @@ describe('modules/manager/gomod/artifacts', () => {

expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
]);
Expand Down Expand Up @@ -507,7 +507,7 @@ describe('modules/manager/gomod/artifacts', () => {

expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -585,7 +585,7 @@ describe('modules/manager/gomod/artifacts', () => {
' bash -l -c "' +
'install-tool golang 1.23.3' +
' && ' +
'go get -d -t ./...' +
'go get -t -d ./...' +
'"',
options: {
cwd: '/tmp/github/some/repo',
Expand Down Expand Up @@ -629,7 +629,7 @@ describe('modules/manager/gomod/artifacts', () => {
expect(execSnapshots).toMatchObject([
{ cmd: 'install-tool golang 1.23.3' },
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: {
cwd: '/tmp/github/some/repo',
env: {},
Expand Down Expand Up @@ -668,7 +668,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
]);
Expand Down Expand Up @@ -750,7 +750,7 @@ describe('modules/manager/gomod/artifacts', () => {
' bash -l -c "' +
'install-tool golang 1.23.3' +
' && ' +
'go get -d -t ./...' +
'go get -t -d ./...' +
'"',
options: {
cwd: '/tmp/github/some/repo',
Expand Down Expand Up @@ -1260,7 +1260,7 @@ describe('modules/manager/gomod/artifacts', () => {
' bash -l -c "' +
'install-tool golang 1.23.3' +
' && ' +
'go get -d -t ./...' +
'go get -t -d ./...' +
' && ' +
'go mod tidy' +
' && ' +
Expand Down Expand Up @@ -1324,7 +1324,7 @@ describe('modules/manager/gomod/artifacts', () => {
' bash -l -c "' +
'install-tool golang 1.23.3' +
' && ' +
'go get -d -t ./...' +
'go get -t -d ./...' +
' && ' +
'go mod tidy -compat=1.17' +
' && ' +
Expand Down Expand Up @@ -1388,7 +1388,7 @@ describe('modules/manager/gomod/artifacts', () => {
' bash -l -c "' +
'install-tool golang 1.23.3' +
' && ' +
'go get -d -t ./...' +
'go get -t -d ./...' +
' && ' +
'go mod tidy -compat=1.17 -e' +
' && ' +
Expand Down Expand Up @@ -1452,7 +1452,7 @@ describe('modules/manager/gomod/artifacts', () => {
' bash -l -c "' +
'install-tool golang 1.23.3' +
' && ' +
'go get -d -t ./...' +
'go get -t -d ./...' +
' && ' +
'go mod tidy -e' +
' && ' +
Expand Down Expand Up @@ -1522,7 +1522,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -1578,7 +1578,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -1635,7 +1635,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -1680,7 +1680,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -1729,7 +1729,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -1774,7 +1774,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
]);
Expand Down Expand Up @@ -1810,7 +1810,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
]);
Expand Down Expand Up @@ -1852,7 +1852,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -1910,7 +1910,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -1964,7 +1964,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -2026,7 +2026,7 @@ describe('modules/manager/gomod/artifacts', () => {
expect(execSnapshots).toMatchObject([
{ cmd: 'install-tool golang 1.23.3' },
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
options: { cwd: '/tmp/github/some/repo' },
},
{
Expand Down Expand Up @@ -2105,7 +2105,7 @@ describe('modules/manager/gomod/artifacts', () => {
' bash -l -c "' +
'install-tool golang 1.23.3' +
' && ' +
'go get -d -t ./...' +
'go get -t ./...' +
' && ' +
'go install github.com/marwan-at-work/mod/cmd/mod@latest' +
' && ' +
Expand Down Expand Up @@ -2134,6 +2134,14 @@ describe('modules/manager/gomod/artifacts', () => {
.mockResolvedValueOnce('New go.sum')
.mockResolvedValueOnce('New go.mod');

datasource.getPkgReleases.mockResolvedValueOnce({
releases: [
{ version: '1.17.0' },
{ version: '1.23.3' },
{ version: '1.23.6' },
],
});

const res = await gomod.updateArtifacts({
packageFileName: 'go.mod',
updatedDeps: [{ depName: 'golang.org/x/crypto', newVersion: '0.35.0' }],
Expand All @@ -2153,11 +2161,9 @@ describe('modules/manager/gomod/artifacts', () => {
cmd: 'install-tool golang 1.23.6',
},
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t -d ./...',
},
]);

expect(datasource.getPkgReleases).toBeCalledTimes(0);
});

it('go.mod file contains full go version without toolchain', async () => {
Expand All @@ -2174,6 +2180,14 @@ describe('modules/manager/gomod/artifacts', () => {
.mockResolvedValueOnce('New go.sum')
.mockResolvedValueOnce('New go.mod');

datasource.getPkgReleases.mockResolvedValueOnce({
releases: [
{ version: '1.17.0' },
{ version: '1.23.3' },
{ version: '1.24.1' },
],
});

const res = await gomod.updateArtifacts({
packageFileName: 'go.mod',
updatedDeps: [{ depName: 'golang.org/x/crypto', newVersion: '0.35.0' }],
Expand All @@ -2190,14 +2204,58 @@ describe('modules/manager/gomod/artifacts', () => {

expect(execSnapshots).toMatchObject([
{
cmd: 'install-tool golang 1.23.5',
cmd: 'install-tool golang 1.24.1',
},
{
cmd: 'go get -d -t ./...',
cmd: 'go get -t ./... toolchain@none [email protected]',
},
]);
});

it('preserve go.mod file without toolchain', async () => {
GlobalConfig.set({ ...adminConfig, binarySource: 'install' });
fs.readLocalFile.mockResolvedValueOnce('Current go.sum');
fs.readLocalFile.mockResolvedValueOnce(null); // vendor modules filename
const execSnapshots = mockExecAll();
git.getRepoStatus.mockResolvedValueOnce(
partial<StatusResult>({
modified: ['go.sum'],
}),
);
fs.readLocalFile
.mockResolvedValueOnce('New go.sum')
.mockResolvedValueOnce('New go.mod');

datasource.getPkgReleases.mockResolvedValueOnce({
releases: [
{ version: '1.17.0' },
{ version: '1.23.3' },
{ version: '1.24.1' },
],
});

const res = await gomod.updateArtifacts({
packageFileName: 'go.mod',
updatedDeps: [{ depName: 'golang.org/x/crypto', newVersion: '0.35.0' }],
newPackageFileContent: `someText\n\ngo 1.23\n\n${gomod1}`,
config: {
updateType: 'minor',
},
});

expect(res).toEqual([
{ file: { type: 'addition', path: 'go.sum', contents: 'New go.sum' } },
{ file: { type: 'addition', path: 'go.mod', contents: 'New go.mod' } },
]);

expect(datasource.getPkgReleases).toBeCalledTimes(0);
expect(execSnapshots).toMatchObject([
{
cmd: 'install-tool golang 1.24.1',
},
{
cmd: 'go get -t ./... toolchain@none [email protected]',
},
]);
});

it('returns artifact notices', async () => {
Expand Down Expand Up @@ -2299,7 +2357,7 @@ describe('modules/manager/gomod/artifacts', () => {
' bash -l -c "' +
'install-tool golang 1.23.3' +
' && ' +
'go get -d -t ./...' +
'go get -t -d ./...' +
' && ' +
'go install github.com/marwan-at-work/mod/cmd/mod@latest' +
' && ' +
Expand Down Expand Up @@ -2337,7 +2395,7 @@ describe('modules/manager/gomod/artifacts', () => {
).toBeNull();
expect(execSnapshots).toMatchObject([
{
cmd: "go get -d -t . foo .bar/... '&&' cat",
cmd: "go get -t -d . foo .bar/... '&&' cat",
options: {
cwd: '/tmp/github/some/repo',
},
Expand Down Expand Up @@ -2377,7 +2435,7 @@ describe('modules/manager/gomod/artifacts', () => {
]);
expect(execSnapshots).toMatchObject([
{
cmd: 'go get -d -t .',
cmd: 'go get -t -d .',
options: {
cwd: '/tmp/github/some/repo',
},
Expand Down
Loading