Support for never expire cache items when expiration time is null

[dev-hancock]

Updated

• SQLCommands now support operations when ExpiresAtTime is NULL
• ItemFull model updated to use a nullable ExpiresAtTime
• PostgreSqlCacheOptions now exposes a nullable DefaultSlidingExpiration, allowing you to override the default ExpiresAtTime behaviour
• PostgreSqlCache respects nullable ExpiresAtTime, and the ValidationOptions method has been removed

[dev-hancock]

PR for #64

[dev-hancock]

Reviewed the existing tests and noticed a number of tests that look auto-generated (likely by an AI tool?) but don’t add meaningful coverage.

Would you be open to carving out a subset of high-value tests and removing or refactoring the rest?

Happy to discuss which tests you think are most valuable and help with the cleanup!

Thanks for considering

[leonibr]

Thanks for the feedback!

Just a heads-up: I’ve been working on a new feature (see #92), which targets the next branch. The tests in that branch have started to been cleaned up and are more streamlined. I just pushed what I had so far but the feature itself it is still in progress and I just committed what is working.
The branch you’re is based on master, which still has some legacy (and auto-generated) tests.

Would it make sense for you to retarget your PR to the next branch instead? That way, you’ll see the updated tests, and we can avoid redundant cleanup.

Let me know what you think, and happy to help coordinate!

Thanks again!

[dev-hancock]

This is now rebased onto next, I’ve sorted out the merge conflicts, and the tests are passing. Please take a look and let me know if anything else needs tweaking!

[leonibr]

Thank you @dev-hancock , I really appreciate it