add basic tests for website & dns targets, fix link with slash

build.sh

@@ -7,6 +7,10 @@ BINARY_NAME=stoppropaganda
 mkdir -p ./dist
 rm -rf ./dist/*
 
+# Run tests
+go test ./...
+
+# Build binaries
 env CGO_ENABLED=0 GOOS=linux GOARCH=386 go build -ldflags="-s -w" -o "dist/${BINARY_NAME}_v${VERSION}_linux_i386" cmd/stoppropaganda/main.go             # Linux i386
 env CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -ldflags="-s -w" -o "dist/${BINARY_NAME}_v${VERSION}_linux_x86_64" cmd/stoppropaganda/main.go         # Linux 64bit
 env CGO_ENABLED=0 GOOS=linux GOARCH=arm GOARM=5 go build -ldflags="-s -w" -o "dist/${BINARY_NAME}_v${VERSION}_linux_arm" cmd/stoppropaganda/main.go      # Linux armv5/armel/arm (it also works on armv6)

internal/targets/dns_test.go

@@ -0,0 +1,14 @@
+package targets
+
+import (
+	"strings"
+	"testing"
+)
+
+func TestDNSTargets(t *testing.T) {
+	for k := range TargetDNSServers {
+		if !strings.HasSuffix(k, ":53") {
+			t.Errorf("Invalid DNS target '%s'", k)
+		}
+	}
+}

internal/targets/websites.go

@@ -291,7 +291,7 @@ var TargetWebsites = map[string]struct{}{
 	"https://www.belapb.by":               {},
 	"https://bankdabrabyt.by":             {},
 	"https://belinvestbank.by/individual": {},
-	"https://api.belinvestbank.by/":       {},
+	"https://api.belinvestbank.by":        {},
 	"https://belpost.by":                  {},
 
 	// by business

internal/targets/websites_test.go

@@ -0,0 +1,19 @@
+package targets
+
+import (
+	"net/url"
+	"strings"
+	"testing"
+)
+
+func TestWebsitesLinks(t *testing.T) {
+	for k := range TargetWebsites {
+		if strings.HasSuffix(k, "/") {
+			t.Errorf("Invalid website '%s': ends with slash", k)
+		}
+		_, err := url.ParseRequestURI(k)
+		if err != nil {
+			t.Errorf("Invalid website '%v':", err)
+		}
+	}
+}