Merge pull request #1409 from prefeiturasp/hotfix/1.33.2

Hotfix/1.33.2
prefeiturasp · Sep 14, 2022 · 1a404b7 · 1a404b7
2 parents 6e7d4bb + 3b453a3
commit 1a404b7
Show file tree

Hide file tree

Showing 22 changed files with 39 additions and 51 deletions.
diff --git a/hotfixes.md b/hotfixes.md
@@ -1,3 +1,7 @@
+### 1.33.2 - 14/09/2022 - Hotfix - Soluções de bugs urgentes durante a sprint 49
+* (72908) Melhorias na ata de parecer técnico
+* (72442) Remoção do campos devolução ao tesouro da PC
+
 ### 1.33.1 - 12/09/2022 - Hotfix - Soluções de bugs urgentes durante a sprint 49
 * (72992) - Corrige problema de duplicação de fechamentos
 * (73629) - Implementa ação para importar implantação de valores reprogramados

diff --git a/sme_ptrf_apps/__init__.py b/sme_ptrf_apps/__init__.py
@@ -1,4 +1,4 @@
-__version__ = "1.33.1"
+__version__ = "1.33.2"
 
 
 __version_info__ = tuple(

diff --git a/sme_ptrf_apps/conftest.py b/sme_ptrf_apps/conftest.py
@@ -631,7 +631,6 @@ def prestacao_conta(periodo, associacao, motivo_aprovacao_ressalva_x, motivo_rep
         data_recebimento=date(2020, 10, 1),
         data_recebimento_apos_acertos=date(2020, 10, 1),
         data_ultima_analise=date(2020, 10, 1),
-        devolucao_tesouro=True,
         motivos_reprovacao=[motivo_reprovacao_x, ],
         outros_motivos_reprovacao="Outros motivos reprovacao",
         motivos_aprovacao_ressalva=[motivo_aprovacao_ressalva_x, ],

diff --git a/sme_ptrf_apps/core/api/serializers/prestacao_conta_serializer.py b/sme_ptrf_apps/core/api/serializers/prestacao_conta_serializer.py
@@ -1,9 +1,11 @@
 from rest_framework import serializers
 
 from sme_ptrf_apps.core.models import PrestacaoConta, ObservacaoConciliacao
-from sme_ptrf_apps.core.api.serializers import (AssociacaoCompletoSerializer, DevolucaoPrestacaoContaRetrieveSerializer,
-                                                AnaliseContaPrestacaoContaRetrieveSerializer,
-                                                DevolucaoAoTesouroRetrieveSerializer)
+from sme_ptrf_apps.core.api.serializers import (
+    AssociacaoCompletoSerializer,
+    DevolucaoPrestacaoContaRetrieveSerializer,
+    DevolucaoAoTesouroRetrieveSerializer
+)
 from sme_ptrf_apps.core.services.processos_services import get_processo_sei_da_prestacao
 
 from sme_ptrf_apps.dre.api.serializers.motivo_aprovacao_ressalva_serializer import MotivoAprovacaoRessalvaSerializer
@@ -238,4 +240,4 @@ class Meta:
 
 
 def _str_devolucao_ao_tesouro(obj):
-    return f'{obj.total_devolucao_ao_tesouro:.2f}'.replace('.', ',') if obj.devolucao_tesouro else 'Não'
+    return f'{obj.total_devolucao_ao_tesouro:.2f}'.replace('.', ',') if obj.devolucoes_ao_tesouro_da_prestacao.count() > 0 else 'Não'
diff --git a/sme_ptrf_apps/core/migrations/0270_remove_prestacaoconta_devolucao_tesouro.py b/sme_ptrf_apps/core/migrations/0270_remove_prestacaoconta_devolucao_tesouro.py
@@ -0,0 +1,17 @@
+# Generated by Django 2.2.10 on 2022-09-05 15:50
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('core', '0269_delete_analisevalorreprogramadoprestacaoconta'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='prestacaoconta',
+            name='devolucao_tesouro',
+        ),
+    ]
diff --git a/sme_ptrf_apps/core/models/prestacao_conta.py b/sme_ptrf_apps/core/models/prestacao_conta.py
@@ -79,8 +79,6 @@ class PrestacaoConta(ModeloBase):
 
     data_ultima_analise = models.DateField('data da última análise pela DRE', blank=True, null=True)
 
-    devolucao_tesouro = models.BooleanField('há devolução ao tesouro', blank=True, null=True, default=False)
-
     motivos_reprovacao = models.ManyToManyField('dre.MotivoReprovacao', blank=True)
 
     outros_motivos_reprovacao = models.TextField('Outros motivos para reprovação pela DRE', blank=True, default='')
@@ -251,8 +249,6 @@ def salvar_devolucoes_ao_tesouro(self, devolucoes_ao_tesouro_da_prestacao=[]):
     def salvar_analise(self, analises_de_conta_da_prestacao=None, resultado_analise=None,
                        motivos_aprovacao_ressalva=[], outros_motivos_aprovacao_ressalva='', motivos_reprovacao=[],
                        outros_motivos_reprovacao='', recomendacoes=''):
-        from ..models.analise_conta_prestacao_conta import AnaliseContaPrestacaoConta
-        from ..models.conta_associacao import ContaAssociacao
 
         self.data_ultima_analise = date.today()
 

diff --git a/sme_ptrf_apps/core/tests/tests_api_prestacoes_contas/test_conclui_analise_prestacao_conta.py b/sme_ptrf_apps/core/tests/tests_api_prestacoes_contas/test_conclui_analise_prestacao_conta.py
@@ -26,7 +26,6 @@ def test_api_conclui_analise_prestacao_conta_exige_resultado_analise(jwt_authent
                                                                      prestacao_conta_em_analise,
                                                                      conta_associacao):
     payload = {
-        'devolucao_tesouro': True,
         'analises_de_conta_da_prestacao': [
             {
                 'conta_associacao': f'{conta_associacao.uuid}',
@@ -57,7 +56,6 @@ def test_api_conclui_analise_prestacao_conta_exige_resultado_analise(jwt_authent
 def test_api_conclui_analise_prestacao_conta_exige_analises_de_conta_da_prestacao(jwt_authenticated_client_a,
                                                                                   prestacao_conta_em_analise):
     payload = {
-        'devolucao_tesouro': True,
         'resultado_analise': PrestacaoConta.STATUS_APROVADA
     }
 
@@ -128,7 +126,6 @@ def test_api_conclui_analise_prestacao_conta_valida_resultado_aanalise(jwt_authe
                                                                        prestacao_conta_recebida,
                                                                        conta_associacao):
     payload = {
-        'devolucao_tesouro': True,
         'analises_de_conta_da_prestacao': [
             {
                 'conta_associacao': f'{conta_associacao.uuid}',

diff --git a/...e/tests/tests_api_prestacoes_contas/test_conclui_analise_prestacao_conta_como_aprovada.py b/...e/tests/tests_api_prestacoes_contas/test_conclui_analise_prestacao_conta_como_aprovada.py
@@ -27,7 +27,6 @@ def prestacao_conta_em_analise(periodo, associacao):
 def test_api_conclui_analise_prestacao_conta_aprovada(jwt_authenticated_client_a, prestacao_conta_em_analise,
                                                       conta_associacao, motivo_reprovacao_x):
     payload = {
-        'devolucao_tesouro': True,
         'analises_de_conta_da_prestacao': [
             {
                 'conta_associacao': f'{conta_associacao.uuid}',

diff --git a/...sts_api_prestacoes_contas/test_conclui_analise_prestacao_conta_como_aprovada_ressalvas.py b/...sts_api_prestacoes_contas/test_conclui_analise_prestacao_conta_como_aprovada_ressalvas.py
@@ -35,7 +35,6 @@ def motivo_aprovacao_ressalva_x():
 def test_api_conclui_analise_prestacao_conta_aprovada_ressalvas(jwt_authenticated_client_a, prestacao_conta_em_analise,
                                                                 conta_associacao, motivo_aprovacao_ressalva_x):
     payload = {
-        'devolucao_tesouro': True,
         'analises_de_conta_da_prestacao': [
             {
                 'conta_associacao': f'{conta_associacao.uuid}',
@@ -66,7 +65,6 @@ def test_api_conclui_analise_prestacao_conta_aprovada_ressalva_exige_ressalva(jw
                                                                               prestacao_conta_em_analise,
                                                                               conta_associacao):
     payload = {
-        'devolucao_tesouro': True,
         'analises_de_conta_da_prestacao': [
             {
                 'conta_associacao': f'{conta_associacao.uuid}',
@@ -98,7 +96,6 @@ def test_api_conclui_analise_prestacao_conta_aprovada_ressalva_exige_recomendaco
                                                                                    prestacao_conta_em_analise,
                                                                                    conta_associacao):
     payload = {
-        'devolucao_tesouro': True,
         'analises_de_conta_da_prestacao': [
             {
                 'conta_associacao': f'{conta_associacao.uuid}',

diff --git a/.../tests/tests_api_prestacoes_contas/test_conclui_analise_prestacao_conta_como_devolvida.py b/.../tests/tests_api_prestacoes_contas/test_conclui_analise_prestacao_conta_como_devolvida.py
@@ -74,7 +74,6 @@ def prestacao_conta_01_pc_posterior(periodo_01, associacao, motivo_aprovacao_res
         associacao=associacao,
         data_recebimento=date(2020, 10, 1),
         data_ultima_analise=date(2020, 10, 1),
-        devolucao_tesouro=True,
         motivos_reprovacao=[motivo_reprovacao_x, ],
         outros_motivos_reprovacao="Outros motivos reprovação",
         motivos_aprovacao_ressalva=[motivo_aprovacao_ressalva_x, ],
@@ -93,7 +92,6 @@ def prestacao_conta_02_pc_posterior(periodo, associacao, motivo_aprovacao_ressal
         associacao=associacao,
         data_recebimento=date(2020, 10, 2),
         data_ultima_analise=date(2020, 10, 2),
-        devolucao_tesouro=True,
         motivos_reprovacao=[motivo_reprovacao_x, ],
         outros_motivos_reprovacao="Outros motivos reprovação",
         motivos_aprovacao_ressalva=[motivo_aprovacao_ressalva_x, ],
@@ -125,7 +123,6 @@ def tipo_devolucao_ao_tesouro():
 def test_api_conclui_analise_prestacao_conta_devolvida(jwt_authenticated_client_a, prestacao_conta_em_analise,
                                                        conta_associacao, despesa, tipo_devolucao_ao_tesouro):
     payload = {
-        'devolucao_tesouro': True,
         'analises_de_conta_da_prestacao': [
             {
                 'conta_associacao': f'{conta_associacao.uuid}',
@@ -154,7 +151,6 @@ def test_api_conclui_analise_prestacao_conta_aprovada_ressalva_exige_data_limite
                                                                                  prestacao_conta_em_analise,
                                                                                  conta_associacao):
     payload = {
-        'devolucao_tesouro': True,
         'analises_de_conta_da_prestacao': [
             {
                 'conta_associacao': f'{conta_associacao.uuid}',

diff --git a/sme_ptrf_apps/core/tests/tests_api_prestacoes_contas/test_list_prestacoes_conta.py b/sme_ptrf_apps/core/tests/tests_api_prestacoes_contas/test_list_prestacoes_conta.py
@@ -106,7 +106,6 @@ def _prestacao_conta_2020_1_unidade_a_dre1(periodo_2020_1, _unidade_a_dre_1, _as
         periodo=periodo_2020_1,
         associacao=_associacao_a_dre_1,
         data_recebimento=date(2020, 1, 1),
-        devolucao_tesouro=True,
         status='APROVADA'
     )
 
@@ -157,7 +156,6 @@ def test_api_list_prestacoes_conta_por_periodo_e_dre(jwt_authenticated_client_a,
     response = jwt_authenticated_client_a.get(url, content_type='application/json')
 
     result = json.loads(response.content)
-
     result_esperado = [
         {
             'periodo_uuid': f'{periodo_2020_1.uuid}',
@@ -171,7 +169,7 @@ def test_api_list_prestacoes_conta_por_periodo_e_dre(jwt_authenticated_client_a,
             'unidade_tipo_unidade': 'EMEI',
             'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
             'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
-            'devolucao_ao_tesouro': '0,00'
+            'devolucao_ao_tesouro': 'Não'
         },
     ]
 
@@ -206,7 +204,7 @@ def test_api_list_prestacoes_conta_por_nome_unidade(jwt_authenticated_client_a,
             'unidade_tipo_unidade': 'EMEI',
             'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
             'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
-            'devolucao_ao_tesouro': '0,00'
+            'devolucao_ao_tesouro': 'Não'
         },
         {
             'periodo_uuid': f'{periodo_2019_2.uuid}',
@@ -257,7 +255,7 @@ def test_api_list_prestacoes_conta_por_nome_associacao(jwt_authenticated_client_
             'unidade_tipo_unidade': 'EMEI',
             'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
             'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
-            'devolucao_ao_tesouro': '0,00'
+            'devolucao_ao_tesouro': 'Não'
         },
         {
             'periodo_uuid': f'{periodo_2019_2.uuid}',
@@ -390,7 +388,7 @@ def test_api_list_prestacoes_conta_por_tecnico(jwt_authenticated_client_a,
             'unidade_tipo_unidade': 'EMEI',
             'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
             'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
-            'devolucao_ao_tesouro': '0,00'
+            'devolucao_ao_tesouro': 'Não'
         },
     ]
 
@@ -481,7 +479,7 @@ def test_api_list_prestacoes_conta_por_status_aprovada(
             'unidade_tipo_unidade': 'EMEI',
             'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
             'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
-            'devolucao_ao_tesouro': '0,00'
+            'devolucao_ao_tesouro': 'Não'
         }
     ]
 

diff --git a/...f_apps/core/tests/tests_api_prestacoes_contas/test_list_prestacoes_conta_nao_recebidas.py b/...f_apps/core/tests/tests_api_prestacoes_contas/test_list_prestacoes_conta_nao_recebidas.py
@@ -106,7 +106,6 @@ def _prestacao_conta_2020_1_unidade_a_dre1(periodo_2020_1, _unidade_a_dre_1, _as
         periodo=periodo_2020_1,
         associacao=_associacao_a_dre_1,
         data_recebimento=date(2020, 1, 1),
-        devolucao_tesouro=True,
         status='NAO_RECEBIDA'
     )
 
@@ -207,7 +206,6 @@ def _prestacao_conta_2020_1_unidade_a_dre1_em_analise(periodo_2020_1, _unidade_a
         periodo=periodo_2020_1,
         associacao=_associacao_a_dre_1,
         data_recebimento=date(2020, 1, 1),
-        devolucao_tesouro=True,
         status='EM_ANALISE'
     )
 

diff --git a/...apps/core/tests/tests_api_prestacoes_contas/test_list_prestacoes_conta_todos_os_status.py b/...apps/core/tests/tests_api_prestacoes_contas/test_list_prestacoes_conta_todos_os_status.py
@@ -107,7 +107,6 @@ def _prestacao_conta_2020_1_unidade_a_dre1_todos_os_status(periodo_2020_1, _unid
         periodo=periodo_2020_1,
         associacao=_associacao_a_dre_1_todos_os_status,
         data_recebimento=date(2020, 1, 1),
-        devolucao_tesouro=True,
         status='NAO_RECEBIDA'
     )
 
@@ -156,7 +155,6 @@ def _prestacao_conta_2020_1_unidade_a_dre1_em_analise(periodo_2020_1, _unidade_a
         periodo=periodo_2020_1,
         associacao=_associacao_a_dre_1_todos_os_status,
         data_recebimento=date(2020, 1, 1),
-        devolucao_tesouro=True,
         status='EM_ANALISE'
     )
 

diff --git a/sme_ptrf_apps/core/tests/tests_api_prestacoes_contas/test_reabre_prestacao_conta.py b/sme_ptrf_apps/core/tests/tests_api_prestacoes_contas/test_reabre_prestacao_conta.py
@@ -44,7 +44,6 @@ def prestacao_conta_01(periodo_01, associacao, motivo_aprovacao_ressalva_x, moti
         associacao=associacao,
         data_recebimento=date(2020, 10, 1),
         data_ultima_analise=date(2020, 10, 1),
-        devolucao_tesouro=True,
         motivos_reprovacao=[motivo_reprovacao_x, ],
         outros_motivos_reprovacao="Outros motivos reprovacao",
         motivos_aprovacao_ressalva=[motivo_aprovacao_ressalva_x, ],

diff --git a/sme_ptrf_apps/core/tests/tests_api_prestacoes_contas/test_salva_analise_prestacao_conta.py b/sme_ptrf_apps/core/tests/tests_api_prestacoes_contas/test_salva_analise_prestacao_conta.py
@@ -47,13 +47,10 @@ def tipo_devolucao_ao_tesouro():
 @freeze_time('2020-09-01')
 def test_api_salva_analise_prestacao_conta(jwt_authenticated_client_a, prestacao_conta_em_analise, conta_associacao,
                                            tipo_devolucao_ao_tesouro, despesa):
-    payload = {
-        'devolucao_tesouro': True,
-    }
 
     url = f'/api/prestacoes-contas/{prestacao_conta_em_analise.uuid}/salvar-analise/'
 
-    response = jwt_authenticated_client_a.patch(url, data=json.dumps(payload), content_type='application/json')
+    response = jwt_authenticated_client_a.patch(url, data={}, content_type='application/json')
 
     assert response.status_code == status.HTTP_200_OK
 
@@ -77,7 +74,6 @@ def test_api_salva_analise_prestacao_conta_nao_pode_aceitar_status_diferente_de_
                                                                                              prestacao_conta_recebida,
                                                                                              conta_associacao):
     payload = {
-        'devolucao_tesouro': True,
         'analises_de_conta_da_prestacao': [
             {
                 'conta_associacao': f'{conta_associacao.uuid}',

diff --git a/sme_ptrf_apps/core/tests/tests_prestacao_conta/test_prestacao_conta_model.py b/sme_ptrf_apps/core/tests/tests_prestacao_conta/test_prestacao_conta_model.py
@@ -2,8 +2,8 @@
 from django.contrib import admin
 from model_bakery import baker
 
-from ...models import PrestacaoConta, Associacao, Periodo, AnalisePrestacaoConta
-from sme_ptrf_apps.dre.models import MotivoAprovacaoRessalva
+from ...models import PrestacaoConta, Associacao, Periodo
+
 
 pytestmark = pytest.mark.django_db
 
@@ -20,7 +20,6 @@ def test_instance_model(prestacao_conta):
     assert model.status
     assert model.data_recebimento
     assert model.data_ultima_analise
-    assert model.devolucao_tesouro
     assert model.motivos_reprovacao
     assert model.outros_motivos_reprovacao
     assert model.motivos_aprovacao_ressalva

diff --git a/sme_ptrf_apps/dre/services/ata_parecer_tecnico_service.py b/sme_ptrf_apps/dre/services/ata_parecer_tecnico_service.py
@@ -49,7 +49,7 @@ def informacoes_execucao_financeira_unidades_ata_parecer_tecnico_consolidado_dre
         sequencia_de_publicacao = parcial['sequencia_de_publicacao_atual']
 
         if eh_parcial == "Parcial":
-            titulo_sequencia_publicacao = f'Parcial #{sequencia_de_publicacao}'
+            titulo_sequencia_publicacao = f'Publicação Parcial #{sequencia_de_publicacao}'
         else:
             titulo_sequencia_publicacao = "Ata final"
 
@@ -213,9 +213,9 @@ def get_presentes_na_ata(ata):
 
 
 def cria_data_geracao_documento(usuario, dre_nome):
-    data_geracao = datetime.datetime.now().strftime("%d/%m/%Y %H:%M:%S")
+    data_geracao = datetime.datetime.now().strftime("%d/%m/%Y às %H:%M:%S")
     quem_gerou = "" if usuario == "" else f"pelo usuário {usuario}"
-    texto = f"Ata PDF gerada pelo Sig_Escola em {data_geracao} {quem_gerou} para a DIRETORIA REGIONAL DE EDUCAÇÃO {formata_nome_dre(dre_nome)}"
+    texto = f"DRE {formata_nome_dre(dre_nome)} - Ata gerada {quem_gerou}, via SIG-Escola, em {data_geracao}"
 
     return texto
 

diff --git a/...i_relatorios_consolidados_dre/test_get_info_devolucoes_conta_relatorio_consolidado_dre.py b/...i_relatorios_consolidados_dre/test_get_info_devolucoes_conta_relatorio_consolidado_dre.py
@@ -42,7 +42,6 @@ def prestacao_conta(periodo, associacao):
         associacao=associacao,
         data_recebimento=date(2020, 10, 1),
         data_ultima_analise=date(2020, 10, 1),
-        devolucao_tesouro=True,
         status='APROVADA',
     )
 

diff --git a/...nsolidados_dre/test_get_info_execucao_financeira_por_unidade_relatorio_consolidado_dre.py b/...nsolidados_dre/test_get_info_execucao_financeira_por_unidade_relatorio_consolidado_dre.py
@@ -35,7 +35,6 @@ def prestacao_conta(periodo, associacao):
         associacao=associacao,
         data_recebimento=date(2020, 10, 1),
         data_ultima_analise=date(2020, 10, 1),
-        devolucao_tesouro=True,
         status='APROVADA',
     )
 

diff --git a/...elatorios_consolidados_dre/test_get_info_execucao_financeira_relatorio_consolidado_dre.py b/...elatorios_consolidados_dre/test_get_info_execucao_financeira_relatorio_consolidado_dre.py
@@ -37,7 +37,6 @@ def prestacao_conta(periodo, associacao):
         associacao=associacao,
         data_recebimento=date(2020, 10, 1),
         data_ultima_analise=date(2020, 10, 1),
-        devolucao_tesouro=True,
         status='APROVADA',
     )
 
@@ -247,7 +246,6 @@ def prestacao_conta_consolidado_dre(periodo_consolidado_dre, associacao, consoli
         associacao=associacao,
         data_recebimento=date(2020, 10, 1),
         data_ultima_analise=date(2020, 10, 1),
-        devolucao_tesouro=True,
         status='APROVADA',
         consolidado_dre=consolidado_dre
     )
@@ -276,7 +274,6 @@ def prestacao_conta_consolidado_dre_2(periodo_consolidado_dre, associacao_2_cons
         associacao=associacao_2_consolidado_dre,
         data_recebimento=date(2020, 10, 1),
         data_ultima_analise=date(2020, 10, 1),
-        devolucao_tesouro=True,
         status='APROVADA',
         consolidado_dre=consolidado_dre
     )

diff --git a/...onsolidados_dre/test_update_observacao_devolucoes_ao_tesouro_relatorio_consolidado_dre.py b/...onsolidados_dre/test_update_observacao_devolucoes_ao_tesouro_relatorio_consolidado_dre.py
@@ -20,7 +20,6 @@ def prestacao_conta(periodo, associacao):
         associacao=associacao,
         data_recebimento=date(2020, 10, 1),
         data_ultima_analise=date(2020, 10, 1),
-        devolucao_tesouro=True,
         status='APROVADA',
     )
 

diff --git a/...ios_consolidados_dre/test_update_observacao_devolucoes_conta_relatorio_consolidado_dre.py b/...ios_consolidados_dre/test_update_observacao_devolucoes_conta_relatorio_consolidado_dre.py
@@ -44,7 +44,6 @@ def prestacao_conta(periodo, associacao):
         associacao=associacao,
         data_recebimento=date(2020, 10, 1),
         data_ultima_analise=date(2020, 10, 1),
-        devolucao_tesouro=True,
         status='APROVADA',
     )