Verificar query e retorno de Services_GET

[paulojna]

GET /applications/services

• query a filtrar a collection pelos valores procurados
• schema a garantir exclusão mútua
• validação do formato UUID no serInstanceId
• Tratamento de erros e envio de uma instância ProblemDetails

[jnbastos]

Alterações feitas em:
models.py
+ class ServiceGet
schemas.py
+ service_get_schema
services_controller.py
+ class ServicesController:
utils.py
+ def none_to_empty_brackets
+ def camel_to_snake

commits:
ATNoG/netedge-mep@main...UMinho-Netedge:netedge-mep-uminho:serv_get

[nflori]

Olá João, Qual o link onde estão as mais recentes versões do abaixo? models.py schemas.py services_controller.py utils.py Obrigado. Nicolás

…

________________________________ De: João Bastos ***@***.***> Enviado: 12 de julho de 2022 16:45:58 Para: UMinho-Netedge/netedge-mep-uminho Cc: Nicolas Francisco Lori; Assign Assunto: Re: [UMinho-Netedge/netedge-mep-uminho] Verificar query e retorno de Services_GET (Issue #6) Alterações feitas em: models.py schemas.py services_controller.py utils.py — Reply to this email directly, view it on GitHub<#6 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVRCTTXAWUFVR3UFUBLMXCLVTWHLNANCNFSM5Z6OBD7Q>. You are receiving this because you were assigned.Message ID: ***@***.***>

[jnbastos]

Olá Nicolás,

Segue o link da branch onde fiz as alterações:

https://github.com/UMinho-Netedge/netedge-mep-uminho/tree/serv_get

Para revisão aconselho a utilização da ferramento GitKraken (https://www.gitkraken.com/), que permite ver com muito maior facilidade todas as alterações feitas no código nas diferentes branches, permitindo ter uma vista mais geral do código também.

João

[nflori]

Olá João, Podemos ter uma reunião os dois para falarmos um pouco do kraken. Já fiz o download do Kraken e liguei-o ao github, mas não percebo como obtenho a imagem que me mostraste. Se podemos reunir, dá-me um link e eu entro no link para partilharmos o ecran. Obrigado. Nicolas

…

________________________________ De: João Bastos ***@***.***> Enviado: 14 de julho de 2022 13:53:52 Para: UMinho-Netedge/netedge-mep-uminho Cc: Nicolas Francisco Lori; Assign Assunto: Re: [UMinho-Netedge/netedge-mep-uminho] Verificar query e retorno de Services_GET (Issue #6) Olá Nicolás, Segue o link da branch onde fiz as alterações: https://github.com/UMinho-Netedge/netedge-mep-uminho/tree/serv_get Para revisão aconselho a utilização da ferramento GitKraken (https://www.gitkraken.com/), que permite ver com muito maior facilidade todas as alterações feitas no código nas diferentes branches, permitindo ter uma vista mais geral do código também. João — Reply to this email directly, view it on GitHub<#6 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AVRCTTTA5XDYDGXXGD62MH3VUAEWBANCNFSM5Z6OBD7Q>. You are receiving this because you were assigned.Message ID: ***@***.***>

[jnbastos]

Claro. Segue o link do zoom: https://videoconf-colibri.zoom.us/j/87005435677

[nflori]

Perguntas de Nicolas e respostas de João Bastos:

1. acerca do models.py: como consegues passar de 2 blocos que definem o dict para apenas um bloco?
   As linhas 821 a 835 form removidas porque ainda estava em fase de testes e estava a tentar perceber como funciona o mecanismo. Mas depois vi que o melhor seria criar a função interna bool_conv. Na remoção da 809 a 819 que depois virou em adição das 813 a 821 se reparar a única alteração foi mesmo de identação pois acrescentei mais um tab,

2. acerca do schemas.py: porque é que os "not required" mudaram?
   Porque não estavam a garantir a exclusão mútua do ser_instance_id, ser_category_id e ser_name, então alterei para quando fizer validate ele recorra ao schema e ele lance um erro caso não haja a tal exclusão mútua.

[nflori]

Declaração de João Bastos:
Já corrigi a validação do formato UUID para o caso em que se passa uma lista de ser_instance_ids. No gitKraken pode se ver o update com a descrição "Update services_get: validate UUID format for multiple ser_instance_id" onde se encontram modificações em dois ficheiros mas a única que interessa é do services_controller.py.

Resposta de Nicolas Lori:
Vi e pareceu-me bem.