From 7d1372f36c058c2945636d679434fbecfecbd644 Mon Sep 17 00:00:00 2001
From: Alexandros Georgantas <alexgeorg86@gmail.com>
Date: Fri, 12 Jan 2018 12:41:36 +0200
Subject: [PATCH] Clean ups  and new snapshots

---
 packages/ui/test/Menu.test.js                          | 10 +---------
 packages/ui/test/Radio.test.js                         |  2 +-
 packages/ui/test/RadioGroup.test.js                    |  4 +---
 packages/ui/test/YesOrNo.test.js                       |  2 +-
 packages/ui/test/__snapshots__/Menu.test.js.snap       |  2 +-
 packages/ui/test/__snapshots__/Radio.test.js.snap      |  2 +-
 packages/ui/test/__snapshots__/RadioGroup.test.js.snap |  2 +-
 packages/ui/test/__snapshots__/YesOrNo.test.js.snap    |  2 +-
 8 files changed, 8 insertions(+), 18 deletions(-)

diff --git a/packages/ui/test/Menu.test.js b/packages/ui/test/Menu.test.js
index b71f3a4d4..a6fd08ff9 100644
--- a/packages/ui/test/Menu.test.js
+++ b/packages/ui/test/Menu.test.js
@@ -1,5 +1,4 @@
 import React from 'react'
-import { shallow } from 'enzyme'
 import renderer from 'react-test-renderer'
 
 import Menu from '../src/atoms/Menu'
@@ -9,16 +8,9 @@ const props = {
   value: 'foo',
 }
 
-const wrapper = shallow(<Menu {...props} />)
-
 describe('Menu', () => {
-  test('Snapshot', () => {
+  test('is rendered correctly', () => {
     const tree = renderer.create(<Menu {...props} />).toJSON()
     expect(tree).toMatchSnapshot()
   })
-
-  test('Renders a Menu', () => {
-    expect(wrapper.is('div')).toBeTruthy()
-    expect(wrapper).toHaveLength(1)
-  })
 })
diff --git a/packages/ui/test/Radio.test.js b/packages/ui/test/Radio.test.js
index 35d330995..3b050b5cc 100644
--- a/packages/ui/test/Radio.test.js
+++ b/packages/ui/test/Radio.test.js
@@ -15,7 +15,7 @@ const props = {
 const wrapper = shallow(<Radio {...props} />)
 
 describe('Radio', () => {
-  test('Snapshot', () => {
+  test('is rendered correctly', () => {
     const tree = renderer.create(<Radio {...props} />).toJSON()
     expect(tree).toMatchSnapshot()
   })
diff --git a/packages/ui/test/RadioGroup.test.js b/packages/ui/test/RadioGroup.test.js
index b6ff69ef3..61d752d72 100644
--- a/packages/ui/test/RadioGroup.test.js
+++ b/packages/ui/test/RadioGroup.test.js
@@ -30,14 +30,12 @@ const wrapper = shallow(<RadioGroup {...props} />)
 const radios = wrapper.find(Radio)
 
 describe('Radio Group', () => {
-  test('Snapshot', () => {
+  test('is rendered correctly', () => {
     const tree = renderer.create(<RadioGroup {...props} />).toJSON()
     expect(tree).toMatchSnapshot()
   })
 
   test('Renders a number of radio buttons', () => {
-    expect(wrapper.is('div')).toBeTruthy()
-
     const len = props.options.length
     expect(wrapper.children()).toHaveLength(len)
 
diff --git a/packages/ui/test/YesOrNo.test.js b/packages/ui/test/YesOrNo.test.js
index b893b903c..625e74cd2 100644
--- a/packages/ui/test/YesOrNo.test.js
+++ b/packages/ui/test/YesOrNo.test.js
@@ -14,7 +14,7 @@ const wrapper = shallow(<YesOrNo {...props} />)
 const radio = wrapper.find(RadioGroup)
 
 describe('Yes or No', () => {
-  test('Snapshot', () => {
+  test('is rendered correclty', () => {
     const tree = renderer.create(<YesOrNo {...props} />).toJSON()
     expect(tree).toMatchSnapshot()
   })
diff --git a/packages/ui/test/__snapshots__/Menu.test.js.snap b/packages/ui/test/__snapshots__/Menu.test.js.snap
index b487f640b..6b493716b 100644
--- a/packages/ui/test/__snapshots__/Menu.test.js.snap
+++ b/packages/ui/test/__snapshots__/Menu.test.js.snap
@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`Menu Snapshot 1`] = `
+exports[`Menu is rendered correctly 1`] = `
 <div
   className="root"
 >
diff --git a/packages/ui/test/__snapshots__/Radio.test.js.snap b/packages/ui/test/__snapshots__/Radio.test.js.snap
index cac26b5df..5b4e9a77f 100644
--- a/packages/ui/test/__snapshots__/Radio.test.js.snap
+++ b/packages/ui/test/__snapshots__/Radio.test.js.snap
@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`Radio Snapshot 1`] = `
+exports[`Radio is rendered correctly 1`] = `
 <label
   className="root"
   style={
diff --git a/packages/ui/test/__snapshots__/RadioGroup.test.js.snap b/packages/ui/test/__snapshots__/RadioGroup.test.js.snap
index 9302b211f..646ee35d9 100644
--- a/packages/ui/test/__snapshots__/RadioGroup.test.js.snap
+++ b/packages/ui/test/__snapshots__/RadioGroup.test.js.snap
@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`Radio Group Snapshot 1`] = `
+exports[`Radio Group is rendered correctly 1`] = `
 <div>
   <label
     className="root"
diff --git a/packages/ui/test/__snapshots__/YesOrNo.test.js.snap b/packages/ui/test/__snapshots__/YesOrNo.test.js.snap
index 13fee7b65..c46a841be 100644
--- a/packages/ui/test/__snapshots__/YesOrNo.test.js.snap
+++ b/packages/ui/test/__snapshots__/YesOrNo.test.js.snap
@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`Yes or No Snapshot 1`] = `
+exports[`Yes or No is rendered correclty 1`] = `
 <div>
   <label
     className="root inline"
-- 
GitLab