Fix stale closure with onCopy

Author: LauraBeatris

packages/ui/src/components/ConfigureSSO/steps/TestConfigurationStep.tsx

@@ -732,7 +732,7 @@ const CopyTestUrlButton = ({ onTestRunCreated }: CopyTestUrlButtonProps): JSX.El
       .createEnterpriseConnectionTestRun(enterpriseConnection.id)
       .then(({ url }) => {
         setTestUrl(url);
-        onCopy();
+        onCopy(url);
         onTestRunCreated?.(url);
       })
       .catch(err => handleError(err as Error, [], card.setError))

packages/ui/src/hooks/useClipboard.ts

@@ -27,10 +27,13 @@ export function useClipboard(text: string, optionsOrTimeout: number | UseClipboa
   const { timeout = 1500, ...copyOptions } =
     typeof optionsOrTimeout === 'number' ? { timeout: optionsOrTimeout } : optionsOrTimeout;
 
-  const onCopy = useCallback(() => {
-    const didCopy = copy(text, copyOptions);
-    setHasCopied(didCopy);
-  }, [text, copyOptions]);
+  const onCopy = useCallback(
+    (overrideText?: string) => {
+      const didCopy = copy(overrideText ?? text, copyOptions);
+      setHasCopied(didCopy);
+    },
+    [text, copyOptions],
+  );
 
   useEffect(() => {
     let timeoutId: number | null = null;