fix(Alert): Properly spread props when transition={false} (#5545)

* Invert ternary

* Add test(s) for prop resting

Co-authored-by: iLan Epstein <iepstein@tableau.com>

src/Alert.tsx

@@ -118,7 +118,7 @@ const Alert = (React.forwardRef<HTMLDivElement, AlertProps>(
     const alert = (
       <div
         role="alert"
-        {...(Transition ? props : undefined)}
+        {...(!Transition ? props : undefined)}
         ref={ref}
         className={classNames(
           className,

test/AlertSpec.js

@@ -48,6 +48,26 @@ describe('<Alert>', () => {
     expect(wrapper.find('.fade').length).to.equal(0);
   });
 
+  it('should spread props to alert when transition=false', () => {
+    const alertId = 'alert-id';
+    const wrapper = mount(
+      <Alert transition={false} id={alertId}>
+        Message
+      </Alert>,
+    );
+    expect(wrapper.getDOMNode().getAttribute('id')).to.equal(alertId);
+  });
+
+  it('should spread props to alert when transition=true', () => {
+    const alertId = 'alert-id';
+    const wrapper = mount(
+      <Alert transition id={alertId}>
+        Message
+      </Alert>,
+    );
+    expect(wrapper.getDOMNode().getAttribute('id')).to.equal(alertId);
+  });
+
   it('should use Fade when transition=true', () => {
     mount(
       <Alert variant="danger" transition>