5 ms·
SerpApi dev here. I've requested, reviewed, tested, and deployed this upgrade of gems. I want to clarify some things. This code was running on staging for thre
by ilyazub 5y ago
SerpApi dev here. I've requested, reviewed, tested, and deployed this upgrade of gems.
I want to clarify some things. This code was running on staging for three weeks before the deployment to production. There were four application errors on staging that were related to the problem with subscription renewals.
I and the author of the pull request have made three mistakes:
- haven't carefully read code in all methods from the backtrace of four app errors on a staging environment
- upgraded several gems at once
- didn't review the changelog of mongoid gem
As multiple people commented here, integration tests for renewals should've been caught that bug. We hadn't a test case when all user's subscriptions are checked after the renewal of a specific test user.
So the lesson here is to not yolo minor upgrades of dependencies and avoid mistakes above.