Fix linter errors
Some checks failed
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is failing

This commit is contained in:
Jan Bader 2022-08-21 19:49:16 +00:00
parent 08fbacc7e6
commit e397c509ca
4 changed files with 18 additions and 4 deletions

View File

@ -14,6 +14,7 @@ linters:
- gci # not working, shows errors on freshly formatted file - gci # not working, shows errors on freshly formatted file
- varnamelen - varnamelen
- lll - lll
linters-settings: linters-settings:
errcheck: errcheck:
exclude-functions: exclude-functions:
@ -25,3 +26,16 @@ linters-settings:
varnamelen: varnamelen:
ignore-decls: ignore-decls:
- c *gin.Context - c *gin.Context
wrapcheck:
ignoreSigs:
- .JSON(
- .Errorf(
- errors.New(
- errors.Unwrap(
- .Wrap(
- .Wrapf(
- .WithMessage(
- .WithMessagef(
- .WithStack(
ignorePackageGlobs:
- git.javil.eu/jacob1123/budgeteer/postgres

View File

@ -1,6 +1,7 @@
package server package server
import ( import (
"context"
"encoding/json" "encoding/json"
"net/http" "net/http"
"net/http/httptest" "net/http/httptest"
@ -35,11 +36,10 @@ func TestRegisterUser(t *testing.T) {
t.Run("RegisterUser", func(t *testing.T) { t.Run("RegisterUser", func(t *testing.T) {
t.Parallel() t.Parallel()
request, err := http.NewRequest( request, err := http.NewRequestWithContext(context.Background(),
http.MethodPost, http.MethodPost,
"/api/v1/user/register", "/api/v1/user/register",
strings.NewReader(`{"password":"pass","email":"info@example.com","name":"Test"}`)) strings.NewReader(`{"password":"pass","email":"info@example.com","name":"Test"}`))
request.Header.Set("Content-Type", "application/json")
context := engine.NewContext(request, recorder) context := engine.NewContext(request, recorder)
if err != nil { if err != nil {
t.Errorf("error creating request: %s", err) t.Errorf("error creating request: %s", err)

View File

@ -41,7 +41,7 @@ func (h *Handler) budgetingForMonth(c echo.Context) error {
month, err := getDate(c) month, err := getDate(c)
if err != nil { if err != nil {
c.Redirect(http.StatusTemporaryRedirect, "/budget/"+budget.ID.String()) return c.Redirect(http.StatusTemporaryRedirect, "/budget/"+budget.ID.String())
} }
data, err := h.getBudgetingViewForMonth(c.Request().Context(), budget, month) data, err := h.getBudgetingViewForMonth(c.Request().Context(), budget, month)

View File

@ -35,7 +35,7 @@ func (h *Handler) verifyLogin(c echo.Context) (budgeteer.Token, error) { //nolin
tokenString = tokenString[7:] tokenString = tokenString[7:]
token, err := h.TokenVerifier.VerifyToken(tokenString) token, err := h.TokenVerifier.VerifyToken(tokenString)
if err != nil { if err != nil {
return nil, fmt.Errorf("verify token '%s': %s", tokenString, err) return nil, fmt.Errorf("verify token '%s': %w", tokenString, err)
} }
return token, nil return token, nil