search

italia / spid-spring

SPID extension for Java Spring
BSD 3-Clause "New" or "Revised" License
29 stars 22 forks source link

Fix test non funzionanti #18

Closed FaberNa closed 3 years ago

FaberNa commented 3 years ago

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

peppelinux commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

CIao, se vuoi puoi aggiungermi al canale, @.*** ... Grazie, Ciao Fabrizio

Il giorno dom 25 lug 2021 alle ore 17:25 Giuseppe De Marco < @.***> ha scritto:

Ciao @FaberNa https://github.com/FaberNa In attesa di una revisione di @marque88 https://github.com/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

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/italia/spid-spring/pull/18#issuecomment-886217440, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACEURO3GMBKXSBHKMBCURZ3TZQUGRANCNFSM5A6TO5QA .

peppelinux commented 3 years ago

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 commented 3 years ago

Chiusa perché presente già in Enhancement/wip java11