[MAINT] Fix actions_hosted_runner tests

[deiga]

Resolves #3405
Closes #3406

---

Before the change?

• resource_github_actions_hosted_runner tests have naming collisions
• resource_github_actions_hosted_runner uses legacy patterns

After the change?

• resource_github_actions_hosted_runner tests use ConfigStateChecks and ConfigPlanChecks
• Ensure resource_github_actions_hosted_runner tests don't use conflicting names
• Refactors resource_github_actions_hosted_runner to use Context-aware CRUD functions
• Refactors resource_github_actions_hosted_runner to not call Read after Update and Create
• Refactors resource_github_actions_hosted_runner to use go-github API methods
• Adds sweeper for Organization Runner Groups

Pull request checklist

• Schema migrations have been created if needed (example)
• Tests for the changes have been added (for bug fixes / features)
• Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

• Yes
• No

---

[github-actions]

👋 Hi, and thank you for this contribution!

This repo is maintained by GitHub and community members on a best-effort basis. We'll get to this as soon as we can.

You can help us prioritize by joining the discussion on open issues and PRs, sharing details on the changes you need, and reviewing other contributions.

---

_{🤖 This is an automated message.}

[stevehipwell]

Just a quick pre-review, the comments apply to all matching patterns not just the ones I commented on.

[Copilot]

Pull request overview

These provider review instructions are being used.

This PR modernizes the github_actions_hosted_runner resource and its acceptance tests. It migrates the resource CRUD functions to the *Context variants returning diag.Diagnostics, replaces hand-rolled client.NewRequest/client.Do calls and map[string]any payload plumbing with typed go-github Actions.*HostedRunner methods, and stops calling Read after Create/Update (consistent with this provider's no-read-after-write convention). The tests are rewritten to use ConfigStateChecks/ConfigPlanChecks, share a single runner group created via a new helper, and use non-colliding subtest names. A new sweeper for organization runner groups (and their hosted runners) is added, and the default sweep target is broadened.

Changes:

• Refactor the hosted-runner resource to context-aware CRUD + typed go-github API calls, with new expand/flatten helpers operating on typed structs.
• Rewrite acceptance tests to statecheck/plancheck, unique subtest names, and a shared mustCreateTestOrganizationActionsRunnerGroup helper.
• Add sweepOrganizationRunnerGroups/sweepRunnerGroupRunners sweepers and change SWEEP default to all.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
github/resource_github_actions_hosted_runner.go	Context-aware CRUD, typed API calls, new flatten/expand helpers; introduces GoString() state bug and nil-runner dereferences on the 202/AcceptedError paths.
github/resource_github_actions_hosted_runner_test.go	Tests migrated to state/plan checks, unique subtest names, shared runner group, updated import-verify ignore list.
github/acc_test.go	Adds organization runner-group sweeper and helper; contains a redundant nested if err != nil.
github/acc_helpers_test.go	Adds mustCreateTestOrganizationActionsRunnerGroup helper with cleanup.
GNUmakefile	Broadens default SWEEP to all and targets ./github for the sweep run.

Key findings

• HIGH — Delete passes runner.GetID() to waitForRunnerDeletion, but runner is nil on the async 202 path, so it polls runner ID 0, returns "deleted" immediately, and skips the real wait.
• MEDIUM — last_active_on is set from Timestamp.GoString(), writing github.Timestamp{...} into state instead of the documented RFC3339 value (3 locations).
• MEDIUM — Create sets the resource ID from a nil runner on the 202 path, producing ID "0" and empty computed state.