[core] fix: get toasters timeout Infinity to match behavior of timeout 0

packages/core/src/components/toast/toast2.tsx

@@ -43,8 +43,7 @@ export const Toast2 = React.forwardRef<HTMLDivElement, ToastProps>((props, ref)
     const startTimeout = React.useCallback(() => setIsTimeoutStarted(true), []);
     const clearTimeout = React.useCallback(() => setIsTimeoutStarted(false), []);
 
-    // Per docs: "Providing a value less than or equal to 0 will disable the timeout (this is discouraged)."
-    const isTimeoutEnabled = timeout != null && timeout > 0;
+    const isTimeoutEnabled = timeout != null && timeout > 0 && timeout !== Infinity;
 
     // timeout is triggered & cancelled by updating `isTimeoutStarted` state
     useTimeout(

packages/core/src/components/toast/toastProps.ts

@@ -47,7 +47,7 @@ export interface ToastProps extends Props, IntentProps {
 
     /**
      * Milliseconds to wait before automatically dismissing toast.
-     * Providing a value less than or equal to 0 will disable the timeout (this is discouraged).
+     * Providing a value less than or equal to 0 or Infinity will disable the timeout (this is discouraged).
      *
      * @default 5000
      */

packages/core/test/toast/toast2Tests.tsx

@@ -98,5 +98,13 @@ describe("<Toast2>", () => {
                 done();
             }, 20);
         });
+
+        it("timeout={Infinity} behaves same as timeout={0} and does not trigger onDismiss", done => {
+            mount(<Toast2 message="Hello" onDismiss={handleDismiss} timeout={Infinity} />);
+            setTimeout(() => {
+                assert.isTrue(handleDismiss.notCalled, "onDismiss was called when it should not have been");
+                done();
+            }, 20);
+        });
     });
 });