Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement profile widget in header #236

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Conversation

mkrause
Copy link
Collaborator

@mkrause mkrause commented Jun 2, 2017

Closes #221

@mkrause
Copy link
Collaborator Author

mkrause commented Jun 2, 2017

@ismay This is done, basically. But there's one test I can't get to work. See Travis.

  it('navigates to the login page when clicking the login button', () => {
    const push = jest.fn()
    const wrapper = shallow(<DumbHeader hasAuth={false} history={{ push }} />)
    wrapper.find(RaisedButton).simulate('click')
    expect(push).toBeCalledWith('/login')
  })

I'm trying to get test coverage for the onTouchTap event in the Header component. I've tried a few variants of the above, but none are able to simulate the event (despite the name, a click event should trigger onTouchTap). Any ideas?

@ghost
Copy link

ghost commented Jun 4, 2017

@mkrause Ah ok. I'll take a look at this next wednesday, will let you know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant