Merge pull request #2284 from prefeiturasp/feature/102412-conciliacao…

…-bancaria-retirar-obrigatoriedade feat(102412): Geração de documentos/Conciliação Bancária: Retirar validação e obrigatoriedade de preenchimento dos campos do Saldo bancário da conta ao concluir acerto/período
prefeiturasp · Aug 28, 2023 · de37ecf · de37ecf
2 parents 14883f0 + 6cc085e
commit de37ecf
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 14 deletions.
diff --git a/sme_ptrf_apps/core/api/views/associacoes_viewset.py b/sme_ptrf_apps/core/api/views/associacoes_viewset.py
@@ -234,12 +234,22 @@ def status_periodo(self, request, uuid=None):
         if prestacao_conta:
             gerar_previas = pc_requer_geracao_documentos(prestacao_conta)
 
+        # TODO código comentado propositalmente em função da história 102412 - Sprint 73 (Conciliação Bancária: Retirar validação e obrigatoriedade de preenchimento dos campos do Saldo bancário da conta ao concluir acerto/período) - que entrou como Hotfix
+        # TODO Remover quando implementado solução definitiva
         pendencias_dados = associacao.pendencias_dados_da_associacao_para_geracao_de_documentos()
-        pendencias_conciliacao = associacao.pendencias_conciliacao_bancaria_por_periodo_para_geracao_de_documentos(periodo)
-        if pendencias_dados or pendencias_conciliacao:
+        pendencias_conciliacao = associacao.pendencias_conciliacao_bancaria_por_periodo_para_geracao_de_documentos(
+            periodo)
+
+        # if pendencias_dados or pendencias_conciliacao:
+        #     pendencias_cadastrais = {
+        #         'dados_associacao': pendencias_dados,
+        #         'conciliacao_bancaria': pendencias_conciliacao,
+        #     }
+
+        if pendencias_dados:
             pendencias_cadastrais = {
                 'dados_associacao': pendencias_dados,
-                'conciliacao_bancaria': pendencias_conciliacao,
+                'conciliacao_bancaria': None,
             }
         else:
             pendencias_cadastrais = None
@@ -740,11 +750,11 @@ def valida_data_de_encerramento(self, request, uuid=None):
                 logger.info('Erro: %r', erro)
                 return Response(erro, status=status.HTTP_400_BAD_REQUEST)
 
-
         data_de_encerramento = datetime.datetime.strptime(data_de_encerramento, '%Y-%m-%d')
         data_de_encerramento = data_de_encerramento.date()
 
-        response = ValidaDataDeEncerramento(associacao=associacao, data_de_encerramento=data_de_encerramento, periodo_inicial=periodo_inicial).response
+        response = ValidaDataDeEncerramento(associacao=associacao, data_de_encerramento=data_de_encerramento,
+                                            periodo_inicial=periodo_inicial).response
 
         status_response = response.pop("status")
 

diff --git a/sme_ptrf_apps/core/tests/tests_api_associacoes/test_action_status_periodo.py b/sme_ptrf_apps/core/tests/tests_api_associacoes/test_action_status_periodo.py
@@ -237,15 +237,18 @@ def test_status_periodo_pendencias_cadastrais_com_contas_pendentes(
     observacao_conciliacao_campos_nao_preenchidos_002,
     periodo_2020_1
 ):
+    # TODO código comentado propositalmente em função da história 102412 - Sprint 73 (Conciliação Bancária: Retirar validação e obrigatoriedade de preenchimento dos campos do Saldo bancário da conta ao concluir acerto/período) - que entrou como Hotfix
+    # TODO Remover quando implementado solução definitiva
 
     response = jwt_authenticated_client_a.get(f'/api/associacoes/{associacao.uuid}/status-periodo/?data={periodo_2020_1.data_inicio_realizacao_despesas}',
                           content_type='application/json')
     result = json.loads(response.content)
 
     pendencias_cadastrais_esperado = {
-        'conciliacao_bancaria': {
-            'contas_pendentes': [f'{observacao_conciliacao_campos_nao_preenchidos.conta_associacao.uuid}', f'{observacao_conciliacao_campos_nao_preenchidos_002.conta_associacao.uuid}',],
-        },
+        # 'conciliacao_bancaria': {
+        #     'contas_pendentes': [f'{observacao_conciliacao_campos_nao_preenchidos.conta_associacao.uuid}', f'{observacao_conciliacao_campos_nao_preenchidos_002.conta_associacao.uuid}',],
+        # },
+        'conciliacao_bancaria': None,
         'dados_associacao': {
             'pendencia_cadastro': False,
             'pendencia_contas': False,
@@ -264,14 +267,18 @@ def test_status_periodo_pendencias_cadastrais_somente_uma_conta_pendente(
     periodo_2020_1
 ):
 
+    # TODO código comentado propositalmente em função da história 102412 - Sprint 73 (Conciliação Bancária: Retirar validação e obrigatoriedade de preenchimento dos campos do Saldo bancário da conta ao concluir acerto/período) - que entrou como Hotfix
+    # TODO Remover quando implementado solução definitiva
+
     response = jwt_authenticated_client_a.get(f'/api/associacoes/{associacao.uuid}/status-periodo/?data={periodo_2020_1.data_inicio_realizacao_despesas}',
                           content_type='application/json')
     result = json.loads(response.content)
 
     pendencias_cadastrais_esperado = {
-        'conciliacao_bancaria': {
-            'contas_pendentes': [f'{observacao_conciliacao_campos_nao_preenchidos_002.conta_associacao.uuid}',],
-        },
+        # 'conciliacao_bancaria': {
+        #     'contas_pendentes': [f'{observacao_conciliacao_campos_nao_preenchidos_002.conta_associacao.uuid}',],
+        # },
+        'conciliacao_bancaria': None,
         'dados_associacao': {
             'pendencia_cadastro': False,
             'pendencia_contas': False,
@@ -335,6 +342,9 @@ def test_status_periodo_todas_as_pendencias_cadastrais(
     periodo_2020_1,
 ):
 
+    # TODO código comentado propositalmente em função da história 102412 - Sprint 73 (Conciliação Bancária: Retirar validação e obrigatoriedade de preenchimento dos campos do Saldo bancário da conta ao concluir acerto/período) - que entrou como Hotfix
+    # TODO Remover quando implementado solução definitiva
+
     response = jwt_authenticated_client_a.get(f'/api/associacoes/{conta_associacao_incompleta.associacao.uuid}/status-periodo/?data={periodo_2020_1.data_inicio_realizacao_despesas}',
                           content_type='application/json')
     result = json.loads(response.content)
@@ -345,9 +355,10 @@ def test_status_periodo_todas_as_pendencias_cadastrais(
             'pendencia_membros': True,
             'pendencia_contas': True,
         },
-        'conciliacao_bancaria': {
-            'contas_pendentes': [f'{conta_associacao_incompleta.uuid}']
-        },
+        'conciliacao_bancaria': None,
+        # 'conciliacao_bancaria': {
+        #     'contas_pendentes': [f'{conta_associacao_incompleta.uuid}']
+        # },
     }
 
     assert response.status_code == status.HTTP_200_OK