test: add additional tests

bigcommerce · Feb 18, 2020 · 96b490f · 96b490f
1 parent e50aa32
commit 96b490f
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 3 deletions.
diff --git a/packages/big-design/src/components/Alert/Alert.tsx b/packages/big-design/src/components/Alert/Alert.tsx
@@ -9,9 +9,6 @@ import { GridItem } from '../Grid';
 
 import { StyledAlert, StyledHeader, StyledLink, StyledMessageItem } from './styled';
 
-// TODO:
-// - animation
-
 export interface AlertProps extends SharedMessagingProps {
   key?: string;
 }

diff --git a/packages/big-design/src/components/Alert/spec.tsx b/packages/big-design/src/components/Alert/spec.tsx
@@ -85,3 +85,12 @@ test('trigger onClose', () => {
 
   expect(fn).toHaveBeenCalled();
 });
+
+test('does not forward styles', () => {
+  const { container } = render(
+    <Alert messages={[{ text: 'Success' }]} className="test" style={{ background: 'red' }} />,
+  );
+
+  expect(container.getElementsByClassName('test').length).toBe(0);
+  expect(container.firstChild).not.toHaveStyle('background: red');
+});
diff --git a/packages/big-design/src/components/InlineAlert/spec.tsx b/packages/big-design/src/components/InlineAlert/spec.tsx
@@ -97,3 +97,12 @@ test('trigger onClose', () => {
 
   expect(fn).toHaveBeenCalled();
 });
+
+test('does not forward styles', () => {
+  const { container } = render(
+    <InlineAlert messages={[{ text: 'Success' }]} className="test" style={{ background: 'red' }} />,
+  );
+
+  expect(container.getElementsByClassName('test').length).toBe(0);
+  expect(container.firstChild).not.toHaveStyle('background: red');
+});
diff --git a/packages/big-design/src/components/Message/spec.tsx b/packages/big-design/src/components/Message/spec.tsx
@@ -95,3 +95,12 @@ test('trigger onClose', () => {
 
   expect(fn).toHaveBeenCalled();
 });
+
+test('does not forward styles', () => {
+  const { container } = render(
+    <Message messages={[{ text: 'Success' }]} className="test" style={{ background: 'red' }} />,
+  );
+
+  expect(container.getElementsByClassName('test').length).toBe(0);
+  expect(container.firstChild).not.toHaveStyle('background: red');
+});