efritz/chevron

MockLogger has 52 methods (exceeds 20 allowed). Consider refactoring.

Open

type MockLogger struct {
    statsLogWithFieldsLock          sync.RWMutex
    statLogWithFieldsFuncCallCount  int
    statLogWithFieldsFuncCallParams []LoggerLogWithFieldsParamSet
    LogWithFieldsFunc               func(logging.LogLevel, logging.Fields, string, ...interface{})

Similar blocks of code found in 2 locations. Consider refactoring.

Open

package middleware

type (
    SchemaConfigFunc func(m *SchemaMiddleware)
)

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

package middleware

type (
    RecoverConfigFunc func(m *RecoverMiddleware)
)

• Read up
  Read up

Method SchemaMiddleware.Convert has 7 return statements (exceeds 4 allowed).

Open

func (m *SchemaMiddleware) Convert(f chevron.Handler) (chevron.Handler, error) {
    schema, err := loadSchema(m.path)
    if err != nil {
        return nil, err
    }

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func (m *MockLogger) FatalWithFields(v0 logging.Fields, v1 string, v2 ...interface{}) {
    m.statsFatalWithFieldsLock.Lock()
    m.statFatalWithFieldsFuncCallCount++
    m.statFatalWithFieldsFuncCallParams = append(m.statFatalWithFieldsFuncCallParams, LoggerFatalWithFieldsParamSet{v0, v1, v2})
    m.statsFatalWithFieldsLock.Unlock()

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func (m *MockLogger) ErrorWithFields(v0 logging.Fields, v1 string, v2 ...interface{}) {
    m.statsErrorWithFieldsLock.Lock()
    m.statErrorWithFieldsFuncCallCount++
    m.statErrorWithFieldsFuncCallParams = append(m.statErrorWithFieldsFuncCallParams, LoggerErrorWithFieldsParamSet{v0, v1, v2})
    m.statsErrorWithFieldsLock.Unlock()

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func (m *MockLogger) DebugWithFields(v0 logging.Fields, v1 string, v2 ...interface{}) {
    m.statsDebugWithFieldsLock.Lock()
    m.statDebugWithFieldsFuncCallCount++
    m.statDebugWithFieldsFuncCallParams = append(m.statDebugWithFieldsFuncCallParams, LoggerDebugWithFieldsParamSet{v0, v1, v2})
    m.statsDebugWithFieldsLock.Unlock()

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func (m *MockLogger) InfoWithFields(v0 logging.Fields, v1 string, v2 ...interface{}) {
    m.statsInfoWithFieldsLock.Lock()
    m.statInfoWithFieldsFuncCallCount++
    m.statInfoWithFieldsFuncCallParams = append(m.statInfoWithFieldsFuncCallParams, LoggerInfoWithFieldsParamSet{v0, v1, v2})
    m.statsInfoWithFieldsLock.Unlock()

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func (m *MockLogger) WarningWithFields(v0 logging.Fields, v1 string, v2 ...interface{}) {
    m.statsWarningWithFieldsLock.Lock()
    m.statWarningWithFieldsFuncCallCount++
    m.statWarningWithFieldsFuncCallParams = append(m.statWarningWithFieldsFuncCallParams, LoggerWarningWithFieldsParamSet{v0, v1, v2})
    m.statsWarningWithFieldsLock.Unlock()

• Read up
  Read up

Method CacheMiddleware.Convert has 5 return statements (exceeds 4 allowed).

Open

func (m *CacheMiddleware) Convert(f chevron.Handler) (chevron.Handler, error) {
    handler := func(ctx context.Context, req *http.Request, logger nacelle.Logger) response.Response {
        // If we don't have a cache instance or if this request can
        // have side effects, do not attempt to touch the cache in
        // either direction.

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func (m *MockLogger) Debug(v0 string, v1 ...interface{}) {
    m.statsDebugLock.Lock()
    m.statDebugFuncCallCount++
    m.statDebugFuncCallParams = append(m.statDebugFuncCallParams, LoggerDebugParamSet{v0, v1})
    m.statsDebugLock.Unlock()

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func (m *MockLogger) Error(v0 string, v1 ...interface{}) {
    m.statsErrorLock.Lock()
    m.statErrorFuncCallCount++
    m.statErrorFuncCallParams = append(m.statErrorFuncCallParams, LoggerErrorParamSet{v0, v1})
    m.statsErrorLock.Unlock()

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func (m *MockLogger) Fatal(v0 string, v1 ...interface{}) {
    m.statsFatalLock.Lock()
    m.statFatalFuncCallCount++
    m.statFatalFuncCallParams = append(m.statFatalFuncCallParams, LoggerFatalParamSet{v0, v1})
    m.statsFatalLock.Unlock()

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func (m *MockLogger) Info(v0 string, v1 ...interface{}) {
    m.statsInfoLock.Lock()
    m.statInfoFuncCallCount++
    m.statInfoFuncCallParams = append(m.statInfoFuncCallParams, LoggerInfoParamSet{v0, v1})
    m.statsInfoLock.Unlock()

• Read up
  Read up

Similar blocks of code found in 5 locations. Consider refactoring.

Open

func (m *MockLogger) Warning(v0 string, v1 ...interface{}) {
    m.statsWarningLock.Lock()
    m.statWarningFuncCallCount++
    m.statWarningFuncCallParams = append(m.statWarningFuncCallParams, LoggerWarningParamSet{v0, v1})
    m.statsWarningLock.Unlock()

• Read up
  Read up

exported method AuthMiddleware.Convert should have comment or be unexported

Open

func (m *AuthMiddleware) Convert(f chevron.Handler) (chevron.Handler, error) {

• Exclude checks
  • Disable engine
  • Disable check

exported type JWTClaimsFactory should have comment or be unexported

Open

    JWTClaimsFactory func() jwt.Claims

• Exclude checks
  • Disable engine
  • Disable check

exported function NewMockCache should have comment or be unexported

Open

func NewMockCache() *MockCache {

• Exclude checks
  • Disable engine
  • Disable check

exported method MockCache.SetValueFuncCallCount should have comment or be unexported

Open

func (m *MockCache) SetValueFuncCallCount() int {

• Exclude checks
  • Disable engine
  • Disable check

exported method MockCache.SetValueFuncCallParams should have comment or be unexported

Open

func (m *MockCache) SetValueFuncCallParams() []CacheSetValueParamSet {

• Exclude checks
  • Disable engine
  • Disable check