[BUG] Ensure `github_emu_group_mapping` behaves correctly if mapping changes upstream#3118

Merged

stevehipwell merged 25 commits intointegrations:mainfrom

F-Secure-web:fix-emu-group-mapping-read-issue

Feb 4, 2026

Commits on Feb 4, 2026

Move `toInt` and `toInt64` functions to `util.go`
deiga
committed
`resource_github_emu_group_mapping`) Change `Read` to use `ListExternalGroupsForTeamBySlug` to ensure we match state and reality
deiga
committed
`resource_github_emu_group_mapping`: Refactor `Create` to have less unnecessary code
deiga
committed
`emu_group_mapping`: Add `group_name` computed attribute
deiga
committed
Add matching of team ID into Read
deiga
committed
Split Create and Update to separate functions
deiga
committed
Update Importer to use new ID pattern
deiga
committed
Add Schema migration for new ID format
deiga
committed
Changes `group_id` to `ForceNew`
deiga
committed
Add skipped test while waiting for `terraform-plugin-testing`
deiga
committed
Add `CustomizeDiff` function to determine if `team_slug` change needs `ForceNew`
deiga
committed
Update docs
deiga
committed
Use `lookupTeamID` instead of `getTeamID`
deiga
committed
Move `Create` before `Read`
deiga
committed
Replace `deep` package with `go-cmp`
deiga
committed
`ListExternalGroupsForTeamBySlug` does not return a nested `Teams` slice. We need to directly lookup TeamID
deiga
committed
Replace `go-github-mock` with `githubApiMock`
deiga
committed
Remove unnecessry `matchTeamID` function
deiga
committed
Address review comments
deiga
committed
Rename state migartion functions
deiga
committed
Inline unnecessary function
deiga
committed
Use new reusable diffing pattern
deiga
committed
Address review comments
deiga
committed
Refactor mockResponse builder to accept inputs
deiga
committed
Delete test
deiga
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BUG] Ensure `github_emu_group_mapping` behaves correctly if mapping changes upstream#3118

[BUG] Ensure `github_emu_group_mapping` behaves correctly if mapping changes upstream#3118
stevehipwell merged 25 commits intointegrations:mainfrom
F-Secure-web:fix-emu-group-mapping-read-issue

Commits on Feb 4, 2026

Move `toInt` and `toInt64` functions to `util.go`

`resource_github_emu_group_mapping`) Change `Read` to use `ListExternalGroupsForTeamBySlug` to ensure we match state and reality

`resource_github_emu_group_mapping`: Refactor `Create` to have less unnecessary code

`emu_group_mapping`: Add `group_name` computed attribute

Add matching of team ID into Read

Split Create and Update to separate functions

Update Importer to use new ID pattern

Add Schema migration for new ID format

Changes `group_id` to `ForceNew`

Add skipped test while waiting for `terraform-plugin-testing`

Add `CustomizeDiff` function to determine if `team_slug` change needs `ForceNew`

Update docs

Use `lookupTeamID` instead of `getTeamID`

Move `Create` before `Read`

Replace `deep` package with `go-cmp`

`ListExternalGroupsForTeamBySlug` does not return a nested `Teams` slice. We need to directly lookup TeamID

Replace `go-github-mock` with `githubApiMock`

Remove unnecessry `matchTeamID` function

Address review comments

Rename state migartion functions

Inline unnecessary function

Use new reusable diffing pattern

Address review comments

Refactor mockResponse builder to accept inputs

Delete test