Merge pull request #7 from l3uddz/develop

Develop

Makefile

@@ -30,7 +30,7 @@ test-all: test lint
 .PHONY: test
 test: ## Run tests
 	@echo "*** go test ***"
-	go test -cover -mod vendor -v -race ${GO_PACKAGES}
+	go test -cover -v -race ${GO_PACKAGES}
 
 .PHONY: lint
 lint: check_golangci ## Run linting

pvr/pvr.go

@@ -13,14 +13,14 @@ var (
 	pvrDefaultPageSize = 1000
 	pvrDefaultTimeout  = 120
 	pvrDefaultRetry    = web.Retry{
-		MaxAttempts: 5,
+		MaxAttempts: 6,
 		RetryableStatusCodes: []int{
 			504,
 		},
 		Backoff: backoff.Backoff{
 			Jitter: true,
-			Min:    1 * time.Second,
-			Max:    5 * time.Second,
+			Min:    500 * time.Millisecond,
+			Max:    10 * time.Second,
 		},
 	}
 )

utils/web/request_test.go

@@ -0,0 +1,20 @@
+package web
+
+import (
+	"os"
+	"testing"
+)
+
+/* Test Get Response Timeout */
+
+func TestGetResponseTimeout(t *testing.T) {
+	// send request
+	resp, err := GetResponse(GET, "https://httpbin.davecheney.com/delay/5", 3)
+	if err != nil && !os.IsTimeout(err) {
+		t.Errorf("Expected timeout in 3 seconds but got error: %v", err)
+		return
+	} else if err == nil {
+		defer resp.Response().Body.Close()
+		t.Errorf("Expected timeout in 3 seconds but got no error...")
+	}
+}