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 4ef0b759ef
4 changed files with 18 additions and 3 deletions

View File

@ -14,6 +14,7 @@ linters:
- gci # not working, shows errors on freshly formatted file
- varnamelen
- lll
linters-settings:
errcheck:
exclude-functions:
@ -25,3 +26,16 @@ linters-settings:
varnamelen:
ignore-decls:
- 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
import (
"context"
"encoding/json"
"net/http"
"net/http/httptest"
@ -35,7 +36,7 @@ func TestRegisterUser(t *testing.T) {
t.Run("RegisterUser", func(t *testing.T) {
t.Parallel()
request, err := http.NewRequest(
request, err := http.NewRequestWithContext(context.Background(),
http.MethodPost,
"/api/v1/user/register",
strings.NewReader(`{"password":"pass","email":"info@example.com","name":"Test"}`))

View File

@ -41,7 +41,7 @@ func (h *Handler) budgetingForMonth(c echo.Context) error {
month, err := getDate(c)
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)

View File

@ -35,7 +35,7 @@ func (h *Handler) verifyLogin(c echo.Context) (budgeteer.Token, error) { //nolin
tokenString = tokenString[7:]
token, err := h.TokenVerifier.VerifyToken(tokenString)
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