Skip to content

Consolidate bind setter test helpers - #520

Merged
linkdata merged 1 commit into
mainfrom
cleanup/issue-494-generic-setter-tests
Oct 2, 2026
Merged

linkdata merged 1 commit into
mainfrom
cleanup/issue-494-generic-setter-tests

Conversation

@linkdata

@linkdata linkdata commented Oct 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Replace four duplicate setter test helpers with one generic helper.
  • Keep each type's next-value operation next to its test and preserve both plain and success-hook setter checks.
  • Improve assertion diagnostics and mark the shared helper with t.Helper().

Closes #494.

Verification

  • go test -race ./...
  • go vet ./...
  • go build ./...
  • staticcheck ./...
  • golangci-lint run
  • gosec -quiet ./...
  • gofumpt -l . and gofmt -l .

@linkdata
linkdata merged commit 3076b42 into main Oct 2, 2026
7 checks passed
@linkdata
linkdata deleted the cleanup/issue-494-generic-setter-tests branch October 2, 2026 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bind: consider consolidating setter test helpers

1 participant