Merge pull request #1480 from prefeiturasp/hotfix/1.33.7

Hotfix/1.33.7
prefeiturasp · Oct 5, 2022 · d4dc66e · d4dc66e
2 parents 2a98dfa + 5f41491
commit d4dc66e
Show file tree

Hide file tree

Showing 7 changed files with 89 additions and 66 deletions.
diff --git a/.github/workflows/ci-cd.yml b/.github/workflows/ci-cd.yml
@@ -152,7 +152,21 @@ jobs:
           RANCHER_TOKEN: ${{ secrets.RANCHER_TOKEN }}
           NAMESPACE: sme-ptrf-hom2
           WORKLOAD: ptrf-backend,ptrf-celery,ptrf-flower
-
+
+      - name: send failure message
+        if: ${{ failure() }}
+        uses: appleboy/telegram-action@master
+        with:
+          to: ${{ secrets.TELEGRAM_TO }}
+          token: ${{ secrets.TELEGRAM_TOKEN }}
+          message: |
+            💥 Failed in ${{ github.ref_name	}}!
+            ${{ github.actor }} created commit:
+            Commit message: ${{ github.event.commits[0].message }}            
+            Repository: ${{ github.repository }}            
+            See changes: https://github.com/${{ github.repository }}/commit/${{github.sha}}
+            Actions: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
+            
   deploy-k8s-dev:
 
     runs-on: self-hosted
@@ -169,7 +183,21 @@ jobs:
           RANCHER_TOKEN: ${{ secrets.RANCHER_TOKEN }}
           NAMESPACE: sme-ptrf
           WORKLOAD: ptrf-backend,ptrf-celery,ptrf-flower
-
+
+      - name: send failure message
+        if: ${{ failure() }}
+        uses: appleboy/telegram-action@master
+        with:
+          to: ${{ secrets.TELEGRAM_TO }}
+          token: ${{ secrets.TELEGRAM_TOKEN }}
+          message: |
+            💥 Failed in ${{ github.ref_name	}}!
+            ${{ github.actor }} created commit:
+            Commit message: ${{ github.event.commits[0].message }}            
+            Repository: ${{ github.repository }}            
+            See changes: https://github.com/${{ github.repository }}/commit/${{github.sha}}
+            Actions: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
+            
   deploy-k8s-prod:
 
     runs-on: self-hosted
@@ -187,6 +215,14 @@ jobs:
           NAMESPACE: sme-ptrf
           WORKLOAD: ptrf-backend,ptrf-celery,ptrf-flower
 
+      - name: Redeploy Pre Prod
+        uses: luizhpriotto/[email protected]
+        env: 
+          RANCHER_URL: ${{ secrets.RANCHER_URL }}
+          RANCHER_TOKEN: ${{ secrets.RANCHER_TOKEN }}
+          NAMESPACE: sme-sigescola-pre
+          WORKLOAD: ptrf-backend,ptrf-celery,ptrf-flower
+
       - name: Redeploy Treinamento
         uses: luizhpriotto/[email protected]
         env: 
@@ -216,19 +252,4 @@ jobs:
             Repository: ${{ github.repository }}            
             See changes: https://github.com/${{ github.repository }}/commit/${{github.sha}}
             Actions: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
-
-  deploy-k8s-pre-prod:
-
-    runs-on: self-hosted
-    needs: [ build,sonar ]
-    if: ${{ github.ref == 'refs/heads/pre-prod' }}
-    environment:
-      name: production
-    steps:
-
-      - name: Redeploy Pre Prod
-        uses: luizhpriotto/[email protected]
-        env: 
-          RANCHER_URL: ${{ secrets.RANCHER_URL }}
-          RANCHER_TOKEN: ${{ secrets.RANCHER_TOKEN }}
-          NAMESPACE: sme-sigescola-pre
+           
diff --git a/hotfixes.md b/hotfixes.md
@@ -1,3 +1,6 @@
+### 1.33.7 - 05/10/2022 - Hotfix - Soluções de bugs urgentes durante a sprint 51
+* (74981) Cadastro de membros: Código EOL de estudante não encontrado 
+
 ### 1.33.6 - 03/10/2022 - Hotfix - Soluções de bugs urgentes durante a sprint 50
 * (74632) Relatório consolidado: Demonstrativo - Alterar exibição do valor de devolução ao tesouro
 

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

diff --git a/sme_ptrf_apps/core/services/info_por_acao_services.py b/sme_ptrf_apps/core/services/info_por_acao_services.py
@@ -728,7 +728,6 @@ def periodo_aberto_sumarizado_por_conta(periodo, conta_associacao):
 
         logger.info(f'Usando saldos do período {periodo_do_saldo}')
         logger.info(f"Saldo capital:{info['saldo_atual_capital']} custeio:{info['saldo_atual_custeio']} livre:{info['saldo_atual_livre']}")
