Report for github.com/smartystreets/httpunit
B Not bad! Found 4 issues across 4 files
gofmt75%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
-
-
httpunit/dump.go
- Line 1: warning: file is not gofmted with -s (gofmt)
go_vet100%
go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.
No problems detected. Good job!
gocyclo100%
Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.
No problems detected. Good job!
golint0%
Golint is a linter for Go source code.
-
-
httpunit/dump.go
- Line 13: warning: exported type DumpHandler should have comment or be unexported (golint)
- Line 20: warning: exported function NewDumpHandler should have comment or be unexported (golint)
- Line 29: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 34: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 38: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 48: warning: exported method DumpHandler.Teardown should have comment or be unexported (golint)
- Line 48: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
-
-
httpunit/fixture.go
- Line 11: warning: exported type HTTPFixture should have comment or be unexported (golint)
- Line 19: warning: exported function NewFixture should have comment or be unexported (golint)
- Line 28: warning: exported method HTTPFixture.Teardown should have comment or be unexported (golint)
- Line 28: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 31: warning: exported method HTTPFixture.Serve should have comment or be unexported (golint)
- Line 31: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 35: warning: exported method HTTPFixture.DeserializeJSONResponseBody should have comment or be unexported (golint)
- Line 35: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 42: warning: exported method HTTPFixture.AssertJSONResponse should have comment or be unexported (golint)
- Line 42: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 51: warning: exported method HTTPFixture.AssertResponseStatusCode should have comment or be unexported (golint)
- Line 51: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 59: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
-
-
httpunit/request.go
- Line 13: warning: exported type RequestBuilder should have comment or be unexported (golint)
- Line 22: warning: exported function NewRequestBuilder should have comment or be unexported (golint)
- Line 30: warning: exported method RequestBuilder.Build should have comment or be unexported (golint)
- Line 30: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 40: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
-
-
httpunit/t.go
- Line 3: warning: exported type T should have comment or be unexported (golint)
ineffassign100%
IneffAssign detects ineffectual assignments in Go code.
No problems detected. Good job!