Yeah, they might have just had unit tests with mocked db, or there wasn't enough combinatorial variety. It's insufficient to test the lack of double billing - you have to specifically induce a race condition, assert the race occurred, and that the interlock prevented double spend. It's tricky. Personally I'd try to engineer around race condition entirely, like some scheme with tokens and pagination.
I agree, this sort of thing seems like it should be extra-well covered, but, y'know, move fast and double charge folks.
I agree, this sort of thing seems like it should be extra-well covered, but, y'know, move fast and double charge folks.