A few other notes:
• I use self. so it's clear where stuff comes from. It takes microseconds to type and it makes the code much more readable later. You know that never_expires? is a method and expires_at is from AR
• I might actually write fewer tests if the code was written this way - with the expressions and yoda statements it's really hard to know what it's doing, so tests help. Here, it seems more obvious