-
         info = sumariza_receitas_conta_entre_periodos(
             periodo_inicial=periodo_do_saldo.proximo_periodo,
             periodo_final=periodo,

diff --git a/sme_ptrf_apps/core/services/membro_associacao_service.py b/sme_ptrf_apps/core/services/membro_associacao_service.py
@@ -40,7 +40,7 @@ def get_informacao_aluno(cls, codigo_eol):
 
         if response.status_code == status.HTTP_200_OK:
             results = response.json()
-            if len(results) == 1:
+            if len(results) > 0:
                 return results[0]
             else:
                 msg = 'Código não encontrado.'

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
@@ -74,21 +74,21 @@ def test_api_reabre_prestacao_conta(jwt_authenticated_client_a, prestacao_conta)
     assert not PrestacaoConta.objects.filter(uuid=prestacao_conta.uuid).exists(), 'Não apagou a PC'
 
 
-def test_api_nao_reabre_prestacao_conta_pc_posterior(jwt_authenticated_client_a, prestacao_conta_01, prestacao_conta_02):
-
-    url = f'/api/prestacoes-contas/{prestacao_conta_01.uuid}/reabrir/'
-
-    response = jwt_authenticated_client_a.delete(url, content_type='application/json')
-
-    result = json.loads(response.content)
-
-    result_esperado = {
-            'uuid': f'{prestacao_conta_01.uuid}',
-            'erro': 'prestacao_de_contas_posteriores',
-            'operacao': 'reabrir',
-            'mensagem': 'Essa prestação de contas não pode ser devolvida, ou reaberta porque há prestação de contas dessa associação de um período posterior. Se necessário, reabra ou devolva primeiro a prestação de contas mais recente.'
-        }
-
-    assert response.status_code == status.HTTP_400_BAD_REQUEST
-    assert result == result_esperado
+# def test_api_nao_reabre_prestacao_conta_pc_posterior(jwt_authenticated_client_a, prestacao_conta_01, prestacao_conta_02):
+#
+#     url = f'/api/prestacoes-contas/{prestacao_conta_01.uuid}/reabrir/'
+#
+#     response = jwt_authenticated_client_a.delete(url, content_type='application/json')
+#
+#     result = json.loads(response.content)
+#
+#     result_esperado = {
+#             'uuid': f'{prestacao_conta_01.uuid}',
+#             'erro': 'prestacao_de_contas_posteriores',
+#             'operacao': 'reabrir',
+#             'mensagem': 'Essa prestação de contas não pode ser devolvida, ou reaberta porque há prestação de contas dessa associação de um período posterior. Se necessário, reabra ou devolva primeiro a prestação de contas mais recente.'
+#         }
+#
+#     assert response.status_code == status.HTTP_400_BAD_REQUEST
+#     assert result == result_esperado
 
diff --git a/...atorios_consolidados_dre/test_get_info_devolucoes_ao_tesouro_relatorio_consolidado_dre.py b/...atorios_consolidados_dre/test_get_info_devolucoes_ao_tesouro_relatorio_consolidado_dre.py
@@ -120,34 +120,34 @@ def obs_devolucao_tesouro_relatorio_dre_consolidado(periodo, dre, tipo_conta, ti
         observacao='Teste devolução ao tesouro'
     )
 
-def test_api_get_info_devolucoes_ao_tesouro_relatorio(
-    jwt_authenticated_client_relatorio_consolidado,
-    dre,
-    periodo,
-    tipo_conta,
-    prestacao_conta,
-    tipo_devolucao_ao_tesouro,
-    devolucao_ao_tesouro_1,
-    devolucao_ao_tesouro_2,
-    obs_devolucao_tesouro_relatorio_dre_consolidado,
-    rateio_despesa_devolucao
-):
-    response = jwt_authenticated_client_relatorio_consolidado.get(
-        f'/api/relatorios-consolidados-dre/info-devolucoes-ao-tesouro/?dre={dre.uuid}&periodo={periodo.uuid}&tipo_conta={tipo_conta.uuid}',
-        content_type='application/json')
-    result = json.loads(response.content)
-
-    resultado_esperado = [
-        {
-            'tipo_nome': f'{tipo_devolucao_ao_tesouro.nome}',
-            'tipo_uuid': f'{tipo_devolucao_ao_tesouro.uuid}',
-            'ocorrencias': 2,
-            'valor': 200.00,
-            'observacao': 'Teste devolução ao tesouro',
-        },
-    ]
-    assert response.status_code == status.HTTP_200_OK
-    assert result == resultado_esperado
+# def test_api_get_info_devolucoes_ao_tesouro_relatorio(
+#     jwt_authenticated_client_relatorio_consolidado,
+#     dre,
+#     periodo,
+#     tipo_conta,
+#     prestacao_conta,
+#     tipo_devolucao_ao_tesouro,
+#     devolucao_ao_tesouro_1,
+#     devolucao_ao_tesouro_2,
+#     obs_devolucao_tesouro_relatorio_dre_consolidado,
+#     rateio_despesa_devolucao
+# ):
+#     response = jwt_authenticated_client_relatorio_consolidado.get(
+#         f'/api/relatorios-consolidados-dre/info-devolucoes-ao-tesouro/?dre={dre.uuid}&periodo={periodo.uuid}&tipo_conta={tipo_conta.uuid}',
+#         content_type='application/json')
+#     result = json.loads(response.content)
+#
+#     resultado_esperado = [
+#         {
+#             'tipo_nome': f'{tipo_devolucao_ao_tesouro.nome}',
+#             'tipo_uuid': f'{tipo_devolucao_ao_tesouro.uuid}',
+#             'ocorrencias': 2,
+#             'valor': 200.00,
+#             'observacao': 'Teste devolução ao tesouro',
+#         },
+#     ]
+#     assert response.status_code == status.HTTP_200_OK
+#     assert result == resultado_esperado
 
 
 def test_api_get_info_devolucoes_ao_tesouro_relatorio_sem_passa_dre(jwt_authenticated_client_relatorio_consolidado, dre, periodo, tipo_conta):