race and linter fixes (#291)

* run ci on every push

* error strings should not be capitalized or end with punctuation or a newline

* skip flaky dns tests locally, move in-place domain sort to main

* make it easier to clean up if test bails midway

* testing

* ruby 3.1?

* new gh action

* one go only

* skip

* go back

Author: nonrational

.github/workflows/ci.yml

@@ -1,27 +1,23 @@
 name: ci
 
-on:
-  push:
-    branches: [master]
-  pull_request:
-    branches: [master]
+on: push
 
 jobs:
   test:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
         os: [macos-latest, macos-11.0, ubuntu-latest]
-        go-version: [1.16.6, 1.17.2]
-        ruby-version: [2.7]
+        go-version: [1.17.2]
+        ruby-version: [3.1]
 
     name: ${{ matrix.os }} / go-${{ matrix.go-version }}
     steps:
       - uses: actions/setup-go@v1
         with:
           go-version: ${{ matrix.go-version }}
 
-      - uses: actions/setup-ruby@v1
+      - uses: ruby/setup-ruby@v1
         with:
           ruby-version: ${{ matrix.ruby-version }}
 

Makefile

@@ -29,11 +29,11 @@ release:
 		zip -v "pkg/puma-dev-$$RELEASE-darwin-$$arch.zip" puma-dev; \
 	done
 
-test:
+test: clean-test
 	go test -v -race -coverprofile=coverage.out -covermode=atomic ./...
 
 clean-test:
-	rm -rf ~/.gotest-macos-puma-dev
+	rm -rf $$HOME/.puma-dev-test_*
 
 test-macos-filesystem-setup:
 	sudo mkdir -p /etc/resolver;

cmd/puma-dev/command.go

@@ -16,7 +16,7 @@ func command() error {
 	case "link":
 		return link()
 	default:
-		return fmt.Errorf("Unknown command: %s\n", flag.Arg(0))
+		return fmt.Errorf("unknown command: %s", flag.Arg(0))
 	}
 }
 
@@ -46,14 +46,14 @@ func link() error {
 		stat, err := os.Stat(dir)
 		if err != nil {
 			if os.IsNotExist(err) {
-				return fmt.Errorf("Invalid directory: %s", dir)
+				return fmt.Errorf("invalid directory: %s", dir)
 			}
 
 			return err
 		}
 
 		if !stat.IsDir() {
-			return fmt.Errorf("Invalid directory: %s", dir)
+			return fmt.Errorf("invalid directory: %s", dir)
 		}
 	}
 

cmd/puma-dev/command_test.go

