fix!: Fix LicenseStatus response and Supportkey type

github/enterprise_manage_ghes_config.go

@@ -87,8 +87,10 @@ type LicenseStatus struct {
 	ReferenceNumber              *string    `json:"referenceNumber,omitempty"`
 	Seats                        *int       `json:"seats,omitempty"`
 	SSHAllowed                   *bool      `json:"sshAllowed,omitempty"`
-	SupportKey                   *string    `json:"supportKey,omitempty"`
-	UnlimitedSeating             *bool      `json:"unlimitedSeating,omitempty"`
+	// SupportKey is documented as a string, but the actual response is a bool.
+	// TODO: Remove this note once GitHub corrects the schema documentation.
+	SupportKey       *bool `json:"supportKey,omitempty"`
+	UnlimitedSeating *bool `json:"unlimitedSeating,omitempty"`
 }
 
 // UploadLicenseOptions is a struct to hold the options for the UploadLicense API.
@@ -351,17 +353,21 @@ func (s *EnterpriseService) InitialConfig(ctx context.Context, license, password
 
 // License gets the current license information for the GitHub Enterprise instance.
 //
+// NOTE: The GitHub documentation incorrectly shows the return type as a list ([{...}]),
+// but the actual response is a single object ({...}).
+// TODO: Remove this note once GitHub corrects the schema documentation.
+//
 // GitHub API docs: https://docs.github.com/enterprise-server@3.21/rest/enterprise-admin/manage-ghes#get-the-enterprise-license-information
 //
 //meta:operation GET /manage/v1/config/license
-func (s *EnterpriseService) License(ctx context.Context) ([]*LicenseStatus, *Response, error) {
+func (s *EnterpriseService) License(ctx context.Context) (*LicenseStatus, *Response, error) {
 	u := "manage/v1/config/license"
 	req, err := s.client.NewRequest(ctx, "GET", u, nil)
 	if err != nil {
 		return nil, nil, err
 	}
 
-	var licenseStatus []*LicenseStatus
+	var licenseStatus *LicenseStatus
 	resp, err := s.client.Do(req, &licenseStatus)
 	if err != nil {
 		return nil, resp, err

github/enterprise_manage_ghes_config_test.go

@@ -407,7 +407,7 @@ func TestEnterpriseService_License(t *testing.T) {
 
 	mux.HandleFunc("/manage/v1/config/license", func(w http.ResponseWriter, r *http.Request) {
 		testMethod(t, r, "GET")
-		fmt.Fprint(w, `[{
+		fmt.Fprint(w, `{
 			"advancedSecurityEnabled": true,
 			"advancedSecuritySeats": 0,
 			"clusterSupport": false,
@@ -424,9 +424,9 @@ func TestEnterpriseService_License(t *testing.T) {
 			"referenceNumber": "32a145",
 			"seats": 0,
 			"sshAllowed": true,
-			"supportKey": "",
+			"supportKey": true,
 			"unlimitedSeating": true
-		}]`)
+		}`)
 	})
 
 	ctx := t.Context()
@@ -435,7 +435,7 @@ func TestEnterpriseService_License(t *testing.T) {
 		t.Errorf("Enterprise.License returned error: %v", err)
 	}
 
-	want := []*LicenseStatus{{
+	want := &LicenseStatus{
 		AdvancedSecurityEnabled:      Ptr(true),
 		AdvancedSecuritySeats:        Ptr(0),
 		ClusterSupport:               Ptr(false),
@@ -452,9 +452,9 @@ func TestEnterpriseService_License(t *testing.T) {
 		ReferenceNumber:              Ptr("32a145"),
 		Seats:                        Ptr(0),
 		SSHAllowed:                   Ptr(true),
-		SupportKey:                   Ptr(""),
+		SupportKey:                   Ptr(true),
 		UnlimitedSeating:             Ptr(true),
-	}}
+	}
 	if diff := cmp.Diff(want, license); diff != "" {
 		t.Errorf("diff mismatch (-want +got):\n%v", diff)
 	}