Browse Source

Merge pull request #726 from interlegis/falso-positivo

Fix Falso Positivo de URLs publicas
pull/732/head
Eduardo Edson Batista Cordeiro Alves 9 years ago
committed by GitHub
parent
commit
2e373e28ab
  1. 14029
      echo
  2. 36
      sapl/test_urls.py

14029
echo

File diff suppressed because it is too large

36
sapl/test_urls.py

@ -272,19 +272,34 @@ urls_publicas_excecoes = {
'/sessao/pauta-sessao/1', '/sessao/pauta-sessao/1',
'/sessao/pauta-sessao/pesquisar-pauta', '/sessao/pauta-sessao/pesquisar-pauta',
'/sessao/pesquisar-sessao', '/sessao/pesquisar-sessao',
'/sessao/1/reordenar-expediente',
'/sessao/1/reordenar-ordem',
'/proposicao/1/ta', # FIXME Compilação deverá tratar '/proposicao/1/ta', # FIXME Compilação deverá tratar
'/materia/1/ta', '/materia/1/ta',
'/norma/1/ta', '/norma/1/ta',
'/comissao/1/materias-em-tramitacao',
'/proposicao/',
'/proposicao/1',
'/proposicao/1/delete',
'/proposicao/1/edit',
'/protocoloadm/pesquisar-autor',
], ],
'post': [ 'post': [
'/norma/pesquisa-resultado', '/norma/pesquisa-resultado',
'/mesa-diretora/' # tratamento de permissão interno. '/mesa-diretora/', # tratamento de permissão interno.
'sessao/1/resumo', '/sessao/1/resumo',
'/sessao/pauta-sessao', '/sessao/pauta-sessao',
'/sessao/pauta-sessao/1', '/sessao/pauta-sessao/1',
'/sessao/pauta-sessao/1/expediente/', '/sessao/pauta-sessao/1/expediente/',
'/sessao/pauta-sessao/1/ordem/', '/sessao/pauta-sessao/1/ordem/',
'/sessao/pesquisar-sessao',
'/sessao/1/reordenar-expediente',
'/sessao/1/reordenar-ordem',
'/sessao/pauta-sessao/pesquisar-pauta',
'/sessao/pesquisar-sessao',
'/comissao/1/materias-em-tramitacao', '/comissao/1/materias-em-tramitacao',
@ -292,8 +307,13 @@ urls_publicas_excecoes = {
'/materia/1/ta', '/materia/1/ta',
'/norma/1/ta', '/norma/1/ta',
'/materia/confirmar/1/1', '/materia/confirmar/1/1',
'/materia/pesquisar-materia' '/materia/pesquisar-materia',
'/proposicao/',
'/proposicao/1',
'/proposicao/1/delete',
'/proposicao/1/edit',
'/protocoloadm/pesquisar-autor',
] ]
} }
@ -324,7 +344,7 @@ def test_permissions_urls_for_users_by_apps(url_item, client):
como é o caso de PainelView que está na app 'sessao' como é o caso de PainelView que está na app 'sessao'
mas é um redirecionamento para 'painel'... aqui é feita mas é um redirecionamento para 'painel'... aqui é feita
a troca da app a ser testada, por essas outras possíveis. a troca da app a ser testada, por essas outras possíveis.
Este, até a ultima versão deste teste é o único tipo de view que Este, até a ultima versão deste teste é o único tipo de view que
possui restrição restrição simples, por permissão, e não por possui restrição restrição simples, por permissão, e não por
container, como é o caso de proposições que possui restrição container, como é o caso de proposições que possui restrição
@ -423,7 +443,7 @@ def test_permissions_urls_for_users_by_apps(url_item, client):
No teste de requisição "%s" a url (%s). No teste de requisição "%s" a url (%s).
App (%s) App (%s)
O usuário (%s) deveria ser redirecionado O usuário (%s) deveria ser redirecionado
para tela de login. para tela de login.
""" % (_type, url, app, username) """ % (_type, url, app, username)
else: else:
assert btn_login not in content, """ assert btn_login not in content, """
@ -434,10 +454,10 @@ def test_permissions_urls_for_users_by_apps(url_item, client):
invariavelmente pública, a adicione na variavel invariavelmente pública, a adicione na variavel
abaixo localizada no arquivo que se encontra este abaixo localizada no arquivo que se encontra este
teste: teste:
urls_publicas_excecoes urls_publicas_excecoes
""" % (_type, url, app, username) """ % (_type, url, app, username)
if username not in users_for_url_atual_app: if username not in users_for_url_atual_app:

Loading…
Cancel
Save