Lea Fairbanks
Lea Fairbanks
Oh, I didn't update the tests, I'll get it tomorrow. My bad.
I generally prefer to avoid those kinds of assumptions. To me, having multiple methods, one for an absolute time, and one for a relative time, is very explicit and behaves...
I initially found the problem because I was trying to see if my tokens were expired and getting an incorrect value. The AbstractToken constructor was taking the absolute expiration time...
I went with your suggested approach, let me know what you think, thanks!
sure thing, work has gotten a bit more stressful in the last couple weeks, I will try to do something this weekend
The CI check seems to fail because it's using an outdated version of bundler. See: https://github.com/bundler/bundler/pull/3559
Sorry for the delay, I went ahead and added a test. Let me know if you like the approach!