Draft
Conversation
I test predisposti nel file eservice-template.feature non furono portati avanti. La funzionalità di get delle versioni era comunque stata testata collateralmente in altri test, ad esempio [INTEROP-EST-M2M-VERSION-CREATE_02]. I test sono stati comunque rivisti e completati.
Durante il test non veniva incluso l'utente "security" come membro del client, come esplicitamente previsto dallo scenario (e come anche suggerito da Web UI)
Lo step "l'utente richiede il caricamento di una chiave pubblica di tipo "NON-RSA" di lunghezza 2048" aveva subito nel tempo dei cambiamenti che hanno portato a non accettare il parametro "NON-RSA".
Stando alla descrizione dello scenario il test era sbagliato, perché solo nei casi per ente uguale a PA1 si aveva un client appartenente allo stesso ente dell'utente chiamante.
Veniva effettuato il cast a un tipo di oggetto diverso da quello effettivamente restituito dalla API
Non veniva correttamente fatta l'associazione tra richiesta di fruizione ed e-service pubblicato. Risolto attraverso refactor dello step tenantHasAlreadyCreatedEservicesWithSpecificState, migliorandone anche la leggibilità. E' stata collateralmente spostata la logica di creazione e-service in un certo stato dallo step createEservice al metodo createEServiceInState. Variazione testate su tutto il file feature e-service-catalog-listing.feature e sui test [M2MG_ESERVICES_41_A], [M2MG_ESERVICES_47_A] e [M2MG_ESERVICES_48_A] con successo.
Il test necessitava di fix e refactor multipli, fra cui: 1. Introduzione dello step di autenticazione M2M 2. Disattivazione dell'example per offset invalido: in questa configurazione, il client intercetta l'errore prima senza effettuare la chiamata, e quindi non c'è un reale test dell'endpoint. Come per altri casi simili si è scelto quindi di non testare questa configurazione. 3. Sostituzione di %null con %blank negli ultimi due examples: %null si traduce nel NON usare il filtro, che è un'operazione lecita; %blank si traduce nel passarlo vuoto (es. 'purposeTitle='), che potrebbe non esserlo (da stabilire) 4. Uso del corretto basePath all'interno del client 5. Setting del bearer token all'interno del client 6. Correzione del parsing di TargetTenantKind 7. Posto 'isFreeOfCharge' a false: se lo si pone a true, allora l'attributo freeOfChargeReason deve essere valorizzato. Non essendo essenziali per i test, si è scelto di rimuoverli entrambi 8. Sostituito buildMandatoryPurposeTemplateSeed con il più aggiornato e funzionante prepareCreationRequest 9. Introdotto polling attivo in vari punti 10. Gestiti casi in cui il risultato atteso è negativo
Il parametro passato era una semplice data, ma quello voluto è un timestamp con fuso orario annesso, come indicato anche nell'OpenAPI BFF. Può aver tratto in inganno il nome "expirationDate".
Veniva impropriamente fatta una seconda chiamata senza il corretto offset
Veniva impropriamente fatta una seconda chiamata senza il corretto offset
…nrt-complementare
Veniva impropriamente fatta una seconda chiamata senza il corretto offset
Lo status code non veniva reperito correttamente
Il parametro passato era una semplice data, ma quello voluto è un timestamp con fuso orario annesso, come indicato anche nell'OpenAPI BFF. Può aver tratto in inganno il nome "expirationDate".
Il parametro passato era una semplice data, ma quello voluto è un timestamp con fuso orario annesso. Oltre questo, non veniva settato il corretto token d'autenticazione.
…_AND_KEYS_13] L'aggiunta del primo client id veniva erroneamente fatta due volte, la seconda attraverso lo step <un "admin" di "PA1" ha caricato una chiave pubblica nel client>. Fix testata su tutto il set client, dpop e voucher, con nessun KO imprevisto.
La fix applicata al commit bc520f6 involontariamente causava il fallimento di altri test come effetto collaterale. E' stata fatta esplicita distinzione tra il comportamento di suddetti test ed il comportamento del test in oggetto, dividendo in due test.
Introdotto NrtConcurrentTest per esecuzione in concorrenza della NRT generale, e organizzate le due classi di test in una gerarchia guidata da AbstractNrtTest
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
QA-13460
Contenuti fix per svariati test