Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix test non funzionanti #18

Closed
wants to merge 1 commit into from

Conversation

FaberNa
Copy link

@FaberNa FaberNa commented Jul 25, 2021

Ho corretto alcuni test che fallivano, per via del cambio da base64 ad xml

@peppelinux peppelinux requested a review from marque88 July 25, 2021 15:22
@peppelinux
Copy link
Member

Ciao @FaberNa
In attesa di una revisione di @marque88 che ne pensi se introducessimo meglio nel readme come eseguire i test e come deployare una installazione demo di questo progetto?

Dimmi cosa ne pensi e se ti andasse di spingere questo contributo. Raggiungici su #spid-spring di Developers Italia slack per parlarne tutti insieme

@FaberNa
Copy link
Author

FaberNa commented Jul 25, 2021

ok, perfetto .... ma intanto cosa ne pensi se si cambiasse qualcosa anche in nel file di travis in modo che partissero anche i test completi ( per qualche motivo quelli di integration sembra non ci siano nell flow ) .... cosa ne pensi ?
https://travis-ci.org/github/italia/spid-spring
image

@FaberNa
Copy link
Author

FaberNa commented Jul 25, 2021 via email

@peppelinux
Copy link
Member

Ottima idea @FaberNa, devo confessarti che ad oggi preferisco di gran lunga github actions, è più svelto d'esecuzione e non in ultimo l'utilizzo di act in locale non è niente male ... Che ne pensi?

Sulla necessita dei test di integration in CI sfondi una porta aperta da parte mia, preferisco di gran lunga questi ai soli unitari, l'ideale sarebbe ottenere un example project sul quale avviare sia i test di integrazione propri del progetto che la SPID QA mediante spid-sp-test.

Dai uno sguardo qui:
https://github.com/italia/Satosa-Saml2Spid/runs/3153957088

come possiamo vedere testo uno IAM proxy tirando su un SP demo e usando come moke idp appunto spid-sp-test.
Complessivamente so per certo che l'example project, con una minima personalizzazione del template, certificati e entityid, passerebbe senza problemi lo SPID onboarding.

Parliamone, è vero che è tanta roba ma "si può fare" e credimi ... Ne vale la pena :)
https://github.com/italia/spid-sp-test

@marque88 marque88 closed this Jul 26, 2021
@marque88
Copy link
Collaborator

Chiusa perché presente già in Enhancement/wip java11

@FaberNa FaberNa deleted the fix/test-SpidIntegration branch July 29, 2021 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants