Skip to content
Snippets Groups Projects
Commit 472ca55b authored by Jure's avatar Jure
Browse files

Revert "Wrap Authorize children to avoid errors when there is more than a single element"

This reverts commit 3b578dbc
parent 9debb4e5
No related branches found
No related tags found
No related merge requests found
...@@ -41,7 +41,7 @@ export class Authorize extends React.Component { ...@@ -41,7 +41,7 @@ export class Authorize extends React.Component {
render() { render() {
return this.state.authorized return this.state.authorized
? <div>{this.props.children}</div> ? this.props.children
: this.props.unauthorized || null : this.props.unauthorized || null
} }
} }
......
...@@ -8,8 +8,7 @@ global.PUBSWEET_COMPONENTS = [] ...@@ -8,8 +8,7 @@ global.PUBSWEET_COMPONENTS = []
function makeWrapper(props = {}) { function makeWrapper(props = {}) {
return shallow( return shallow(
<Authorize currentUser={{ id: 'user1' }} {...props}> <Authorize currentUser={{ id: 'user1' }} {...props}>
<span className="secret" /> <div />
<span className="sibling" />
</Authorize>, </Authorize>,
) )
} }
...@@ -45,12 +44,11 @@ describe('<Authorize/>', () => { ...@@ -45,12 +44,11 @@ describe('<Authorize/>', () => {
}) })
it('optionally shows alternative content when not authorized', async () => { it('optionally shows alternative content when not authorized', async () => {
const wrapper = makeWrapper({ authsome, unauthorized: <span className="message" /> }) const wrapper = makeWrapper({ authsome, unauthorized: <span /> })
resolveMode(false) resolveMode(false)
await modePromise await modePromise
expect(wrapper.find('.secret')).toHaveLength(0) expect(wrapper.type()).toBe('span')
expect(wrapper.find('.message')).toHaveLength(1)
}) })
it('is empty when authsome throws', async () => { it('is empty when authsome throws', async () => {
...@@ -74,7 +72,7 @@ describe('<Authorize/>', () => { ...@@ -74,7 +72,7 @@ describe('<Authorize/>', () => {
resolveMode(true) resolveMode(true)
await modePromise await modePromise
expect(wrapper.find('.secret')).toHaveLength(1) expect(wrapper.type()).toBe('div')
}) })
it('rechecks auth when props change', () => { it('rechecks auth when props change', () => {
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment