Не так.
Инфы в аутпуте теста должно быть достаточно, что бы по её совокупности было понятно что это за тест, на каких данных он гонится, где конкретно и почему он упал.
На том проекте, где это было сделано - хватало в 99% случаев.
И да, разработчики ходили и правили тесты упавшие на их PR.
ну просто я не вижу смысла для этого делать скриншот
если разраб знаком с тестами на том уровне, что может и реально ДЕЛАЕТ в них сам правки
обычно просто разработчика и палками не загонишь фиксить то, что он напрудил
и объективно их время стоит дороже qa-шного