fixes broken link and adds test

This commit is contained in:
Alex Corey
2021-01-07 13:07:19 -05:00
parent e1fb6542c3
commit 1cd7f42a27
2 changed files with 9 additions and 2 deletions

View File

@@ -34,7 +34,6 @@ function PageHeaderToolbar({
const handleUserSelect = () => { const handleUserSelect = () => {
setIsUserOpen(!isUserOpen); setIsUserOpen(!isUserOpen);
}; };
return ( return (
<PageHeaderTools> <PageHeaderTools>
<PageHeaderToolsGroup> <PageHeaderToolsGroup>
@@ -90,8 +89,11 @@ function PageHeaderToolbar({
dropdownItems={[ dropdownItems={[
<DropdownItem <DropdownItem
key="user" key="user"
aria-label={i18n._(t`User details`)}
href={ href={
loggedInUser ? `/users/${loggedInUser.id}/details` : '/home' loggedInUser
? `/#/users/${loggedInUser.id}/details`
: '/#/home'
} }
> >
{i18n._(t`User Details`)} {i18n._(t`User Details`)}

View File

@@ -25,6 +25,7 @@ describe('PageHeaderToolbar', () => {
<PageHeaderToolbar <PageHeaderToolbar
onAboutClick={onAboutClick} onAboutClick={onAboutClick}
onLogoutClick={onLogoutClick} onLogoutClick={onLogoutClick}
loggedInUser={{ id: 1 }}
/> />
); );
expect(wrapper.find('DropdownItem')).toHaveLength(0); expect(wrapper.find('DropdownItem')).toHaveLength(0);
@@ -37,6 +38,10 @@ describe('PageHeaderToolbar', () => {
expect(wrapper.find('DropdownItem')).toHaveLength(0); expect(wrapper.find('DropdownItem')).toHaveLength(0);
wrapper.find(pageUserDropdownSelector).simulate('click'); wrapper.find(pageUserDropdownSelector).simulate('click');
wrapper.update();
expect(
wrapper.find('DropdownItem[aria-label="User details"]').prop('href')
).toBe('/#/users/1/details');
expect(wrapper.find('DropdownItem')).toHaveLength(2); expect(wrapper.find('DropdownItem')).toHaveLength(2);
const logout = wrapper.find('DropdownItem li button'); const logout = wrapper.find('DropdownItem li button');