Skip to content

chore: replace archived golang/mock with go.uber.org/mock - #1645

Merged
developerkunal merged 7 commits into
mainfrom
DXCDT-2383-replace-archived-golang-mock-with-uber-go-mock
Sep 9, 2026
Merged

chore: replace archived golang/mock with go.uber.org/mock#1645
developerkunal merged 7 commits into
mainfrom
DXCDT-2383-replace-archived-golang-mock-with-uber-go-mock

Conversation

@ramya18101

Copy link
Copy Markdown
Contributor

🔧 Changes

  • Replaced github.com/golang/mock v1.6.0 with go.uber.org/mock v0.6.0 in go.mod/go.sum.
  • Removed tools/tools.go; added a tool go.uber.org/mock/mockgen directive to go.mod.
  • Makefile: install the pinned mockgen (dropped @latest).
  • Regenerated all 30 mocks under internal/{auth,auth0}/mock (import → go.uber.org/mock/gomock).
  • Updated the gomock import across 28 internal/cli/*_test.go files.
  • Updated references/testing.md.

No changes to any command, flag, or output.

📚 References

🔬 Testing

  • make test-mocks — regenerates cleanly and idempotently.
  • Unit tests pass: internal/cli, internal/auth0, internal/auth.

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@ramya18101
ramya18101 marked this pull request as ready for review September 8, 2026 04:59
@ramya18101
ramya18101 requested a review from a team as a code owner September 8, 2026 04:59
@ramya18101
ramya18101 force-pushed the DXCDT-2383-replace-archived-golang-mock-with-uber-go-mock branch from 525d2dd to 013326a Compare September 9, 2026 14:47

@developerkunal developerkunal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@developerkunal
developerkunal merged commit b514c1c into main Sep 9, 2026
6 checks passed
@developerkunal
developerkunal deleted the DXCDT-2383-replace-archived-golang-mock-with-uber-go-mock branch September 9, 2026 15:41
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.

2 participants