def test_defeito_fechado_tem_data_de_fechamento(db):
    ruins = db.execute(
        """
        SELECT id, status
        FROM defeitos
        WHERE status IN ('fechado', 'resolvido')
          AND data_fechamento IS NULL
        """
    ).fetchall()
    assert ruins == [], f"defeitos fechados sem data: {[r['id'] for r in ruins]}"

Se ficasse vermelho num projeto real, a sequência seria:

Confirmar no banco: rodar a query à mão e olhar as linhas: quantas são, de quando, de qual módulo.
Decidir o que é o defeito: o dado está torto (alguém fechou sem preencher) ou a regra é que está errada (existe um fluxo legítimo de fechar sem data)? Essa pergunta é do Módulo 5 de Fundamentos: pode ser que o defeito esteja na especificação.
Abrir o relatório com os ids como evidência, severidade e prioridade justificadas separadamente.
Manter o teste vermelho, ou marcá-lo com xfail(strict=True) enquanto o dado não é corrigido, nunca apagá-lo. Com strict, no dia da correção o build avisa que o marcador pode sair.

O que não se faz: afrouxar o assert para a suíte ficar verde. Isso transforma o teste num enfeite.

Prova do Módulo 3