From 1185d9e9a8f9311a31f2e38ba12d5c5634372b9a Mon Sep 17 00:00:00 2001
From: John OHara <johara@redhat.com>
Date: Tue, 1 Oct 2024 19:02:06 +0100
Subject: [PATCH] repopulate datasources when team owner changes for test.
 Fixes: #2049

---
 horreum-web/src/domain/tests/TestSettings.tsx | 31 ++++++++++---------
 1 file changed, 16 insertions(+), 15 deletions(-)

diff --git a/horreum-web/src/domain/tests/TestSettings.tsx b/horreum-web/src/domain/tests/TestSettings.tsx
index 6f1cdfab5..ae1b13c2d 100644
--- a/horreum-web/src/domain/tests/TestSettings.tsx
+++ b/horreum-web/src/domain/tests/TestSettings.tsx
@@ -66,7 +66,7 @@ export default function TestSettings({ test, onTestIdChange, onModified, funcsRe
     useEffect( () => {
         apiCall(configApi.datastores(owner), alerting, "DATASTORE", "Error occurred fetching datastores")
             .then(ds => setDatastores(ds))
-    }, [test])
+    }, [test, owner])
 
     const updateState = (t?: Test) => {
         setName(t?.name || "")
@@ -148,6 +148,20 @@ export default function TestSettings({ test, onTestIdChange, onModified, funcsRe
                         </HelperText>
                     </FormHelperText>
                 </FormGroup>
+                <FormGroup label="Team" fieldId="testOwner">
+                    {isTester ? (
+                        <TeamSelect
+                            includeGeneral={false}
+                            selection={teamToName(owner) || ""}
+                            onSelect={selection => {
+                                setOwner(selection.key)
+                                onModified(true)
+                            }}
+                        />
+                    ) : (
+                        <TextInput value={teamToName(owner) || ""} id="testOwner"  readOnlyVariant="default" />
+                    )}
+                </FormGroup>
                 <FormGroup label="Datastore" fieldId="datastoreId">
                     <FormSelect
                         value={datastoreId?.toString()}
@@ -222,20 +236,7 @@ export default function TestSettings({ test, onTestIdChange, onModified, funcsRe
                 <Divider/>
 
                 <Title headingLevel="h2">Permissions</Title>
-                <FormGroup label="Owner" fieldId="testOwner">
-                    {isTester ? (
-                        <TeamSelect
-                            includeGeneral={false}
-                            selection={teamToName(owner) || ""}
-                            onSelect={selection => {
-                                setOwner(selection.key)
-                                onModified(true)
-                            }}
-                        />
-                    ) : (
-                        <TextInput value={teamToName(owner) || ""} id="testOwner"  readOnlyVariant="default" />
-                    )}
-                </FormGroup>
+
                 <FormGroup label="Access rights" fieldId="testAccess">
                     {isTester ? (
                         <AccessChoice