From ca325eee4e59adc80ba16036208ce62bb4b3480a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1s=20Castillo?= <tcastilloboireau@gmail.com>
Date: Mon, 10 Aug 2026 16:18:09 -0300
Subject: [PATCH 1/4] fix: add default summits array value on import users
 popup at sponsor users tab
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
---
 .../components/import-users-popup.js                             | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/import-users-popup.js b/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/import-users-popup.js
index 496be723e..64888922b 100644
--- a/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/import-users-popup.js
+++ b/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/import-users-popup.js
@@ -100,6 +100,7 @@ const ImportUsersPopup = ({
         <SummitsDropdown
           onChange={setSelectedSummit}
           excludeSummitIds={[currentSummit.id]}
+          summits={[]}
         />
         {selectedSummit && userOptions && (
           <>

From d7f191c0e17c38538e2a5e4c7f0ea115ef6240cf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1s=20Castillo?= <tcastilloboireau@gmail.com>
Date: Mon, 10 Aug 2026 16:27:06 -0300
Subject: [PATCH 2/4] fix: add unit test with regression test to check render
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
---
 .../__tests__/import-users-popup.test.js      | 57 +++++++++++++++++++
 1 file changed, 57 insertions(+)
 create mode 100644 src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js

diff --git a/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js b/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js
new file mode 100644
index 000000000..089272a6e
--- /dev/null
+++ b/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js
@@ -0,0 +1,57 @@
+import React from "react";
+import { screen } from "@testing-library/react";
+import { renderWithRedux } from "../../../../../../../utils/test-utils";
+import ImportUsersPopup from "../import-users-popup";
+
+jest.mock("i18n-react/dist/i18n-react", () => ({
+  __esModule: true,
+  default: { translate: (key) => key }
+}));
+
+jest.mock(
+  "openstack-uicore-foundation/lib/components/mui/summits-dropdown",
+  () => ({
+    __esModule: true,
+    default: ({ onChange, excludeSummitIds }) => (
+      <button
+        data-testid="summit-select"
+        type="button"
+        onClick={() => onChange(2)}
+      >
+        {`Select Summit (excluding: ${excludeSummitIds})`}
+      </button>
+    )
+  })
+);
+
+const mockCurrentSummit = { id: 1, name: "Current Summit" };
+
+const baseState = {
+  currentSummitState: { currentSummit: mockCurrentSummit },
+  sponsorUsersListState: { userGroups: [] }
+};
+
+const renderPopup = (props = {}) =>
+  renderWithRedux(
+    <ImportUsersPopup
+      sponsorId={42}
+      companyId={99}
+      onClose={jest.fn()}
+      {...props}
+    />,
+    { initialState: baseState }
+  );
+
+describe("ImportUsersPopup (per-sponsor)", () => {
+  it("mounts and shows the summit selector without throwing", () => {
+    renderPopup();
+
+    expect(
+      screen.getByText("sponsor_users.import_users.title")
+    ).toBeInTheDocument();
+    expect(screen.getByTestId("summit-select")).toBeInTheDocument();
+    expect(
+      screen.getByText(`Select Summit (excluding: ${mockCurrentSummit.id})`)
+    ).toBeInTheDocument();
+  });
+});

From cafefcb21731edb42610cabe63f3ac9d581a29c9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1s=20Castillo?= <tcastilloboireau@gmail.com>
Date: Mon, 10 Aug 2026 16:47:46 -0300
Subject: [PATCH 3/4] fix: adjust regression test to check the summits initial
 value
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
---
 .../__tests__/import-users-popup.test.js           | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js b/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js
index 089272a6e..08a92d0d5 100644
--- a/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js
+++ b/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js
@@ -12,9 +12,10 @@ jest.mock(
   "openstack-uicore-foundation/lib/components/mui/summits-dropdown",
   () => ({
     __esModule: true,
-    default: ({ onChange, excludeSummitIds }) => (
+    default: ({ onChange, excludeSummitIds, summits }) => (
       <button
         data-testid="summit-select"
+        data-summits={JSON.stringify(summits)}
         type="button"
         onClick={() => onChange(2)}
       >
@@ -50,8 +51,13 @@ describe("ImportUsersPopup (per-sponsor)", () => {
       screen.getByText("sponsor_users.import_users.title")
     ).toBeInTheDocument();
     expect(screen.getByTestId("summit-select")).toBeInTheDocument();
-    expect(
-      screen.getByText(`Select Summit (excluding: ${mockCurrentSummit.id})`)
-    ).toBeInTheDocument();
+    // Regression guard: SummitsDropdown has no default for `summits` and reads
+    // `.length` on it during render, so a missing prop crashes. Asserting the
+    // exact value passed down catches that regression even though the mock
+    // above doesn't reproduce the real component's crash itself.
+    expect(screen.getByTestId("summit-select")).toHaveAttribute(
+      "data-summits",
+      "[]"
+    );
   });
 });

From 8eea6c1b0dad9876cc98671141ab1f30581ca582 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1s=20Castillo?= <tcastilloboireau@gmail.com>
Date: Tue, 11 Aug 2026 18:50:42 -0300
Subject: [PATCH 4/4] fix: update uicore version, remove [] from summits param
 on SummitsDropdown, fix test
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
---
 .../components/__tests__/import-users-popup.test.js   | 11 +----------
 .../components/import-users-popup.js                  |  1 -
 .../components/sponsor-global-import-users-popup.js   |  1 -
 yarn.lock                                             |  2 +-
 4 files changed, 2 insertions(+), 13 deletions(-)

diff --git a/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js b/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js
index 08a92d0d5..fdc05a5f3 100644
--- a/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js
+++ b/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/__tests__/import-users-popup.test.js
@@ -12,10 +12,9 @@ jest.mock(
   "openstack-uicore-foundation/lib/components/mui/summits-dropdown",
   () => ({
     __esModule: true,
-    default: ({ onChange, excludeSummitIds, summits }) => (
+    default: ({ onChange, excludeSummitIds }) => (
       <button
         data-testid="summit-select"
-        data-summits={JSON.stringify(summits)}
         type="button"
         onClick={() => onChange(2)}
       >
@@ -51,13 +50,5 @@ describe("ImportUsersPopup (per-sponsor)", () => {
       screen.getByText("sponsor_users.import_users.title")
     ).toBeInTheDocument();
     expect(screen.getByTestId("summit-select")).toBeInTheDocument();
-    // Regression guard: SummitsDropdown has no default for `summits` and reads
-    // `.length` on it during render, so a missing prop crashes. Asserting the
-    // exact value passed down catches that regression even though the mock
-    // above doesn't reproduce the real component's crash itself.
-    expect(screen.getByTestId("summit-select")).toHaveAttribute(
-      "data-summits",
-      "[]"
-    );
   });
 });
diff --git a/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/import-users-popup.js b/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/import-users-popup.js
index 64888922b..496be723e 100644
--- a/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/import-users-popup.js
+++ b/src/pages/sponsors/sponsor-page/tabs/sponsor-users-list-per-sponsor/components/import-users-popup.js
@@ -100,7 +100,6 @@ const ImportUsersPopup = ({
         <SummitsDropdown
           onChange={setSelectedSummit}
           excludeSummitIds={[currentSummit.id]}
-          summits={[]}
         />
         {selectedSummit && userOptions && (
           <>
diff --git a/src/pages/sponsors/sponsor-users-list-page/components/sponsor-global-import-users-popup.js b/src/pages/sponsors/sponsor-users-list-page/components/sponsor-global-import-users-popup.js
index 2a4886017..b67c7b843 100644
--- a/src/pages/sponsors/sponsor-users-list-page/components/sponsor-global-import-users-popup.js
+++ b/src/pages/sponsors/sponsor-users-list-page/components/sponsor-global-import-users-popup.js
@@ -190,7 +190,6 @@ const SponsorGlobalImportUsersPopup = ({
               setUserOptions(null);
               setSelectedUsers([]);
             }}
-            summits={[]}
             excludeSummitIds={[summitId]}
           />
           {selectedSummit && (
diff --git a/yarn.lock b/yarn.lock
index 57c547313..186748ebd 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -9044,7 +9044,7 @@ open@^10.0.3:
 
 openstack-uicore-foundation@5.0.47:
   version "5.0.47"
-  resolved "https://registry.npmjs.org/openstack-uicore-foundation/-/openstack-uicore-foundation-5.0.47.tgz#32500fb4d8392cbf5713e968afa875cea42a67a2"
+  resolved "https://registry.yarnpkg.com/openstack-uicore-foundation/-/openstack-uicore-foundation-5.0.47.tgz#32500fb4d8392cbf5713e968afa875cea42a67a2"
   integrity sha512-E/6uO7Q5MtTfhWExshuNIrwVcxDewDvA76YmGzQz2f5LeZ8Gsv22eqBNbylDakwxuPmyD16r/AQEURr5soALAw==
   dependencies:
     use-sync-external-store "^1.6.0"