Mapeamento de Tipos de Processos: Importar Tipos de Processos para mapeamento

[euviniciusmendonca]

Prezados,

A funcionalidade foi incorporada dentro da tela de alteração ou cadastro de novo mapeamento, a qual é acessada por meio da tela: https://www.figma.com/file/w2MIP7290Ms7VIktsbTGdX/Tramita---Mapeamento-de-Tipo-de-Processo?type=design&node-id=18-1592&mode=design&t=L4HtvO1LR6mNKMjy-0.

Após clicar em Novo Mapeamentou Alterar, a tela a seguir é aberta: https://www.figma.com/file/w2MIP7290Ms7VIktsbTGdX/Tramita---Mapeamento-de-Tipo-de-Processo?type=design&node-id=17-825&mode=design&t=L4HtvO1LR6mNKMjy-0.

A importação é um botão na tela supracitada.

[davidnascimento1618]

Corrigir os conflitos mencionados aqui.

[davidnascimento1618]

Alguns pontos que foram acordados na agenda do dia 24/10/2023:

Após implementar os pontos descritos na issue 251 (tela da listagem de relacionamento entre órgãos):

• [x] 1. Criar a tela de 'Mapeamento de Tipos de Processo', seguindo como base este protótipo aqui;
• [x] 2. Para dar mais destaque, inserir um box com o botão 'Importar', que deverá exibir um modal a ser clicado, com select de arquivo para importação. Uma vez carregado o arquivo, o sistema deverá validar o seu teor e exibir um preview do seu conteúdo para confirmação do usuário, quando este clicar no botão de 'Salvar', importar o teor do arquivo, que será exibido na coluna 'Tipo de Processo Origem';
• [x] 3. Posteriormente, em uma outra agenda, o PO irá tratar da questão da exclusão ou desativação de um mapeamento na tela de 'Mapeamento de Tipos de Processo' (issue #384), que seria no caso do usuário importar um novo arquivo .csv, diretamente nesta tela, com tipos de processos a mais, a menos ou diferentes dos que já foram importados e carregados anteriormente.

Functional tests:

Corrigir os pontos elencados no pull request relacionado.

[mrglaydson]

@davidnascimento1618 Vídeo evidencia, mostrando ponto 1(nova tela):

https://drive.google.com/file/d/1EHZ9h1oFL0DD3MNq979-HZ515jolisF8/view?usp=sharing

O modal (ponto 2) está encaixado na issue 251

[davidnascimento1618]

Issue movida da sprint 28 para a 29 para validação.

[davidnascimento1618]

Prezados @Mauro1706 e @mrglaydson,

Ao realizar os testes manuais, foram encontrados os seguintes problemas:

• [ ] 1. Título da tela deveria estar descrito como: 'Mapeamento de Tipos de Processo' ao invés de 'Mapeamento de Tipo de Processo';
• [x] 2. Título do checkbox deveria estar descrito como: 'Exibir apenas Tipos de Processo sem mapeamento definido' ao invés de 'Exibir apenas assuntos sem mapeamento definido';
• [x] 3. Ação do checkbox 'Exibir apenas Tipos de Processo sem mapeamento definido' não está funcional;
• [x] 4. Descrições apresentando caracteres inválidos em tipos de processo que contém acentuação (coluna 'Tipo de Processo Origem');
• [ ] 5. Ação de ordenação dos tipos de processo (tanto ascendente como descendente) não estão funcionais;
• [ ] 6. Botão de 'Salvar' está sendo exibido mesmo sem nenhum mapeamento, o certo seria exibir o tal botão quando houver pelo menos 1 mapeamento realizado;
• [x] 7. Sistema está apresentando o seguinte erro abaixo, quando se tenta mapear (salvar) um tipo de processo dentro de um segundo relacionamento entre órgãos:

OBS.: O item 2 descrito neste comentário ainda não foi implementado.

_"Consulta retornou mais de um registro de MD_PEN_MAP_TIPO_PROCESSO. Detalhes: SELECT md_pen_map_tipo_processo.id_tipo_processo_origem AS idtipoprocessoorigem, md_pen_map_tipo_processo.id FROM md_pen_map_tipo_processo WHERE md_pen_map_tipo_processo.id_tipo_processoorigem=?"

Abaixo segue um vídeo com os steps para reprodução do erro encontrado no item 7:

https://github.com/pengovbr/mod-sei-pen/assets/87980947/c41093f5-0ae1-4549-9c06-832bcfc514c2

[davidnascimento1618]

@Mauro1706, @mrglaydson, corrigir os pontos elencados nos testes funcionais automatizados do pull request em questão.

[Mauro1706]

Movida da sprint 28 para 29

[davidnascimento1618]

Prezado @Mauro1706, os itens 1, 2, 3, 6 e 7 desse comment ainda continuam pendentes.

Além do mais, os botões que aparecem na parte de cima da tela ('Pesquisar', 'Salvar', 'Excluir' e 'Fechar') não estão sendo exibidos na parte inferior da tela, como é mostrado no .gif abaixo:

OBS.: O item 2 descrito neste comment ainda não foi implementado.

[davidnascimento1618]

Prezado @Mauro1706, corrigir os seguintes itens nos testes funcionais:

• [ ] 1. Foi verificado que os cenários de testes não estão testando a tela de mapeamento de tipo de processo em si, e sim, as ações da tela de relacionamento entre órgãos: cadastrar, editar, desativar, reativar e excluir um relacionamento, esperava-se que a execução do teste em questão cobrisse a tela de mapeamento com as ações de salvar, importar, pesquisar e exibir apenas assuntos sem mapeamento definido (checkbox) e, ao final, realizar um assert na listagem dessas ações mencionadas, a fim de verificar que de fato os resultados foram satisfatórios;

• [x] 2. Atribuir um nome mais intuitivo ao teste (isso inclui nomenclatura dos arquivos e classes correlacionadas), ou seja, um sugestão seria: 'MapeamentoTipoProcessoTest' ao invés de 'MapeamentoTipoProcessoRelacionamentoOrgaosListagemImportacaoTest', uma vez que este último está mais ligado ao teste que abrange a listagem e suas ações (tela de relacionamento entre órgãos). Se for o caso dos testes estarem todos juntos em um mesmo pull request, separa-los em roteiros: onde cada roteiro (arquivo) terão os seus respetivos casos de testes.

Ex.:

• MapeamentoTipoProcessoRelacionamentoOrgaosCadastroTest (p/ a tela de cadastro do relacionamento entre órgãos);

• MapeamentoTipoProcessoRelacionamentoOrgaosListagemImportacaoTest (p/ a tela de listagem, e suas ações, do relacionamento entre órgãos);

• MapeamentoTipoProcessoDesativarReativarTest (p/ a tela de listagem de relacionamento entre órgãos no tocante à desativação/reativação do relacionamento entre órgãos);

• MapeamentoTipoProcessoExportarTest (p/ a tela de exportação de tipos de processo);

• MapeamentoTipoProcessoTest (p/ a tela de mapeamento de tipos de processo e suas ações);

• MapeamentoTipoProcessoExcluirTest (p/ a tela de mapeamento de tipos de processo e a ação de excluir).

• [ ] 3. Realizar as mesmas correções dos itens 1 e 2 também nos testes funcionais das versões 3 e 4 do SEI.

[Mauro1706]

tarefa movida de sprint 29 para sprint 30

[yurioliveiramgi]

Em reunião ficou estabelecida a inclusão de uma nova tela relacionada à "Reativação dos Mapeamentos de Tipo de Processo". A inclusão dessa funcionalidade deve ser feita através de um submenu "Reativar Mapeamento de Tipo de Processo" localizado abaixo do submenu "Relacionamento entre Órgãos" do Menu "Mapeamento de Tipo de Processo". Na tela devem estar disponíveis apenas as ações "Reativar" e "Excluir" (lembrando que para mapeamentos que tenham sido utilizados a ação "Excluir" deve ser apresentada desabilitada, não permitindo a exclusão do mapeamento, assim como no caso do relacionamento entre órgãos). Segue imagem com a descrição da solicitação.

A funcionalidade de "Desativar" o mapeamento deve constar na lista de ações e na tela dos mapeamentos já configurados, conforme imagem a seguir. Note que houve remoção da opção "Incluir desativados" ao lado do campo de busca.

• [ ] Inclusão do submenu "Reativar Mapeamento de Processo"
• [ ] Remoção da opção "Incluir desativados" na página de "Mapeamento de Tipo de Processo"
• [ ] Inclusão da ação "Desativar" na tela de "Mapeamento de Tipo de Processo"
• [ ] Inativação da opção "Excluir" caso o mapeamento já tenha sido utilizado por algum processo tramitado (similar ao relacionamento entre órgãos"

@euviniciusmendonca , gentileza aprovar o texto.

[euviniciusmendonca]

@yurioliveiramgi, aprovado conforme solicitado.

[davidnascimento1618]

Prezado @Mauro1706, ao tentar salvar um mapeamento de tipo de processo, está ocasionando o seguinte erro abaixo:

_"Consulta retornou mais de um registro de MD_PEN_MAP_TIPOPROCESSO."

Como simular?

Crie dois relacionamentos entre órgãos e, depois, importe o mesmo arquivo .csv ou arquivos diferentes (desde que tenha tipos de processos em comum) para ambos os relacionamentos.

https://github.com/pengovbr/mod-sei-pen/assets/87980947/9eb3992d-77dc-4e8f-99c8-d8af421c670c

[davidnascimento1618]

Prezado @Mauro1706, os itens 1, 3 e 6 desse comment ainda continuam pendentes. Ademais, faltou implementar o requisito descrito nesse comment.

Outros itens a serem corrigidos, são eles:

• [x] 1. Caso os tipos de processos importados sejam todos excluídos na tela de mapeamento de tipos de processo, ao tentar realizar uma nova importação, a pré-visualização está vindo vazia;

https://github.com/pengovbr/mod-sei-pen/assets/87980947/12231071-8ffb-4f06-bb67-f519708f8b1e

• [x] 2. Caso seja excluído um ou mais tipos de processo, na tela de mapeamento de tipos de processo, ao tentar realizar uma nova importação e tentar voltar à tela anterior (via botão de 'Fechar'), o sistema está ocasionando o seguinte erro: _"Ação 'pen_map_orgaos_externos_mapeamentolistar' não reconhecida pelo controlador.";

https://github.com/pengovbr/mod-sei-pen/assets/87980947/f57db7c9-9f63-4b3e-ad06-4a46ba16a347

• [x] 3. Sistema exibe mensagem de sucesso, mas não importa o tipo de processo, ao se tentar realizar uma nova importação para um tipo de processo no qual fora desativado.

https://github.com/pengovbr/mod-sei-pen/assets/87980947/1e25c753-ebbe-4533-ae1d-6a45884ecbb5

[mateussbh]

Prezado @Mauro1706, ao tentar salvar um mapeamento de tipo de processo, está ocasionando o seguinte erro abaixo:

_"Consulta retornou mais de um registro de MD_PEN_MAP_TIPOPROCESSO."

Como simular?

Crie dois relacionamentos entre órgãos e, depois, importe o mesmo arquivo .csv ou arquivos diferentes (desde que tenha tipos de processos em comum) para ambos os relacionamentos.

Erro.Consulta.retornou.mais.de.um.registro.de.MD_PEN_MAP_TIPO_PROCESSO.mp4

Confirmada a correção.

[mateussbh]

Prezado @Mauro1706, os itens 1, 3 e 6 desse comment ainda continuam pendentes. Ademais, faltou implementar o requisito descrito nesse comment.

Outros itens a serem corrigidos, são eles:

• [x] 1. Caso os tipos de processos importados sejam todos excluídos na tela de mapeamento de tipos de processo, ao tentar realizar uma nova importação, a pré-visualização está vindo vazia;

  Pre-visualizacao.vindo.vazia.apos.excluir.tipos.de.processo.e.tentar.importar.novamente.mp4

• [x] 2. Caso seja excluído um ou mais tipos de processo, na tela de mapeamento de tipos de processo, ao tentar realizar uma nova importação e tentar voltar à tela anterior (via botão de 'Fechar'), o sistema está ocasionando o seguinte erro: _"Ação 'pen_map_orgaos_externos_mapeamentolistar' não reconhecida pelo controlador.";

  Erro.Acao.pen_map_orgaos_externos_mapeamento_listar.nao.reconhecida.pelo.controlador.mp4

• [x] 3. Sistema exibe mensagem de sucesso, mas não importa o tipo de processo, ao se tentar realizar uma nova importação para um tipo de processo no qual fora desativado.

  Sistema.exibe.mensagem.de.sucesso.mas.nao.importa.o.tipo.de.processo.ao.se.tentar.realizar.uma.nova.importacao.para.um.tipo.de.processo.no.qual.fora.desativado.mp4

Confirmadas as 3 correções.

[davidnascimento1618]

Prezado @Mauro1706, os itens 1, 5, 6, desse comment, ainda continuam pendentes.

Outros itens a serem corrigidos:

• [ ] Inativação da opção "Excluir", caso o mapeamento já tenha sido utilizado por algum processo tramitado (conforme descrito neste comentário);
• [x] Adicionar o botão de 'Fechar', também na parte inferior da tela;
• [ ] O título da tela de reativação do mapeamento deveria ser: "Reativar Mapeamento de Tipos de Processo" ao invés de "Reativar de Mapeamento de Tipo de Processo"
• [ ] Ação de ordenação dos tipos de processo na tela de reativação do mapeamento (tanto ascendente como descendente da coluna 'ID Tipo de Processo Origem' e Tipo de Processo Origem') não estão funcionais;

@yurioliveiramgi, favor discriminar quais as colunas deverão ser exibidas na tela de 'Reativar Mapeamento de Tipos de Processo', assim como a ordem delas (da esquerda para a direita) e, se em todas elas, terão as ações de ordenação (ascendente e descendente). Outro ponto a ser salientado, seria com relação a qual campo de pesquisa deverá ser exibido nesta tela, atualmente só tem um: 'Tipo de Processo Origem:'

A forma como a tela está desenhada hoje é esta:

[yurioliveiramgi]

Prezados @Mauro1706 e @davidnascimento1618, em resposta ao comentário acima, pontuo o seguinte:

• [x] Se puder colocar a evidência de em qual tela o botão "Excluir" deve ser inativado, agradeço.

  • Nas telas que visualizei, o ícone de "Excluir" (excluir relacionamento) fica inativo sempre que um mapeamento já existe ou foi importada a lista de tipos de processo. Vide figura abaixo.
  • Caso sejam selecionados relacionamentos com mapeamento ativo ou importação de lista iniciada, e clicado o botão excluir no topo da tela, a ação é interrompida, conforme esperado. Vide figura abaixo.

• [x] Adicionar o botão "Fechar" na parte inferior da tela "Mapeamento de Tipo de Processo" .

• [x] Na tela de "Reativar Mapeamento de Tipo de Processo", confirmo o não funcionamento das ações de ordenação, e pontuo que elas devem estar presentes também em "Órgão Origem" e "Órgão Destino" assim como na tela "Relacionamento entre Órgãos"

  • Alterar o título para "Reativar Mapeamento de Tipos de Processo"
  • Acrescentar o sublinhado no "R" do botão "Reativar", tanto no topo quanto na parte debaixo da tela
  • Retirar os ":" das colunas "Órgão Origem" e "Órgão Destino".
  • Notei ainda que a solicitação de correção feita no menu de navegação foi equivocada, deve ser apresentado o texto "Reativar Mapeamento de Tipos de Processo" e não somente "Reativar Tipos de Processo", conforme comentado aqui
  • Quanto à exibição da tela e o campo de busca, as informações apresentadas são suficientes. A coluna "ID Tipo de Processo Origem" pode ser removida. Quanto à ordenação pode ser mantida a sequência "Select all", "Órgão Origem", Órgão Destino", "Tipo de Processo Origem", "Tipo de Processo Destino" e "Ações".

• [ ] Na tela "Mapeamento de Tipo de Processo" deve ser corrigido o título para "Mapeamento de Tipos de Processo", e a ação de ordenação não está funcionando.

• [x] Acrescentar o sublinhado no "D" do botão "Desativar", tanto no topo quanto na parte debaixo da tela

[yurioliveiramgi]

@davidnascimento1618, apenas confirmando, o texto do último item desse comentário ficou um pouco confuso. A opção de "excluir" deve ser desativada enquanto a lista de mapeamento não estiver vazia. Havendo mapeamentos preenchidos ou lista de tipos (ainda que sem mapeamento) a opção segue desativada na tela de "relacionamento entre órgãos". Todavia, o usuário tem total autonomia para limpar e excluir o mapeamento, inclusive o relacionamento entre órgãos, caso deseje.

[mateussbh]

Prezado @Mauro1706, corrigir os seguintes itens nos testes funcionais:

• [ ] 1. Foi verificado que os cenários de testes não estão testando a tela de mapeamento de tipo de processo em si, e sim, as ações da tela de relacionamento entre órgãos: cadastrar, editar, desativar, reativar e excluir um relacionamento, esperava-se que a execução do teste em questão cobrisse a tela de mapeamento com as ações de salvar, importar, pesquisar e exibir apenas assuntos sem mapeamento definido (checkbox) e, ao final, realizar um assert na listagem dessas ações mencionadas, a fim de verificar que de fato os resultados foram satisfatórios;
• [x] 2. Atribuir um nome mais intuitivo ao teste (isso inclui nomenclatura dos arquivos e classes correlacionadas), ou seja, um sugestão seria: 'MapeamentoTipoProcessoTest' ao invés de 'MapeamentoTipoProcessoRelacionamentoOrgaosListagemImportacaoTest', uma vez que este último está mais ligado ao teste que abrange a listagem e suas ações (tela de relacionamento entre órgãos). Se for o caso dos testes estarem todos juntos em um mesmo pull request, separa-los em roteiros: onde cada roteiro (arquivo) terão os seus respetivos casos de testes.

Ex.:

• MapeamentoTipoProcessoRelacionamentoOrgaosCadastroTest (p/ a tela de cadastro do relacionamento entre órgãos);
• MapeamentoTipoProcessoRelacionamentoOrgaosListagemImportacaoTest (p/ a tela de listagem, e suas ações, do relacionamento entre órgãos);
• MapeamentoTipoProcessoDesativarReativarTest (p/ a tela de listagem de relacionamento entre órgãos no tocante à desativação/reativação do relacionamento entre órgãos);
• MapeamentoTipoProcessoExportarTest (p/ a tela de exportação de tipos de processo);
• MapeamentoTipoProcessoTest (p/ a tela de mapeamento de tipos de processo e suas ações);
• MapeamentoTipoProcessoExcluirTest (p/ a tela de mapeamento de tipos de processo e a ação de excluir).
• [ ] 3. Realizar as mesmas correções dos itens 1 e 2 também nos testes funcionais das versões 3 e 4 do SEI.

Apesar dos testes de recebimento e mapeamento terem sido feitos com sucesso, de forma manual, está mapeada a necessidade de inclusão de novos testes funcionais automatizados para completar essa lista de testes.

[mateussbh]

Issue duplicada com a issue https://github.com/pengovbr/mod-sei-pen/issues/254. Realizando sua conclusão. Itens pendentes (1 e 5) deste comentário, https://github.com/pengovbr/mod-sei-pen/issues/257#issuecomment-1811922633, serão tratados em issue a parte.