fix!: Fix LicenseStatus response and Supportkey type

github/enterprise_manage_ghes_config.go

@@ -70,6 +70,8 @@
 }
 
 // LicenseStatus is a struct to hold the response from the License API.
+// SupportKey is documented as string but is actual a bool.
+// TODO: Remove comment after github updates schema documentation
 type LicenseStatus struct {
 	AdvancedSecurityEnabled      *bool      `json:"advancedSecurityEnabled,omitempty"`
 	AdvancedSecuritySeats        *int       `json:"advancedSecuritySeats,omitempty"`
@@ -87,7 +89,7 @@
 	ReferenceNumber              *string    `json:"referenceNumber,omitempty"`
 	Seats                        *int       `json:"seats,omitempty"`
 	SSHAllowed                   *bool      `json:"sshAllowed,omitempty"`
-	SupportKey                   *string    `json:"supportKey,omitempty"`
+	SupportKey                   *bool      `json:"supportKey,omitempty"`
 	UnlimitedSeating             *bool      `json:"unlimitedSeating,omitempty"`
 }
 
@@ -353,15 +355,17 @@
 //
 // 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) {
+// Current Docs shouw incorrect return type. They list as [{...}] but actual is {...}
+// TODO: Remove comment after github updates schema documentation
+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)
 	}