DE-1392 Remove deprecated GetStats

events.go

@@ -10,7 +10,7 @@ import (
 	"github.com/mailgun/mailgun-go/v4/events"
 )
 
-// ListEventOptions{} modifies the behavior of ListEvents()
+// ListEventOptions modifies the behavior of ListEvents()
 type ListEventOptions struct {
 	// Limits the results to a specific start and end time
 	Begin, End time.Time

examples/examples.go

@@ -572,17 +572,6 @@ func ListRoutes(domain, apiKey string) ([]mailgun.Route, error) {
 	return result, nil
 }
 
-func GetStats(domain, apiKey string) ([]mailgun.Stats, error) {
-	mg := mailgun.NewMailgun(domain, apiKey)
-
-	ctx, cancel := context.WithTimeout(context.Background(), time.Second*30)
-	defer cancel()
-
-	return mg.GetStats(ctx, []string{"accepted", "delivered", "failed"}, &mailgun.GetStatOptions{
-		Duration: "1m",
-	})
-}
-
 func ListTags(domain, apiKey string) ([]mailgun.Tag, error) {
 	mg := mailgun.NewMailgun(domain, apiKey)
 	it := mg.ListTags(nil)

mailgun.go

@@ -96,14 +96,13 @@ var CaptureCurlOutput = false
 var RedactCurlAuth = false
 
 const (
-	// Base Url the library uses to contact mailgun. Use SetAPIBase() to override
+	// APIBase - base URL the library uses to contact mailgun. Use SetAPIBase() to override
 	APIBase              = "https://api.mailgun.net/v3"
 	APIBaseUS            = APIBase
 	APIBaseEU            = "https://api.eu.mailgun.net/v3"
 	messagesEndpoint     = "messages"
 	mimeMessagesEndpoint = "messages.mime"
 	bouncesEndpoint      = "bounces"
-	statsTotalEndpoint   = "stats/total"
 	metricsEndpoint      = "analytics/metrics"
 	domainsEndpoint      = "domains"
 	tagsEndpoint         = "tags"
@@ -151,8 +150,6 @@ type Mailgun interface {
 
 	ListMetrics(opts MetricsOptions) (*MetricsIterator, error)
 
-	// Deprecated: Use ListMetrics instead.
-	GetStats(ctx context.Context, events []string, opts *GetStatOptions) ([]Stats, error)
 	GetTag(ctx context.Context, tag string) (Tag, error)
 	DeleteTag(ctx context.Context, tag string) error
 	ListTags(*ListTagOptions) *TagIterator

messages.go

@@ -113,6 +113,7 @@ type MimeMessage struct {
 	body io.ReadCloser
 }
 
+// TODO(v5): return from Send()
 type sendMessageResponse struct {
 	Message string `json:"message"`
 	Id      string `json:"id"`
@@ -596,6 +597,7 @@ func (m *CommonMessage) Headers() map[string]string {
 // ErrInvalidMessage is returned by `Send()` when the `mailgun.CommonMessage` struct is incomplete
 var ErrInvalidMessage = errors.New("message not valid")
 
+// TODO(v5): rename to Message?
 type SendableMessage interface {
 	Domain() string
 	To() []string

mock.go

@@ -47,7 +47,6 @@ type mockServer struct {
 	complaints       []Complaint
 	bounces          []Bounce
 	credentials      []Credential
-	stats            []Stats
 	tags             []Tag
 	subaccountList   []Subaccount
 	webhooks         WebHooksListResponse
@@ -136,7 +135,6 @@ func NewMockServer() MockServer {
 		ms.addComplaintsRoutes(r)
 		ms.addBouncesRoutes(r)
 		ms.addCredentialsRoutes(r)
-		ms.addStatsRoutes(r)
 		ms.addTagsRoutes(r)
 		ms.addSubaccountRoutes(r)
 	})