feat!: Add pagination to Enterprise.ListBudgets and add GetUserStatesForBudget (#4408)

[devaniketh]

BREAKING CHANGE: EnterpriseService.ListBudgets now takes EnterpriseListBudgetsOptions.

Fixes #4408

Summary

• Added EnterpriseListBudgetsOptions embedding ListOptions to enable pagination (page, per_page) in EnterpriseService.ListBudgets.
• Added EnterpriseService.GetUserStatesForBudget along with EnterpriseBudgetUserState, EnterpriseBudgetUserStates, and EnterpriseGetUserStatesOptions for endpoint GET /enterprises/{enterprise}/settings/billing/budgets/{budget_id}/user-states.
• Regenerated accessors, iterators (ListBudgetsIter), and stringifier test fixtures via script/generate.sh.
• Added full unit test coverage in enterprise_budgets_test.go.

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.53%. Comparing base (b8acf75) to head (009c461).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #4501   +/-   ##
=======================================
  Coverage   98.53%   98.53%           
=======================================
  Files         195      195           
  Lines       17795    17848   +53     
=======================================
+ Hits        17534    17587   +53     
  Misses        261      261           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[devaniketh]

Thanks for the review! @Not-Dhananjay-Mishra

I've updated:

1.EnterpriseBudgetUserState to use float64 for ConsumedAmount and TargetAmount without omitempty.
2. EnterpriseBudgetUserStates to use bool and int for HasNextPage and TotalCount without omitempty.
3.Added ListBudgetsOptions with Scope, User, and embedded ListOptions, and updated ListBudgets / ListBudgetsIter.
4. Fixed the spacing after fmt.Sprintf and regenerated all accessors, iterators, and stringifier test fixtures.

[gmlewis]

Thank you, @devaniketh.
testBadOptions always needs a non-nil value for opts to be passed in for it to work properly.

[Not-Dhananjay-Mishra]

Iterator for EnterpriseService.GetUserStatesForBudget isn't generated, Since it accepts ListOptions and is an exception case can you please update this

go-github/github/gen-iterators.go

Lines 165 to 170 in 1217174

	// customNames provides custom names for iterator methods where the default methodName + "Iter" would be confusing.
	var customNames = map[string]string{
	"RepositoriesService.GetCommit": "ListCommitFiles",
	"RepositoriesService.CompareCommits": "ListCommitComparisonFiles",
	"RepositoriesService.GetCombinedStatus": "ListCombinedStatus",
	}

[devaniketh]

Thank you for pointing that out! @gmlewis I've updated both testBadOptions calls in enterprise_budgets_test.go to pass opts instead of nil. All tests and linters are passing cleanly.

[gmlewis]

Thank you, @devaniketh!
LGTM.

cc: @Not-Dhananjay-Mishra

[Not-Dhananjay-Mishra]

LGTM

[gmlewis]

Thank you, @Not-Dhananjay-Mishra!
Merging.