deps: bump github.com/stretchr/testify from 1.11.1 to 1.12.1 - #83
Conversation
Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.11.1 to 1.12.1. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.11.1...v1.12.1) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-version: 1.12.1 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo '--- PR metadata and changed files ---'
gh pr view 83 --repo pgEdge/coldfront --json title,body,files,commits \
--jq '{title, body, commits: [.commits[].messageHeadline], files: [.files[] | {path, additions, deletions}]}'
echo
echo '--- go.mod ---'
cat go.mod
echo
echo '--- Relevant Go imports and dependency references ---'
rg -n -C 2 \
'"(github\.com/stretchr/testify|go\.yaml\.in/yaml/v3|gopkg\.in/yaml\.v3|github\.com/davecgh/go-spew|github\.com/pmezard/go-difflib)' \
-g '*.go' .
echo
echo '--- Module graph explanation ---'
go mod why -m github.com/stretchr/testify
go mod why -m go.yaml.in/yaml/v3
go mod why -m github.com/davecgh/go-spew
go mod why -m github.com/pmezard/go-difflibLength of output: 21150
@vyruss, this update is feasible. I assess the production risk as low.
- Pull request
#83changes onlygo.modandgo.sum. github.com/stretchr/testifyis used only by test files throughassertandrequire.- The application continues to use
gopkg.in/yaml.v3 v3.0.1for configuration parsing. go.yaml.in/yaml/v3 v3.0.5is an indirect dependency introduced bytestify. It does not replace the application YAML dependency.- The removed
github.com/davecgh/go-spewandgithub.com/pmezard/go-difflibchecksums are expected.go mod whyconfirms that the resolved module graph no longer needs these modules.
The remaining risk is limited to test compatibility. The v1.12.x release includes assertion and diagnostic behavior changes. The repository does not import testify/mock, which reduces the main API-compatibility risk.
Recommendation: merge after the normal Go test suite passes. No source changes are required based on the current dependency graph.
You are interacting with an AI system.
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo '== Go module files =='
fd -a '^(go\.mod|go\.sum)$' . -x sh -c 'echo "--- $1"; sed -n "1,240p" "$1"' sh {}
echo
echo '== Testify and YAML imports in Go source =='
rg -n -C 2 --glob '*.go' \
'"github\.com/stretchr/testify/|"gopkg\.in/yaml\.v3"|"go\.yaml\.in/yaml/v3"' \
. || true
echo
echo '== Dependency declarations and sums for affected modules =='
rg -n \
'github\.com/stretchr/testify|github\.com/davecgh/go-spew|github\.com/pmezard/go-difflib|gopkg\.in/yaml\.v3|go\.yaml\.in/yaml/v3' \
go.mod go.sum 2>/dev/null || true
echo
echo '== Testify module metadata and dependency manifest =='
modcache="$(go env GOMODCACHE 2>/dev/null || true)"
if [ -n "$modcache" ]; then
testify_mod="$(find "$modcache" -path '*/github.com/stretchr/testify@v1.12.1/go.mod' -print -quit 2>/dev/null || true)"
if [ -n "$testify_mod" ]; then
echo "--- $testify_mod"
cat "$testify_mod"
else
echo 'testify v1.12.1 is not present in the local module cache.'
fi
fi
echo
echo '== GitHub security advisories for testify v1.12.1 =='
gh api '/repos/stretchr/testify/security-advisories?state=published&per_page=100' \
--jq '.[] | {ghsa_id, severity, published_at, vulnerable_version_range, patched_versions}' \
2>/dev/null || echo 'GitHub advisory query was unavailable.'Length of output: 39110
@vyruss, feasibility is high and functional risk is low for the root module.
github.com/stretchr/testifyis used only by root-module test files.- The update does not change application imports or production code paths.
gopkg.in/yaml.v3 v3.0.1remains the YAML module used by production code.go.yaml.in/yaml/v3 v3.0.5is a separate module path. No repository source imports it directly. Its addition affects the resolved test dependency graph only.- Removing
github.com/davecgh/go-spewandgithub.com/pmezard/go-difflibfromgo.modis consistent with the new resolved graph. Their historicalgo.sumentries are harmless. - The advisory query returned no published GitHub security advisories for Testify.
One scope note: cmd/compactor has its own go.mod and remains on github.com/stretchr/testify v1.11.1 indirectly. This root-module change does not update that module.
Residual risk: Testify can change assertion formatting or edge-case assertion behavior. CI should run the root module test suite before merge. The sandbox did not execute tests.
You are interacting with an AI system.
Bumps github.com/stretchr/testify from 1.11.1 to 1.12.1.
Release notes
Sourced from github.com/stretchr/testify's releases.
... (truncated)
Commits
959dbdaMerge pull request #1935 from harryzcy/yaml-update9bb7176Update go.yaml.in/yaml/v3 to v3.0.5001eb79Merge pull request #1905 from Kentzo/patch-1ad40f38Merge pull request #1906 from stretchr/dependabot/github_actions/actions/chec...3bae017build(deps): bump actions/checkout from 6.0.2 to 6.0.3f8c01f3mock: Mock.Return does not exist anymore12f8b56Merge pull request #1563 from stretchr/make-AssertionFunc-types-aliasesa11649eassert: make *AssertionFunc type just aliasesdc20f41Merge pull request #1890 from stretchr/dolmen/codegen-modernize098f8d7_codegen: use strings.BuilderDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)