fix: always set ticket expires at as it can't be null

[dbarrosop]

Fixes nhost/nhost#3035

PR Type

Bug fix

---

Description

• Updated TicketExpiresAt field in multiple test cases to use sql.TimestampTz(time.Now()) ensuring it is always set to a non-null value.
• Modified ticketExpiresAt initialization in #UserWithouthSession method to ensure it is always set to a non-null value.

---

Changes walkthrough 📝

Relevant files

Bug fix

post_signin_idtoken_test.go Set non-null `TicketExpiresAt` in test cases                          go/controller/post_signin_idtoken_test.go Updated TicketExpiresAt field to use sql.TimestampTz(time.Now()) instead of pgtype.Timestamptz{}. Ensured TicketExpiresAt is always set to a non-null value in test cases. +4/-4      workflows.go Ensure `ticketExpiresAt` is non-null during user #    go/controller/workflows.go Changed ticketExpiresAt initialization to use sql.TimestampTz(time.Now()). Ensured ticketExpiresAt is always set to a non-null value. +1/-1

---

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Possible Bug Ensure that sql.TimestampTz(time.Now()) correctly initializes TicketExpiresAt and does not introduce any unintended side effects, especially considering time zone differences and precision. Code Smell The initialization of ticketExpiresAt with sql.TimestampTz(time.Now()) is duplicated. Consider refactoring to avoid redundancy and improve maintainability.

[github-actions]

PR Code Suggestions ✨