Fixed navigation to mappers tab and added test (#3985)

apps/admin-ui/cypress/e2e/identity_providers_oidc_test.spec.ts

@@ -121,6 +121,15 @@ describe("OIDC identity provider test", () => {
       masthead.checkNotificationMessage(createMapperSuccessMsg, true);
     });
 
+    it("should cancel the addition of the OIDC mapper", () => {
+      sidebarPage.goToIdentityProviders();
+      listingPage.goToItemDetails(oidcProviderName);
+      addMapperPage.goToMappersTab();
+      addMapperPage.addMapper();
+      addMapperPage.cancelNewMapper();
+      addMapperPage.shouldGoToMappersTab();
+    });
+
     it("clean up providers", () => {
       const modalUtils = new ModalUtils();
 

apps/admin-ui/cypress/support/pages/admin-ui/manage/identity_providers/AddMapperPage.ts

@@ -19,6 +19,8 @@ export default class AddMapperPage {
   private userSessionAttribute = "attribute";
   private userSessionAttributeValue = "attribute.value";
   private newMapperSaveButton = "new-mapper-save-button";
+  private newMapperCancelButton = "new-mapper-cancel-button";
+  private mappersUrl = "/oidc/mappers";
   private regexAttributeValuesSwitch = "are.attribute.values.regex";
   private syncmodeSelectToggle = "#syncMode";
   private attributesKeyInput = '[data-testid="config.attributes[0].key"]';
@@ -51,6 +53,11 @@ export default class AddMapperPage {
     return this;
   }
 
+  cancelNewMapper() {
+    cy.findByTestId(this.newMapperCancelButton).click();
+    return this;
+  }
+
   toggleSwitch(switchName: string) {
     cy.findByTestId(switchName).click({ force: true });
 
@@ -396,4 +403,10 @@ export default class AddMapperPage {
 
     return this;
   }
+
+  shouldGoToMappersTab() {
+    cy.url().should("include", this.mappersUrl);
+
+    return this;
+  }
 }

apps/admin-ui/src/identity-providers/add/AddMapper.tsx

@@ -245,6 +245,7 @@ export default function AddMapper() {
             {t("common:save")}
           </Button>
           <Button
+            data-testid="new-mapper-cancel-button"
             variant="link"
             component={(props) => (
               <Link
@@ -253,7 +254,7 @@ export default function AddMapper() {
                   realm,
                   providerId,
                   alias: alias!,
-                  tab: "settings",
+                  tab: "mappers",
                 })}
               />
             )}