Fix default values in help (#109)

* Fix default values in help

* Remove graceful handling of invalid argument in throttled http client

* Bump version

* Replace printHelp function

* Remove unused constant

* Ignore errcheck

.snapshots/TestPrintHelp → .snapshots/TestHelp

@@ -2,17 +2,21 @@ Usage:
   muffet.test [options] <url>
 
 Application Options:
-  -b, --buffer-size=              HTTP response buffer size in bytes
-  -c, --max-connections=          Maximum number of HTTP connections
+  -b, --buffer-size=              HTTP response buffer size in bytes (default:
+                                  4096)
+  -c, --max-connections=          Maximum number of HTTP connections (default:
+                                  512)
       --max-connections-per-host= Maximum number of HTTP connections per host
+                                  (default: 512)
   -e, --exclude=                  Exclude URLs matched with given regular
                                   expressions
       --follow-robots-txt         Follow robots.txt when scraping pages
       --follow-sitemap-xml        Scrape only pages listed in sitemap.xml
       --header=                   Custom headers
   -f, --ignore-fragments          Ignore URL fragments
-  -r, --max-redirections=         Maximum number of redirections
+  -r, --max-redirections=         Maximum number of redirections (default: 64)
   -t, --timeout=                  Timeout for HTTP requests in seconds
+                                  (default: 10)
   -v, --verbose                   Show successful results too
       --skip-tls-verification     Skip TLS certificate verification
       --one-page-only             Only check links found in the given URL

arguments.go

@@ -1,8 +1,8 @@
 package main
 
 import (
+	"bytes"
 	"errors"
-	"io"
 	"regexp"
 	"strings"
 
@@ -61,11 +61,17 @@ func getArguments(ss []string) (*arguments, error) {
 	return &args, nil
 }
 
-func printHelp(w io.Writer) {
+func help() string {
 	p := flags.NewParser(&arguments{}, flags.PassDoubleDash)
 	p.Usage = "[options] <url>"
 
-	p.WriteHelp(w)
+	// Parse() is run here to show default values in help.
+	// This seems to be a bug in go-flags.
+	p.Parse() // nolint:errcheck
+
+	b := &bytes.Buffer{}
+	p.WriteHelp(b)
+	return b.String()
 }
 
 func compileRegexps(regexps []string) ([]*regexp.Regexp, error) {

arguments_test.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"bytes"
 	"testing"
 
 	"github.com/bradleyjkemp/cupaloy"
@@ -60,10 +59,8 @@ func TestGetArgumentsError(t *testing.T) {
 	}
 }
 
-func TestPrintHelp(t *testing.T) {
-	b := &bytes.Buffer{}
-	printHelp(b)
-	cupaloy.SnapshotT(t, b.String())
+func TestHelp(t *testing.T) {
+	cupaloy.SnapshotT(t, help())
 }
 
 func TestParseHeaders(t *testing.T) {

command.go

@@ -34,7 +34,7 @@ func (c *command) runWithError(ss []string) (bool, error) {
 	if err != nil {
 		return false, err
 	} else if args.Help {
-		printHelp(c.stdout)
+		c.print(help())
 		return true, nil
 	} else if args.Version {
 		c.print(version)

configuration.go

@@ -3,9 +3,8 @@ package main
 import "time"
 
 const (
-	version               = "2.0.1"
-	agentName             = "muffet"
-	concurrency           = 1024
-	tcpTimeout            = time.Minute
-	defaultMaxConnections = 512
+	version     = "2.0.2"
+	agentName   = "muffet"
+	concurrency = 1024
+	tcpTimeout  = time.Minute
 )

throttled_http_client.go

@@ -8,10 +8,6 @@ type throttledHTTPClient struct {
 }
 
 func newThrottledHTTPClient(c httpClient, maxConnections int) httpClient {
-	if maxConnections < 1 {
-		maxConnections = defaultMaxConnections
-	}
-
 	return &throttledHTTPClient{c, newSemaphore(maxConnections)}
 }