Просмотр исходного кода

Fix failing tests caused by CORWEB-141

Tests previously checked whether Providers in the Wizard which had no
Endpoints, had `Add New...` dropdown item.
The tests should check whether Providers with no Endpoints have `Add`
button.
Sergiu Miclea 8 лет назад
Родитель
Сommit
b3fdfcda67

+ 3 - 1
src/components/atoms/EndpointLogos/test.jsx

@@ -21,5 +21,7 @@ const wrap = props => shallow(<EndpointLogos {...props} />).dive()
 it('passes down props', () => {
 it('passes down props', () => {
   let wrapper = wrap({ height: 32, endpoint: 'aws' })
   let wrapper = wrap({ height: 32, endpoint: 'aws' })
   expect(wrapper.prop('height')).toBe(32)
   expect(wrapper.prop('height')).toBe(32)
-  expect(wrapper.children().prop('endpoint')).toBe('aws')
+  let imageInfo = wrapper.findWhere(w => w.name() === 'styled.div' && w.prop('imageInfo')).prop('imageInfo')
+  expect(imageInfo.h).toBe(32)
+  expect(imageInfo.image).toBe('file')
 })
 })

+ 1 - 3
src/components/organisms/WizardEndpointList/test.jsx

@@ -52,8 +52,7 @@ it('has providers with correct enpoints available', () => {
 
 
 it('renders add new', () => {
 it('renders add new', () => {
   let wrapper = wrap({ endpoints, providers })
   let wrapper = wrap({ endpoints, providers })
-  expect(wrapper.find('Dropdown').at(2).prop('items').length).toBe(1)
-  expect(wrapper.find('Dropdown').at(2).prop('items')[0].id).toBe('addNew')
+  expect(wrapper.findWhere(w => w.name() === 'Button' && w.html().indexOf('Add') > -1).length).toBe(4)
 })
 })
 
 
 it('renders loading', () => {
 it('renders loading', () => {
@@ -65,7 +64,6 @@ it('renders dropdown as primary if endpoint is selected', () => {
   let wrapper = wrap({ endpoints, providers, selectedEndpoint: { ...endpoints[1] } })
   let wrapper = wrap({ endpoints, providers, selectedEndpoint: { ...endpoints[1] } })
   expect(wrapper.find('Dropdown').at(1).prop('primary')).toBe(true)
   expect(wrapper.find('Dropdown').at(1).prop('primary')).toBe(true)
   expect(wrapper.find('Dropdown').at(0).prop('primary')).toBe(false)
   expect(wrapper.find('Dropdown').at(0).prop('primary')).toBe(false)
-  expect(wrapper.find('Dropdown').at(2).prop('primary')).toBe(false)
 })
 })
 
 
 it('doesn\'t render endpoint if another endpoint is supplied', () => {
 it('doesn\'t render endpoint if another endpoint is supplied', () => {