Merge pull request #2619 from prefeiturasp/hotfix/8.16.1

Hotfix/8.16.1

sme_ptrf_apps/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "8.16.0"
+__version__ = "8.16.1"
 
 __version_info__ = tuple(
     [

sme_ptrf_apps/core/api/serializers/acao_associacao_serializer.py

@@ -58,6 +58,18 @@ class Meta:
         model = AcaoAssociacao
         fields = ('uuid', 'associacao', 'acao', 'status')
 
+    def validate(self, data):
+        from sme_ptrf_apps.core.services.acoes_associacoes_service import validate_acao_associacao
+        associacao = data['associacao'] if 'associacao' in data else None
+        acao = data['acao'] if 'acao' in data else None
+
+        if associacao and acao:
+            try:
+                validate_acao_associacao(associacao, acao, self.instance)
+            except Exception as error:
+                raise error
+        return data
+
 
 class AcaoAssociacaoRetrieveSerializer(serializers.ModelSerializer):
     associacao = AssociacaoListSerializer()
@@ -73,4 +85,5 @@ def get_tooltip_associacao_encerrada(self, obj):
 
     class Meta:
         model = AcaoAssociacao
-        fields = ('uuid', 'id', 'associacao', 'data_de_encerramento_associacao', 'tooltip_associacao_encerrada', 'acao', 'status', 'criado_em')
+        fields = ('uuid', 'id', 'associacao', 'data_de_encerramento_associacao',
+                  'tooltip_associacao_encerrada', 'acao', 'status', 'criado_em')

sme_ptrf_apps/core/api/views/acao_associacao_viewset.py

@@ -1,5 +1,4 @@
 from django.db.models import Q
-
 from django_filters import rest_framework as filters
 
 from rest_framework import mixins, status

sme_ptrf_apps/core/models/acao_associacao.py

@@ -2,7 +2,6 @@
 
 from django.db import models
 from django.db.models.deletion import ProtectedError
-
 from sme_ptrf_apps.core.models_abstracts import ModeloBase
 from auditlog.models import AuditlogHistoryField
 from auditlog.registry import auditlog
@@ -25,7 +24,8 @@ class AcaoAssociacao(ModeloBase):
     )
 
     associacao = models.ForeignKey('Associacao', on_delete=models.CASCADE, related_name='acoes', blank=True, null=True)
-    acao = models.ForeignKey('Acao', on_delete=models.PROTECT, blank=True, null=True, related_name='associacoes_da_acao')
+    acao = models.ForeignKey('Acao', on_delete=models.PROTECT, blank=True,
+                             null=True, related_name='associacoes_da_acao')
     status = models.CharField(
         'status',
         max_length=15,

sme_ptrf_apps/core/services/acoes_associacoes_service.py

@@ -1,7 +1,19 @@
-from ..models import Associacao
+from django.core.exceptions import ValidationError
+from ..models import Associacao, AcaoAssociacao
 
 
 def associacoes_nao_vinculadas_a_acao(acao):
     associacoes_vinculadas = acao.associacoes_da_acao.values_list('associacao__id', flat=True)
     result = Associacao.objects.exclude(id__in=associacoes_vinculadas)
     return result
+
+
+def validate_acao_associacao(associacao, acao, instance=None):
+    existing_records = AcaoAssociacao.objects.filter(
+        associacao=associacao,
+        acao=acao,
+        status=AcaoAssociacao.STATUS_ATIVA
+    ).exclude(pk=instance.pk if instance else None)
+
+    if existing_records.exists():
+        raise ValidationError('Já existe um registro com a mesma associação e ação com status ativa')

sme_ptrf_apps/core/tests/tests_api_acoes_associacoes/test_acao_associacao_create.py

@@ -4,6 +4,7 @@
 from rest_framework import status
 
 from sme_ptrf_apps.core.models import AcaoAssociacao
+from sme_ptrf_apps.core.fixtures.factories import AcaoFactory, AcaoAssociacaoFactory, AssociacaoFactory
 
 pytestmark = pytest.mark.django_db
 
@@ -122,3 +123,22 @@ def test_create_acao_associacao_em_lote_uuid_associacao_invalido(
     assert response.status_code == status.HTTP_201_CREATED
     assert result == esperado
     assert AcaoAssociacao.objects.exists()
+
+
+def test_validacao_create_acao_associacao(jwt_authenticated_client_a):
+    associacao = AssociacaoFactory()
+    acao = AcaoFactory()
+    acao_associacao_existente = AcaoAssociacaoFactory(associacao=associacao, acao=acao)
+
+    payload = {
+        'acao': f"{acao.uuid}",
+        'associacao': f"{associacao.uuid}"
+    }
+
+    response = jwt_authenticated_client_a.post(
+        '/api/acoes-associacoes/', data=json.dumps(payload), content_type='application/json')
+
+    result = json.loads(response.content)
+
+    assert response.status_code == status.HTTP_400_BAD_REQUEST
+    assert 'Já existe um registro com a mesma associação e ação com status ativa' in str(result)

sme_ptrf_apps/core/tests/tests_api_acoes_associacoes/test_acao_associacao_update.py

@@ -4,6 +4,7 @@
 from rest_framework import status
 
 from sme_ptrf_apps.core.models import AcaoAssociacao
+from sme_ptrf_apps.core.fixtures.factories import AcaoFactory, AcaoAssociacaoFactory, AssociacaoFactory
 
 pytestmark = pytest.mark.django_db
 
@@ -39,3 +40,24 @@ def test_update_acao_associacao(
 
     assert acao_associacao.acao == acao_y, "Deve atualizar a ação."
     assert acao_associacao.status == AcaoAssociacao.STATUS_INATIVA, "Deve atualizar o status."
+
+
+def test_validacao_update_acao_associacao(jwt_authenticated_client_a):
+    associacao = AssociacaoFactory()
+    acao = AcaoFactory()
+    outra_acao = AcaoFactory()
+
+    acao_associacao_existente = AcaoAssociacaoFactory(associacao=associacao, acao=acao)
+
+    payload = {
+        'acao': f"{outra_acao.uuid}",
+        'associacao': f"{associacao.uuid}"
+    }
+
+    response = jwt_authenticated_client_a.put(
+        f'/api/acoes-associacoes/{acao_associacao_existente.uuid}/', data=json.dumps(payload), content_type='application/json')
+
+    result = json.loads(response.content)
+
+    assert response.status_code == status.HTTP_200_OK
+    assert result['acao'] == f"{outra_acao.uuid}"

sme_ptrf_apps/despesas/api/serializers/despesa_serializer.py

@@ -138,8 +138,8 @@ def create(self, validated_data):
 
         despesas_impostos = validated_data.pop('despesas_impostos') if validated_data.get('despesas_impostos') else None
 
-        motivos_pagamento_antecipado = validated_data.pop('motivos_pagamento_antecipado')
-        outros_motivos_pagamento_antecipado = validated_data['outros_motivos_pagamento_antecipado']
+        motivos_pagamento_antecipado = validated_data.pop('motivos_pagamento_antecipado', None)
+        outros_motivos_pagamento_antecipado = validated_data.get('outros_motivos_pagamento_antecipado', None)
 
         data_transacao = validated_data['data_transacao']
         data_documento = validated_data['data_documento']
@@ -253,8 +253,8 @@ def update(self, instance, validated_data):
 
         despesas_impostos = validated_data.pop('despesas_impostos') if 'despesas_impostos' in validated_data else []
 
-        motivos_pagamento_antecipado = validated_data.pop('motivos_pagamento_antecipado')
-        outros_motivos_pagamento_antecipado = validated_data['outros_motivos_pagamento_antecipado']
+        motivos_pagamento_antecipado = validated_data.pop('motivos_pagamento_antecipado', None)
+        outros_motivos_pagamento_antecipado = validated_data.get('outros_motivos_pagamento_antecipado', None)
 
         data_transacao = validated_data['data_transacao']
         data_documento = validated_data['data_documento']