Merge pull request #101 from Henry-Sarabia/iss-100

Iss 100
Henry-Sarabia · Dec 15, 2020 · 8ae2243 · 8ae2243
2 parents e41b178 + da175e4
commit 8ae2243
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/options.go b/options.go
@@ -94,7 +94,7 @@ func SetOrder(field string, order order) Option {
 // For more information, visit: https://api-docs.igdb.com/#pagination
 func SetLimit(lim int) Option {
 	return func() (apicalypse.Option, error) {
-		if lim <= 0 || lim > 5000 {
+		if lim <= 0 || lim > 500 {
 			return nil, ErrOutOfRange
 		}
 
@@ -103,12 +103,12 @@ func SetLimit(lim int) Option {
 }
 
 // SetOffset is a functional option used to offset the results from an API
-// call. The default offset is 0. The maximum offest is 5000.
+// call. The default offset is 0.
 //
 // For more information, visit: https://api-docs.igdb.com/#pagination
 func SetOffset(off int) Option {
 	return func() (apicalypse.Option, error) {
-		if off < 0 || off > 5000 {
+		if off < 0 {
 			return nil, ErrOutOfRange
 		}
 

diff --git a/options_test.go b/options_test.go
@@ -2,11 +2,12 @@ package igdb
 
 import (
 	"fmt"
-	"github.com/Henry-Sarabia/apicalypse"
-	"github.com/pkg/errors"
 	"log"
 	"strings"
 	"testing"
+
+	"github.com/Henry-Sarabia/apicalypse"
+	"github.com/pkg/errors"
 )
 
 func TestComposeOptions(t *testing.T) {
@@ -159,7 +160,7 @@ func TestSetLimit(t *testing.T) {
 		{"Limit within range", 5, "5", nil},
 		{"Zero limit", 0, "", ErrOutOfRange},
 		{"Limit below range", -10, "", ErrOutOfRange},
-		{"Limit above range", 5001, "", ErrOutOfRange},
+		{"Limit above range", 501, "", ErrOutOfRange},
 	}
 
 	for _, test := range tests {