search

interlegis / sapl

Sistema de Apoio ao Processo Legislativo
GNU General Public License v3.0
86 stars 81 forks source link

Refatorar teste lento de permissionamento #998

Open marciomazza opened 7 years ago

marciomazza commented 7 years ago

Um teste muito lento teve de ser desligado em 9e185ed56597f2581f7df13635277bb4029bf537. É importante refatorá-lo e ligá-lo novamente quanto rodar em um tempo aceitável.

Notei que um setup muito demorado está sendo feito a cada chamada do método. Uma ideia é não usar o @pytest.mark.parametrize('url_item', _lista_urls) (que obriga a repetição do setup para cada url_item) e fazer simplesmente um loop.

LeandroJatai commented 7 years ago

isso... pra cada url ele tem que criar os usuários novamente... hoje o teste assim já não é mais produtivo, mas no passado foi pois de outra forma ficava muito teste mascarado visto que dependendo de como faz vc não consegue andar nas folhas da arvore de url totalmente... ainda assim, um erro pode ser gerado para um usuário e só depois para outro, mas daí a árvore é muito menor... hoje pode ser revertido, ou seja, os parâmetros serem os usuários e não as urls já que todas as urls vão passar e só as novas que forem surgindo vão caindo no teste, antes era muitas inviável ficar rodando teste e recebendo erro nó por nó.