Recently, I had a conversation with a junior developer on my team. Let’s call him Alan. We were talking about a new notification feature that was going to be used to send reminder e-mails to potentially thousands of people if they had forgotten to enter certain data in the last month or so. Alan was confident that the code he’d written was correct. “I’ve tested it well.”, he said…
I’ve caught problems in code review and had to do this even.
Often it’s reading it and realizing there’s a complicated edge case or they missed something entirely.
Sure we can make a different ticket for that to move this along, but we’re getting product to agree first.
Ooof, I’m glad I never worked there.
QA’s job should be to make sure the bugs are known/documented/prioritised. It should never be a roadblock that interrupts work while several departments argue over what to do with a ticket.
Seriously who cares if the current ticket is left open with a “still need to do XYZ” or it gets closed and a new one is open “need to do XYZ”. Both are perfectly valid, do whichever one you think is appropriate and don’t waste anyone’s time discussing it.
No one cares if you leave a ticket open due to a bug or incomplete feature
Product sure as hell cares if you’re going to ship a bug or incomplete feature.
Never worked at company that wasn’t the case in over 15 years.
Product owns the work they ask us to do. We do their bidding.
And we certainly aren’t allowed to just change the scope of tickets at our own discretion without checking in