Skip to content

Commit ff3a8ce

Browse files
vflauxtroll-os
authored andcommitted
chore(cloudflare): remove unused funcs (kubernetes-sigs#5553)
1 parent dc40613 commit ff3a8ce

File tree

2 files changed

+0
-33
lines changed

2 files changed

+0
-33
lines changed

provider/cloudflare/cloudflare.go

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -112,9 +112,7 @@ var recordTypeCustomHostnameSupported = map[string]bool{
112112

113113
// cloudFlareDNS is the subset of the CloudFlare API that we actually use. Add methods as required. Signatures must match exactly.
114114
type cloudFlareDNS interface {
115-
UserDetails(ctx context.Context) (cloudflare.User, error)
116115
ZoneIDByName(zoneName string) (string, error)
117-
ListZones(ctx context.Context, zoneID ...string) ([]cloudflare.Zone, error)
118116
ListZonesContext(ctx context.Context, opts ...cloudflare.ReqOption) (cloudflare.ZonesResponse, error)
119117
ZoneDetails(ctx context.Context, zoneID string) (cloudflare.Zone, error)
120118
ListDNSRecords(ctx context.Context, rc *cloudflare.ResourceContainer, rp cloudflare.ListDNSRecordsParams) ([]cloudflare.DNSRecord, *cloudflare.ResultInfo, error)
@@ -133,14 +131,6 @@ type zoneService struct {
133131
service *cloudflare.API
134132
}
135133

136-
func (z zoneService) UserDetails(ctx context.Context) (cloudflare.User, error) {
137-
return z.service.UserDetails(ctx)
138-
}
139-
140-
func (z zoneService) ListZones(ctx context.Context, zoneID ...string) ([]cloudflare.Zone, error) {
141-
return z.service.ListZones(ctx, zoneID...)
142-
}
143-
144134
func (z zoneService) ZoneIDByName(zoneName string) (string, error) {
145135
return z.service.ZoneIDByName(zoneName)
146136
}

provider/cloudflare/cloudflare_test.go

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@ type MockAction struct {
4747
}
4848

4949
type mockCloudFlareClient struct {
50-
User cloudflare.User
5150
Zones map[string]string
5251
Records map[string]map[string]cloudflare.DNSRecord
5352
Actions []MockAction
@@ -88,7 +87,6 @@ var ExampleDomain = []cloudflare.DNSRecord{
8887

8988
func NewMockCloudFlareClient() *mockCloudFlareClient {
9089
return &mockCloudFlareClient{
91-
User: cloudflare.User{ID: "xxxxxxxxxxxxxxxxxxx"},
9290
Zones: map[string]string{
9391
"001": "bar.com",
9492
"002": "foo.com",
@@ -285,10 +283,6 @@ func (m *mockCloudFlareClient) DeleteDNSRecord(ctx context.Context, rc *cloudfla
285283
return nil
286284
}
287285

288-
func (m *mockCloudFlareClient) UserDetails(ctx context.Context) (cloudflare.User, error) {
289-
return m.User, nil
290-
}
291-
292286
func (m *mockCloudFlareClient) CustomHostnames(ctx context.Context, zoneID string, page int, filter cloudflare.CustomHostname) ([]cloudflare.CustomHostname, cloudflare.ResultInfo, error) {
293287
var err error = nil
294288
perPage := 50 // cloudflare-go v0 API hardcoded
@@ -372,23 +366,6 @@ func (m *mockCloudFlareClient) ZoneIDByName(zoneName string) (string, error) {
372366
return "", errors.New("Unknown zone: " + zoneName)
373367
}
374368

375-
func (m *mockCloudFlareClient) ListZones(ctx context.Context, zoneID ...string) ([]cloudflare.Zone, error) {
376-
if m.listZonesError != nil {
377-
return nil, m.listZonesError
378-
}
379-
380-
result := []cloudflare.Zone{}
381-
382-
for zoneID, zoneName := range m.Zones {
383-
result = append(result, cloudflare.Zone{
384-
ID: zoneID,
385-
Name: zoneName,
386-
})
387-
}
388-
389-
return result, nil
390-
}
391-
392369
func (m *mockCloudFlareClient) ListZonesContext(ctx context.Context, opts ...cloudflare.ReqOption) (cloudflare.ZonesResponse, error) {
393370
if m.listZonesContextError != nil {
394371
return cloudflare.ZonesResponse{}, m.listZonesContextError

0 commit comments

Comments
 (0)