Skip to content

Commit

Permalink
fix golangci-lint errors
Browse files Browse the repository at this point in the history
  • Loading branch information
negrel committed Jan 15, 2024
1 parent 2613fbe commit ff396bc
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
8 changes: 5 additions & 3 deletions internal/services/sessions/service.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,7 @@ const (
)

var (
ErrSessionNotFound = errors.New("session not found")
errSessionIsAnonymous = errors.New("anonymous session")
ErrSessionNotFound = errors.New("session not found")

sessionConfig = session.Config{
Storage: nil, // in memory storage.
Expand Down Expand Up @@ -61,7 +60,10 @@ func (s service) CreateSession(c *fiber.Ctx, userId users.UserId) error {

// User already have a session, resets it first.
if !session.Fresh() {
session.Reset()
err := session.Reset()
if err != nil {
return fmt.Errorf("failed to reset session: %w", err)
}
}

session.Set(UserIdKey, userId.String())
Expand Down
2 changes: 1 addition & 1 deletion internal/services/sessions/service_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@ func TestService(t *testing.T) {
require.NoError(t, err)
require.Len(t, resp.Cookies(), 1)
sessCookie := resp.Cookies()[0]
require.Positive(t, time.Now().Sub(sessCookie.Expires))
require.Positive(t, time.Since(sessCookie.Expires))
require.Equal(t, 0, sessCookie.MaxAge)
})

Expand Down
2 changes: 1 addition & 1 deletion internal/services/users/service_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ func TestService(t *testing.T) {
expectedUser := User{
Id: NewUserId(),
Email: email,
Password: testutils.Must(NewPassword)(secret.New("s3cretPassw0rd")),
Password: secret.New("p4ssW0rdHash"),
Name: testutils.Must(NewUserName)("foo"),
CreatedAt: time.Now(),
}
Expand Down

0 comments on commit ff396bc

Please sign in to comment.