@@ -13,13 +13,13 @@ import (
 func TestCommand_noCommandArg(t *testing.T) {
 	StubCommandLineArgs()
 	err := command()
-	assert.Equal(t, "Unknown command: \n", err.Error())
+	assert.Equal(t, "unknown command: ", err.Error())
 }
 
 func TestCommand_badCommandArg(t *testing.T) {
 	StubCommandLineArgs("doesnotexist")
 	err := command()
-	assert.Equal(t, "Unknown command: doesnotexist\n", err.Error())
+	assert.Equal(t, "unknown command: doesnotexist", err.Error())
 }
 
 func TestCommand_link_noArgs(t *testing.T) {
@@ -66,7 +66,7 @@ func TestCommand_link_invalidDirectory(t *testing.T) {
 
 	err := command()
 
-	assert.Equal(t, "Invalid directory: /this/path/does/not/exist", err.Error())
+	assert.Equal(t, "invalid directory: /this/path/does/not/exist", err.Error())
 }
 
 func TestCommand_link_reassignExistingApp(t *testing.T) {

cmd/puma-dev/main.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"os"
 	"runtime"
+	"strings"
 )
 
 var (
@@ -21,6 +22,14 @@ type CommandResult struct {
 	shouldExit     bool
 }
 
+type ByDecreasingTLDComplexity []string
+
+func (a ByDecreasingTLDComplexity) Len() int      { return len(a) }
+func (a ByDecreasingTLDComplexity) Swap(i, j int) { a[i], a[j] = a[j], a[i] }
+func (a ByDecreasingTLDComplexity) Less(i, j int) bool {
+	return strings.Count(a[i], ".") > strings.Count(a[j], ".")
+}
+
 func allCheck() {
 	if result := execWithExitStatus(); result.shouldExit {
 		os.Exit(result.exitStatusCode)

cmd/puma-dev/main_darwin.go

@@ -6,6 +6,7 @@ import (
 	"log"
 	"os"
 	"os/signal"
+	"sort"
 	"strings"
 	"syscall"
 	"time"
@@ -54,6 +55,7 @@ func main() {
 	}
 
 	domains := strings.Split(*fDomains, ":")
+	sort.Sort(ByDecreasingTLDComplexity(domains))
 
 	if *fCleanup {
 		dev.Cleanup()

cmd/puma-dev/main_darwin_test.go

@@ -19,12 +19,12 @@ import (
 )
 
 func TestMainPumaDev_Darwin(t *testing.T) {
-	appLinkDir := homedir.MustExpand("~/.gotest-macos-puma-dev")
+	appLinkDir := homedir.MustExpand("~/.puma-dev-test_macos-puma-dev")
 
 	defer LinkAllTestApps(t, appLinkDir)()
 
 	serveErr := configureAndBootPumaDevServer(t, map[string]string{
-		"d":                     "test:puma",
+		"d":                     "test:puma:puma.dev",
 		"dir":                   appLinkDir,
 		"dns-port":              "65053",
 		"http-port":             "65080",
@@ -57,7 +57,14 @@ func TestMainPumaDev_Darwin(t *testing.T) {
 		assert.NoError(t, err)
 		assert.Equal(t, net.ParseIP("127.0.0.1").To4(), ips[0].IP.To4())
 
-		_, err = r.LookupIPAddr(ctx, "foo.tlddoesnotexist")
+		ips, err = r.LookupIPAddr(ctx, "foo.puma.dev")
+		assert.NoError(t, err)
+		assert.Equal(t, net.ParseIP("127.0.0.1").To4(), ips[0].IP.To4())
+
+		_, err = r.LookupIPAddr(ctx, "foo.dev")
+		assert.Error(t, err)
+
+		_, err = r.LookupIPAddr(ctx, "foo.dev-does-not-exist")
 		assert.Error(t, err)
 	})
 }

cmd/puma-dev/main_linux.go

@@ -6,6 +6,7 @@ import (
 	"log"
 	"os"
 	"os/signal"
+	"sort"
 	"strings"
 	"syscall"
 	"time"
@@ -32,6 +33,7 @@ func main() {
 	allCheck()
 
 	domains := strings.Split(*fDomains, ":")
+	sort.Sort(ByDecreasingTLDComplexity(domains))
 
 	if *fStop {
 		err := dev.Stop()

cmd/puma-dev/main_linux_test.go

@@ -8,7 +8,7 @@ import (
 )
 
 func TestMainPumaDev_Linux(t *testing.T) {
-	appLinkDir := homedir.MustExpand("~/.gotest-linux-puma-dev")
+	appLinkDir := homedir.MustExpand("~/.puma-dev-test_linux-puma-dev")
 
 	defer LinkAllTestApps(t, appLinkDir)()
 

cmd/puma-dev/main_test.go

@@ -63,7 +63,7 @@ func TestMain_execWithExitStatus_commandArgs(t *testing.T) {
 		assert.Equal(t, true, result.shouldExit)
 	})
 
-	assert.Equal(t, "Error: Unknown command: nosoupforyou\n\n", execStdOut)
+	assert.Equal(t, "Error: unknown command: nosoupforyou\n", execStdOut)
 }
 
 func TestMain_allCheck_versionFlag(t *testing.T) {