Merge pull request #162 from nhooyr/coveralls

Switch to coveralls.io from codecov.io

Author: nhooyr

.github/CONTRIBUTING.md

@@ -28,7 +28,7 @@ CI must pass on your changes for them to be merged.
 ### CI
 
 CI will ensure your code is formatted, lints and passes tests.
-It will collect coverage and report it to [codecov](https://codecov.io/gh/nhooyr/websocket)
+It will collect coverage and report it to [coveralls](https://coveralls.io/github/nhooyr/websocket)
 and also upload a html `coverage` artifact that you can download to browse coverage.
 
 You can run CI locally.
@@ -42,7 +42,4 @@ See [ci/image/Dockerfile](../ci/image/Dockerfile) for the installation of the CI
 
 For coverage details locally, see `ci/out/coverage.html` after running `make test`.
 
-You can also run tests normally with `go test`. `make test` just passes a default set of flags to
-`go test` to collect coverage and runs the WASM tests.
-
-Coverage percentage from codecov and the CI scripts will be different because they are calculated differently.
+You can run tests normally with `go test`. `make test` wraps around `go test` to collect coverage.

.github/workflows/ci.yml

@@ -1,27 +1,27 @@
 name: ci
-on: [push]
+on: [push, pull_request]
 
 jobs:
   fmt:
     runs-on: ubuntu-latest
-    container: nhooyr/websocket-ci@sha256:ea94e078d2d589d654a2c759d952bf4199c754d80dadb20696dc3902359027cb
+    container: nhooyr/websocket-ci@sha256:8a8fd73fdea33585d50a33619c4936adfd016246a2ed6bbfbf06def24b518a6a
     steps:
       - uses: actions/checkout@v1
       - run: make fmt
   lint:
     runs-on: ubuntu-latest
-    container: nhooyr/websocket-ci@sha256:ea94e078d2d589d654a2c759d952bf4199c754d80dadb20696dc3902359027cb
+    container: nhooyr/websocket-ci@sha256:8a8fd73fdea33585d50a33619c4936adfd016246a2ed6bbfbf06def24b518a6a
     steps:
       - uses: actions/checkout@v1
       - run: make lint
   test:
     runs-on: ubuntu-latest
-    container: nhooyr/websocket-ci@sha256:ea94e078d2d589d654a2c759d952bf4199c754d80dadb20696dc3902359027cb
+    container: nhooyr/websocket-ci@sha256:8a8fd73fdea33585d50a33619c4936adfd016246a2ed6bbfbf06def24b518a6a
     steps:
       - uses: actions/checkout@v1
       - run: make test
         env:
-          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+          COVERALLS_TOKEN: ${{ secrets.github_token }}
       - name: Upload coverage.html
         uses: actions/upload-artifact@master
         with:

Makefile

@@ -4,10 +4,14 @@ all: fmt lint test
 
 .PHONY: *
 
+.ONESHELL:
+SHELL = bash
+.SHELLFLAGS = -ceuo pipefail
+
 include ci/fmt.mk
 include ci/lint.mk
 include ci/test.mk
 
 ci-image:
-	docker build -f ./ci/image/Dockerfile -t nhooyr/websocket-ci .
+	docker build -f ./ci/Dockerfile -t nhooyr/websocket-ci .
 	docker push nhooyr/websocket-ci

README.md

@@ -1,8 +1,8 @@
 # websocket
 
-[![GitHub release (latest SemVer)](https://img.shields.io/github/v/release/nhooyr/websocket?color=6b9ded&sort=semver)](https://github.com/nhooyr/websocket/releases)
+[![GitHub Release](https://img.shields.io/github/v/release/nhooyr/websocket?color=6b9ded&sort=semver)](https://github.com/nhooyr/websocket/releases)
 [![GoDoc](https://godoc.org/nhooyr.io/websocket?status.svg)](https://godoc.org/nhooyr.io/websocket)
-[![Codecov](https://img.shields.io/codecov/c/github/nhooyr/websocket.svg?color=65d6a4)](https://codecov.io/gh/nhooyr/websocket)
+[![Coveralls](https://img.shields.io/coveralls/github/nhooyr/websocket?color=65d6a4)](https://coveralls.io/github/nhooyr/websocket)
 [![Actions Status](https://github.com/nhooyr/websocket/workflows/ci/badge.svg)](https://github.com/nhooyr/websocket/actions)
 
 websocket is a minimal and idiomatic WebSocket library for Go.

ci/.codecov.yml

@@ -3,7 +3,7 @@ FROM golang:1
 RUN apt-get update
 RUN apt-get install -y chromium
 RUN apt-get install -y npm
-RUN apt-get install -y shellcheck
+RUN apt-get install -y jq
 
 ENV GOPATH=/root/gopath
 ENV PATH=$GOPATH/bin:$PATH
@@ -12,15 +12,12 @@ ENV PAGER=cat
 ENV CI=true
 ENV MAKEFLAGS="--jobs=8 --output-sync=target"
 
-COPY ./ci/image/gitignore /root/.config/git/ignore
-RUN git config --system color.ui always
-
 RUN npm install -g prettier
 RUN go get golang.org/x/tools/cmd/stringer
 RUN go get golang.org/x/tools/cmd/goimports
-RUN go get mvdan.cc/sh/cmd/shfmt
 RUN go get golang.org/x/lint/golint
 RUN go get github.com/agnivade/wasmbrowsertest
+RUN go get github.com/mattn/goveralls
 
 # Cache go modules and build cache.
 COPY . /tmp/websocket

ci/image/Dockerfile renamed to ci/Dockerfile

@@ -1,6 +1,12 @@
-fmt: modtidy gofmt goimports prettier shfmt
+fmt: modtidy gofmt goimports prettier
 ifdef CI
-	./ci/fmtcheck.sh
+	if [[ $$(git ls-files --other --modified --exclude-standard) != "" ]]; then
+	  echo "Files need generation or are formatted incorrectly:"
+	  git -c color.ui=always status | grep --color=no '\e\[31m'
+	  echo "Please run the following locally:"
+	  echo "  make fmt"
+	  exit 1
+	fi
 endif
 
 modtidy: gen
@@ -12,11 +18,8 @@ gofmt: gen
 goimports: gen
 	goimports -w "-local=$$(go list -m)" .
 
-prettier: gen
-	prettier --write --print-width=120 --no-semi --trailing-comma=all --loglevel=warn $$(git ls-files "*.yaml" "*.yml" "*.md" "*.ts")
-
-shfmt: gen
-	shfmt -i 2 -w -s -sr .
+prettier:
+	prettier --write --print-width=120 --no-semi --trailing-comma=all --loglevel=warn $$(git ls-files "*.yml" "*.md")
 
 gen:
 	go generate ./...

ci/fmt.mk

@@ -1,4 +1,4 @@
-lint: govet golint govet-wasm golint-wasm shellcheck
+lint: govet golint govet-wasm golint-wasm
 
 govet:
 	go vet ./...
@@ -11,6 +11,3 @@ golint:
 
 golint-wasm:
 	GOOS=js GOARCH=wasm golint -set_exit_status ./...
-
-shellcheck:
-	shellcheck -x $$(git ls-files "*.sh")

ci/fmtcheck.sh

@@ -1,18 +1,23 @@
-test: gotest
-
-gotest: _gotest htmlcov
+test: gotest ci/out/coverage.html
 ifdef CI
-gotest: codecov
+test: coveralls
 endif
 
-htmlcov: _gotest
+ci/out/coverage.html: gotest
 	go tool cover -html=ci/out/coverage.prof -o=ci/out/coverage.html
 
-codecov: _gotest
-	curl -s https://codecov.io/bash | bash -s -- -Z -f ci/out/coverage.prof
-
-_gotest:
-	go test -parallel=32 -coverprofile=ci/out/coverage.prof -coverpkg=./... $$TESTFLAGS ./...
+coveralls: gotest
+	# https://github.com/coverallsapp/github-action/blob/master/src/run.ts
+	echo "--- coveralls"
+	export GIT_BRANCH="$$GITHUB_REF"
+	export BUILD_NUMBER="$$GITHUB_SHA"
+	if [[ $$GITHUB_EVENT_NAME == pull_request ]]; then
+	  export CI_PULL_REQUEST="$$(jq .number "$$GITHUB_EVENT_PATH")"
+	  BUILD_NUMBER="$$BUILD_NUMBER-PR-$$CI_PULL_REQUEST"
+	fi
+	goveralls -coverprofile=ci/out/coverage.prof -service=github
+gotest:
+	go test -covermode=count -coverprofile=ci/out/coverage.prof -coverpkg=./... $${GOTESTFLAGS-} ./...
 	sed -i '/_stringer\.go/d' ci/out/coverage.prof
 	sed -i '/wsecho\.go/d' ci/out/coverage.prof
 	sed -i '/assert\.go/d' ci/out/coverage.prof

ci/image/gitignore

@@ -1023,14 +1023,7 @@ func TestAutobahn(t *testing.T) {
 		t.Run(name, func(t *testing.T) {
 			t.Parallel()
 
-			t.Run("server", func(t *testing.T) {
-				t.Parallel()
-				run2(t, false)
-			})
-			t.Run("client", func(t *testing.T) {
-				t.Parallel()
-				run2(t, true)
-			})
+			run2(t, true)
 		})
 	}