From f205b5824c9ed887674e1c6769fc7e031c1af20c Mon Sep 17 00:00:00 2001 From: Eduard Urbach Date: Wed, 6 Mar 2019 11:02:49 +0900 Subject: [PATCH] Improved testing --- .travis.yml | 26 ++++++++++++++------------ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/.travis.yml b/.travis.yml index 3511d253..7a28503e 100644 --- a/.travis.yml +++ b/.travis.yml @@ -26,7 +26,7 @@ matrix: before_script: - GO_FILES=$(find . -iname '*.go' -type f | grep -v /vendor/) # All the .go files, excluding vendor/ - go get golang.org/x/lint/golint # Linter - - go get honnef.co/go/tools/cmd/megacheck # Badass static analyzer/linter + - go get honnef.co/go/tools/cmd/staticcheck # Badass static analyzer/linter - go get github.com/fzipp/gocyclo # Cyclomatic complexity - go get github.com/3rf/codecoroner # Dead code analysis - go get github.com/dustin/go-humanize # Dependency used in pixy files only @@ -36,14 +36,16 @@ before_script: # script always run to completion (set +e). All of these code checks are must haves # in a modern Go project. script: - - make deps # Get dependencies. Allow this to fail because components cannot be found. - - make all # Build -# - test -z $(gofmt -s -l $GO_FILES) # Fail if a .go file hasn't been formatted with gofmt - - go test -v . # Run all the tests -# - go test -v -race ./... # Run all the tests with the race detector enabled - - go vet ./... # go vet is the official Go static analyzer - - megacheck ./... # "go vet on steroids" + linter - - gocyclo -over 19 $GO_FILES # forbid code with huge functions - - golint -set_exit_status $(go list ./...) # one last linter - - codecoroner -ignore components funcs ./... # dead code analysis - - curl -s https://codecov.io/bash \ No newline at end of file + - make deps # Get dependencies. Allow this to fail because components cannot be found. + - make all # Build +# - test -z $(gofmt -s -l $GO_FILES) # Fail if a .go file hasn't been formatted with gofmt + - go test -v . # Run all the tests +# - go test -v -race ./... # Run all the tests with the race detector enabled + - go vet ./... # go vet is the official Go static analyzer + - staticcheck -checks all,-ST1000,-ST1005 ./... # static analysis + - gocyclo -over 19 $GO_FILES # forbid code with huge functions + - golint -set_exit_status $(go list ./...) # one last linter + - codecoroner -ignore components funcs ./... # dead code analysis + +after_success: + - bash <(curl -s https://codecov.io/bash) \ No newline at end of file