Merge remote-tracking branch 'origin/refactor/QA-4334-Refactoring-dei…

…-Test-Destinatario-Persona-Fisica-bis' into refactor/QA-4069-Roadmap-per-l-Integrazione-di-Spring-dopo-l-integrazione-con-JUnit-5
pagopa · Nov 5, 2024 · abd67e6 · abd67e6
2 parents e191e78 + 12ca27b
commit abd67e6
Show file tree

Hide file tree

Showing 10 changed files with 36 additions and 43 deletions.
diff --git a/...ava/it/pn/frontend/e2e/pages/destinatario/personaGiuridica/AccediAreaRiservataPGPage.java b/...ava/it/pn/frontend/e2e/pages/destinatario/personaGiuridica/AccediAreaRiservataPGPage.java
@@ -11,7 +11,9 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.stereotype.Component;
 
+@Component
 public class AccediAreaRiservataPGPage extends BasePage {
     private final Logger logger = LoggerFactory.getLogger("AccediAreaRiservataPGPage");
 

diff --git a/src/main/java/it/pn/frontend/e2e/pages/destinatario/personaGiuridica/GruppiPGPage.java b/src/main/java/it/pn/frontend/e2e/pages/destinatario/personaGiuridica/GruppiPGPage.java
@@ -24,7 +24,6 @@
 @Component
 public class GruppiPGPage extends BasePage {
 
-    private final AccediAreaRiservataPGPage accediAreaRiservataPGPage = new AccediAreaRiservataPGPage(driver);
     private final Actions actions = new Actions(driver);
 
     @FindBy(id = "side-item-Gruppi")
@@ -47,6 +46,14 @@ public class GruppiPGPage extends BasePage {
 
     @Autowired
     private WebTool webTool;
+    @Autowired
+    private AccediAreaRiservataPGPage accediAreaRiservataPGPage;
+    @Autowired
+    private ScegliSpidPGPage scegliSpidPGPage ;
+    @Autowired
+    private LoginPGPagoPAPage loginPGPagoPAPage;
+    @Autowired
+    private AutorizzaInvioDatiPGPage autorizzaInvioDatiPGPage ;
 
     @Autowired
     public GruppiPGPage(WebDriver driver) {
@@ -64,14 +71,14 @@ public void clickGruppiButton() {
     public void loginGruppi(String nome, String pwd) {
         accediAreaRiservataPGPage.waitLoadAccediAreaRiservataPGPage();
         accediAreaRiservataPGPage.clickSpidButton();
-        ScegliSpidPGPage scegliSpidPGPage = new ScegliSpidPGPage(driver);
+
         scegliSpidPGPage.clickTestButton();
-        LoginPGPagoPAPage loginPGPagoPAPage = new LoginPGPagoPAPage(driver);
+
         loginPGPagoPAPage.waitLoadLoginPGPage();
         loginPGPagoPAPage.insertUsername(nome);
         loginPGPagoPAPage.insertPassword(pwd);
         loginPGPagoPAPage.clickInviaButton();
-        AutorizzaInvioDatiPGPage autorizzaInvioDatiPGPage = new AutorizzaInvioDatiPGPage(driver);
+
         autorizzaInvioDatiPGPage.waitLoadAutorizzaInvioDatiPGPage();
         autorizzaInvioDatiPGPage.clickInviaButton();
     }

diff --git a/.../it/pn/frontend/e2e/pages/destinatario/personaGiuridica/PiattaformaNotifichePGPAPage.java b/.../it/pn/frontend/e2e/pages/destinatario/personaGiuridica/PiattaformaNotifichePGPAPage.java
@@ -79,6 +79,14 @@ public class PiattaformaNotifichePGPAPage extends BasePage {
     @Autowired
     private WebDriverConfig webDriverConfig;
     @Autowired
+    private DownloadFile downloadFile;
+    @Autowired
+    private DettaglioNotificaMittenteSection dettaglioNotificaMittenteSection;
+    @Autowired
+    private HelpdeskPage helpdeskPage ;
+    @Autowired
+    private DettaglioNotificaSection dettaglioNotificaSection ;
+    @Autowired
     @Lazy
     private  WebTool webTool;
 
@@ -273,8 +281,6 @@ public void clickRicevutaDiConsegna() throws AWTException, IOException {
             // Generate a unique filename for the downloaded ZIP file
             String fileName = "downloaded_" + System.currentTimeMillis() + ".zip";
 
-            DownloadFile downloadFile = new DownloadFile(this.driver);
-            DettaglioNotificaMittenteSection dettaglioNotificaMittenteSection = new DettaglioNotificaMittenteSection(this.driver);
             String codiceIUN;
             webTool.waitTime(1);
             codiceIUN = dettaglioNotificaMittenteSection.getInfoNotifica(3);
@@ -344,7 +350,6 @@ public void extractZipWithoutPassword() throws IOException {
         String zipDirectoryPath = workingDirectory + "/src/test/resources/dataPopulation/zip";
         String extractDirectoryPath = zipDirectoryPath + "/extract";
 
-        HelpdeskPage helpdeskPage = new HelpdeskPage(this.driver);
         // Find the latest ZIP file
         File latestZipFile = helpdeskPage.findLatestZipFile(zipDirectoryPath);
         if (latestZipFile == null) {
@@ -364,8 +369,7 @@ public void extractZipWithoutPassword() throws IOException {
     }
 
    public void controllaTesto(String nomeFile) {
-       DettaglioNotificaSection dettaglioNotificaSection = new DettaglioNotificaSection(this.driver);
-       DettaglioNotificaMittenteSection dettaglioNotificaMittenteSection = new DettaglioNotificaMittenteSection(this.driver);
+
        Map<String, String> infoNotifiche = dettaglioNotificaSection.recuperoInfoNotificheDestinatario();
        if (nomeFile.contains("PN_NOTIFICATION_ATTACHMENTS")) {
            if (dettaglioNotificaMittenteSection.controlloTestoFile(nomeFile, "A Simple PDF File")) {

diff --git a/src/main/java/it/pn/frontend/e2e/pages/destinatario/personaGiuridica/UtentiPGPage.java b/src/main/java/it/pn/frontend/e2e/pages/destinatario/personaGiuridica/UtentiPGPage.java
@@ -21,8 +21,9 @@
 public class UtentiPGPage extends BasePage {
 
     private final Logger logger = LoggerFactory.getLogger("UtentiPGPage");
-    private final AccediAreaRiservataPGPage accediAreaRiservataPGPage = new AccediAreaRiservataPGPage(this.driver);
+
     private final Actions actions = new Actions(this.driver);
+
     @FindBy(xpath = "//button[contains(text(),'Aggiungi utente')]")
     WebElement addUserButton;
     @FindBy(xpath = "//button[contains(text(),'Indietro')]")
@@ -77,9 +78,18 @@ public class UtentiPGPage extends BasePage {
     @Autowired
     private WebDriverConfig webDriverConfig;
     @Autowired
+    private ScegliSpidPGPage scegliSpidPGPage ;
+    @Autowired
+    private AccediAreaRiservataPGPage accediAreaRiservataPGPage;
+    @Autowired
+    private LoginPGPagoPAPage loginPGPagoPAPage;
+    @Autowired
+    private AutorizzaInvioDatiPGPage autorizzaInvioDatiPGPage;
+    @Autowired
     @Lazy
     private  WebTool webTool;
 
+
     @Autowired
     public UtentiPGPage(WebDriver driver) {
         super(driver);
@@ -113,16 +123,16 @@ public void loginUtenti(String nome, String pwd) {
         accediAreaRiservataPGPage.waitLoadAccediAreaRiservataPGPage();
         accediAreaRiservataPGPage.clickSpidButton();
 
-        ScegliSpidPGPage scegliSpidPGPage = new ScegliSpidPGPage(this.driver);
+
         scegliSpidPGPage.clickTestButton();
 
-        LoginPGPagoPAPage loginPGPagoPAPage = new LoginPGPagoPAPage(this.driver);
+
         loginPGPagoPAPage.waitLoadLoginPGPage();
         loginPGPagoPAPage.insertUsername(nome);
         loginPGPagoPAPage.insertPassword(pwd);
         loginPGPagoPAPage.clickInviaButton();
 
-        AutorizzaInvioDatiPGPage autorizzaInvioDatiPGPage = new AutorizzaInvioDatiPGPage(this.driver);
+
         autorizzaInvioDatiPGPage.waitLoadAutorizzaInvioDatiPGPage();
         autorizzaInvioDatiPGPage.clickInviaButton();
     }

diff --git a/src/main/java/it/pn/frontend/e2e/pages/mittente/PiattaformaNotifichePage.java b/src/main/java/it/pn/frontend/e2e/pages/mittente/PiattaformaNotifichePage.java
@@ -1146,7 +1146,6 @@ public void verificaPresenzaStato(String stato) {
     }
 
     public void verificaNotificaCreata() {
-//        RestNotification restNotification = new RestNotification();
         String notificationRequestId = "";
         for (NetWorkInfo netWorkInfo : webDriverConfig.getNetWorkInfos()) {
             if (netWorkInfo.getRequestUrl().contains("bff/v1/notifications/sent") && netWorkInfo.getRequestMethod().equals("POST")) {

diff --git a/...pn/frontend/e2e/stepDefinitions/destinatario/personaFisica/RecapitiPersonaFisicaTest.java b/...pn/frontend/e2e/stepDefinitions/destinatario/personaFisica/RecapitiPersonaFisicaTest.java
@@ -189,7 +189,6 @@ public void nellaPaginaITuoiRecapitiSiControllaCheIlTastoAvvisamiViaEmailSiaBloc
     public void nellaPaginaITuoiRecapitiSiInserisceLEmailDelPFECliccaSulBottoneAvvisamiViaEmail(String dpFile) {
         logger.info("Si inserisce la email");
 
-//        DataPopulation dataPopulation = new DataPopulation();
         Map<String, Object> personaFisica = dataPopulation.readDataPopulation(dpFile + ".yaml");
         String email = personaFisica.get("mail").toString();
 
@@ -217,7 +216,6 @@ public void nellaPaginaITuoiRecapitiSiInserisceIlNumeroDiTelefonoDelPF(String dp
 
         logger.info("Si inserisce il numero di telefono PF");
 
-//        DataPopulation dataPopulation = new DataPopulation();
         Map<String, Object> personaFisica = dataPopulation.readDataPopulation(dpFile + ".yaml");
         String phoneNumber = personaFisica.get("telefono").toString();
 

diff --git a/...a/it/pn/frontend/e2e/stepDefinitions/destinatario/personaGiuridica/LoginPGPagoPATest.java b/...a/it/pn/frontend/e2e/stepDefinitions/destinatario/personaGiuridica/LoginPGPagoPATest.java
@@ -361,7 +361,6 @@ public void loginPortalePersonaGiuridicaTramiteTokenExchange(String dpFile) {
     }
 
     public String getTokenExchangePGFromFile(String personaGiuridica) {
-        //DataPopulation dataPopulation = new DataPopulation();
         String environment = webDriverConfig.getEnvironment();
         String token = "";
         switch (environment) {

diff --git a/.../pn/frontend/e2e/stepDefinitions/destinatario/personaGiuridica/NotifichePGPagoPATest.java b/.../pn/frontend/e2e/stepDefinitions/destinatario/personaGiuridica/NotifichePGPagoPATest.java
@@ -91,7 +91,6 @@ public class NotifichePGPagoPATest {
     public void clickSendNotificheDigitali() {
         this.logger.info("Si clicca su Send Notifiche Digitali");
 
-//        HomePagePG homePagePG = new HomePagePG(this.driver);
         homePagePG.waitLoadHomePagePGPage();
         String variabileAmbiente = webDriverConfig.getEnvironment();
         switch (variabileAmbiente) {
@@ -106,7 +105,6 @@ public void clickSendNotificheDigitali() {
     @And("Si visualizza correttamente la Pagina Notifiche persona giuridica {string}")
     public void siVisualizzaCorrettamenteLaPaginaNotifichePersonaGiuridica(String ragioneSociale) {
         if (!cookieConfig.isCookieEnabled()) {
-//            CookiesSection cookiesSection = new CookiesSection(this.driver);
             if (cookiesSection.waitLoadCookiesPage()) {
                 logger.info("Si accettano i cookies");
                 cookiesSection.selezionaAccettaTuttiButton();
@@ -195,10 +193,7 @@ public void nellaPaginaPiattaformaNotifichePersonaGiuridicaSiCliccaSulBottoneITu
 
     @Then("Si selezionano i file attestazioni opponibili da scaricare, all'interno della notifica persona giuridica, e si controlla che il download sia avvenuto {string}")
     public void siSelezionanoIFileAttestazioniOpponibiliDaScaricareAllInternoDellaNotificaPersonaGiuridicaESiControllaCheIlDownloadSiaAvvenuto(String dpFile) {
-//        DettaglioNotificaSection dettaglioNotificaSection = new DettaglioNotificaSection(this.driver);
         int numeroLinkAttestazioniOpponibile = dettaglioNotificaSection.getLinkAttestazioniOpponibili();
-//        DownloadFile downloadFile = new DownloadFile(this.driver);
-//        DataPopulation dataPopulation = new DataPopulation();
         Map<String, Object> datiNotifica = dataPopulation.readDataPopulation(dpFile + ".yaml");
         String workingDirectory = System.getProperty("user.dir");
         File pathCartella = new File(workingDirectory + "/src/test/resources/dataPopulation/downloadFileNotifica/destinatario/personaGiuridica");
@@ -239,11 +234,9 @@ private String getBearerToken() {
 
     public void siRecuperaBearerToken(String dpFile) {
         logger.info("Si recupera il bearer token");
-//        DataPopulation dataPopulation = new DataPopulation();
         personaGiuridica = dataPopulation.readDataPopulation(dpFile + ".yaml");
 
         if (!cookieConfig.isCookieEnabled()) {
-//            CookiesSection cookiesSection = new CookiesSection(this.driver);
             if (cookiesSection.waitLoadCookiesPage()) {
                 logger.info("Si accettano i cookies");
                 cookiesSection.selezionaAccettaTuttiButton();
@@ -280,14 +273,12 @@ public void siVerificaLaPresenzaDelloStatoNellaTimelineDellaNotifica(String stat
 
     @And("Si clicca sul radio bottone di pagamento")
     public void siCliccaRadioButtonPagamento() {
-//        PiattaformaNotifichePGPAPage piattaformaNotifichePGPAPage = new PiattaformaNotifichePGPAPage(this.driver);
         piattaformaNotifichePGPAPage.clickRadioBoxButton();
     }
 
     @And("Si controlla sia presente il modello F24 destinatario")
     public void siControllaSiaPresenteIlModelloF24Destinatario() {
         logger.info("Si controlla sia presente il modello F24 PG");
-//        PiattaformaNotifichePGPAPage piattaformaNotifichePGPAPage = new PiattaformaNotifichePGPAPage(this.driver);
         if (piattaformaNotifichePGPAPage.modelloF24Displayed()) {
             logger.info("Il modello F24 è trovato");
         } else {
@@ -299,7 +290,6 @@ public void siControllaSiaPresenteIlModelloF24Destinatario() {
     @And("Si controlla non sia presente il modello F24 destinatario")
     public void siControllaNonSiaPresenteIlModelloF24Destinatario() {
         logger.info("Si controlla non sia presente il modello F24 PG");
-//        PiattaformaNotifichePGPAPage piattaformaNotifichePGPAPage = new PiattaformaNotifichePGPAPage(this.driver);
         if (!piattaformaNotifichePGPAPage.modelloF24Displayed()) {
             logger.info("Il modello F24 non è trovato");
         } else {
@@ -311,7 +301,6 @@ public void siControllaNonSiaPresenteIlModelloF24Destinatario() {
     @And("Si clicca sul modello F24 destinatario numero {int}")
     public void siCliccaIlModelloF24Destinatario(int numOfF24) {
         logger.info("Si clicca  modello F24 destinatario");
-//        PiattaformaNotifichePGPAPage piattaformaNotifichePGPAPage = new PiattaformaNotifichePGPAPage(this.driver);
         piattaformaNotifichePGPAPage.vaiInFondoAllaPagina();
         piattaformaNotifichePGPAPage.clickModelloF24Numero(numOfF24);
     }
@@ -320,7 +309,6 @@ public void siCliccaIlModelloF24Destinatario(int numOfF24) {
     @And("Si controlla sia presente l'avviso PagoPa destinatario")
     public void siControllaSiaPresenteLAvvisoPagoPaDestinatario() {
         logger.info("Si controlla la presenza del box per il pagamento della notifica");
-//        AccediAPiattaformaNotifichePage accediAPiattaformaNotifichePage = new AccediAPiattaformaNotifichePage(this.driver);
         if (accediAPiattaformaNotifichePage.scaricaAvvisoDisplayed()) {
             logger.info("Avviso PagoPA è trovato");
         } else {
@@ -331,7 +319,6 @@ public void siControllaSiaPresenteLAvvisoPagoPaDestinatario() {
 
     @And("Si controlla sia presente piu avvisi PagoPa PG")
     public void siControllaSiaPresentePiuAvvisiPagoPaPG() {
-//        AccediAPiattaformaNotifichePage accediAPiattaformaNotifichePage = new AccediAPiattaformaNotifichePage(this.driver);
         Assertions.assertTrue(accediAPiattaformaNotifichePage.piuAvvisiDisplayed(),"Avvissi PagoPA non sono trovati");
         logger.info("Avvissi PagoPA sono trovati");
 
@@ -340,7 +327,6 @@ public void siControllaSiaPresentePiuAvvisiPagoPaPG() {
     @And("Si controlla sia presente l'avvisi PagoPa PG")
     public void siControllaSiaPresenteLAvvisiPagoPaPG() {
         logger.info("Si controlla la presenza del box per il pagamento della notifica");
-//        AccediAPiattaformaNotifichePage accediAPiattaformaNotifichePage = new AccediAPiattaformaNotifichePage(this.driver);
         Assertions.assertTrue(accediAPiattaformaNotifichePage.piuAvvisiDisplayed(),"Avviso PagoPA non è trovato");
         logger.info("Avviso PagoPA è trovato");
     }
@@ -366,29 +352,25 @@ public void siControllaNonSiaPresenteLAllegatoPagoPa() {
     @And("Si clicca l'avviso PagoPa destinatario")
     public void siCliccaLAvvisoPagoPaDestinatario() {
         logger.info("Si clicca l'avviso PagoPa destinatario");
-//        AccediAPiattaformaNotifichePage accediAPiattaformaNotifichePage = new AccediAPiattaformaNotifichePage(this.driver);
         accediAPiattaformaNotifichePage.clickAvvisoPagoPADestinatario();
     }
 
     @And("Si controlla sia visualizza box allegati modelli F24 PG")
     public void siControllaSiaVisualizzaBoxF24PG() {
         logger.info("Si controlla sia presente il box allegati modelli F24");
-//        PiattaformaNotifichePGPAPage piattaformaNotifichePGPAPage = new PiattaformaNotifichePGPAPage(this.driver);
         piattaformaNotifichePGPAPage.checkBoxModelloF24PG();
     }
 
     @And("Nella pagina piattaforma notifiche PG si effettua la ricerca per codice IUN {string}")
     public void nellaPaginaPiattformaNotificheSiEffettuaLaRicercaPerCodiceIUN(String codiceIUN) {
         logger.info("Si cerca una notifica tramite IUN: " + codiceIUN);
-//        PiattaformaNotifichePage piattaformaNotifichePage = new PiattaformaNotifichePage(driver);
         piattaformaNotifichePage.inserimentoCodiceIUN(codiceIUN);
         piattaformaNotifichePage.selectFiltraNotificaButtonDestinatario();
     }
 
     @And("Si controlla la presenza di codice avviso")
     public void siControllaLaPresenzaDiCodiceAvviso() {
         logger.info("Si controlla la presenza di codice avviso");
-//        AccediAPiattaformaNotifichePage accediAPiattaformaNotifichePage = new AccediAPiattaformaNotifichePage(this.driver);
         if (accediAPiattaformaNotifichePage.codiceAvvisoDisplayed()) {
             logger.info("Codice avviso è visuallizato corrttamente");
         } else {
@@ -399,19 +381,16 @@ public void siControllaLaPresenzaDiCodiceAvviso() {
 
     @And("controllo link per scaricare zip e scarico file Ricevuta di consegna")
     public void scaricaRicevutaDiConsegna() throws AWTException, IOException {
-//        PiattaformaNotifichePGPAPage piattaformaNotifichePGPAPage = new PiattaformaNotifichePGPAPage(this.driver);
         piattaformaNotifichePGPAPage.clickRicevutaDiConsegna();
     }
 
     @And("controllo Ricevuta di consegna link cliccabile")
     public void controlloRicevutaDiConsegnaLinkCliccabile() {
-//        NotificheDestinatarioPage notificheDestinatarioPage = new NotificheDestinatarioPage(this.driver);
         notificheDestinatarioPage.checkRicevutaConsegnaCliccabile();
     }
 
     @And("Controllo sia presente documento pdf")
     public void controlloPresenteDocumento() throws IOException {
-//        PiattaformaNotifichePGPAPage piattaformaNotifichePGPAPage = new PiattaformaNotifichePGPAPage(this.driver);
         logger.info("Controllo sia presente documento pdf");
         Assertions.assertTrue(piattaformaNotifichePGPAPage.checkIfPdfExists(), "Documento non è trovato");
         logger.info("Documento è trovato");
@@ -420,7 +399,6 @@ public void controlloPresenteDocumento() throws IOException {
     @And("estraggo il file zip")
     public void inseriscoPasswordEdEstraggoZip() throws IOException {
         logger.info("estraggo il file zip");
-//        PiattaformaNotifichePGPAPage piattaformaNotifichePGPAPage = new PiattaformaNotifichePGPAPage(this.driver);
         piattaformaNotifichePGPAPage.extractZipWithoutPassword();
     }
 
@@ -429,7 +407,6 @@ public void siSelezionanoIlFileDaScaricare(String nomeFile) throws IOException {
         logger.info("Si cerca di scaricare il file " + nomeFile);
 
         boolean headless = webDriverConfig.getHeadless().equalsIgnoreCase("true");
-//        DettaglioNotificaMittenteSection dettaglioNotificaMittenteSection = new DettaglioNotificaMittenteSection(this.driver);
         dettaglioNotificaMittenteSection.clickLinkAttestazioneOpponibile(nomeFile);
         webTool.waitTime(5);
 //        DownloadFile downloadFile = new DownloadFile(this.driver);
@@ -457,7 +434,6 @@ public void siControllaIlTestoAlSuoInternoDestonatario(String nomeFile) {
     @And("Si controlla il SHA all interno del file atteztazione")
     public void siControllaIlShaAllInternoAot() {
         logger.info("Si controlla che il testo al suo interno si corretto");
-//        DettaglioNotificaMittenteSection dettaglioNotificaMittenteSection = new DettaglioNotificaMittenteSection(this.driver);
         if (dettaglioNotificaMittenteSection.controlloSHAFile("Attestazione_opponibile_a_terzi_notifica_presa_in_carico")) {
             logger.info("Il codice SHA all'interno del file è corretto");
         } else {

diff --git a/src/test/java/it/pn/frontend/e2e/stepDefinitions/mittente/LoginMittentePagoPA.java b/src/test/java/it/pn/frontend/e2e/stepDefinitions/mittente/LoginMittentePagoPA.java
@@ -316,7 +316,6 @@ private void readurlPortaleMittente(String user, String password) {
             Assertions.fail("relayStateFromSpidLoginMittente is null");
         }
 
-//        BasicCookieStore cookieStore = new BasicCookieStore();
         BasicClientCookie cookie = new BasicClientCookie(cookiesNameFromSpidLoginMittente, cookiesValueFromSpidLoginMittente);
         cookie.setDomain(cookiesDomainFromSpidLoginMittente);
         cookie.setPath(cookiesPathFromSpidLoginMittente);

diff --git a/src/test/java/it/pn/frontend/e2e/stepDefinitions/mittente/NotificaMittentePagoPATest.java b/src/test/java/it/pn/frontend/e2e/stepDefinitions/mittente/NotificaMittentePagoPATest.java
@@ -242,7 +242,6 @@ private void aggiornamentoNumeroProtocollo() {
     @And("Cliccare su continua")
     public void cliccareSuContinua() {
         logger.info("Cliccare sul bottone continua");
-//        InvioNotifichePAPage invioNotifichePAPage = new InvioNotifichePAPage(hooks.getDriver());
         invioNotifichePAPage.selezionareContinuaButton();
     }