If disableDefaultBehaviour is set, dont set the href on the skiplink

Author: Luke Pearson

src/components/skip-link/SkipLink.tsx

@@ -84,9 +84,14 @@ class SkipLink extends React.Component<SkipLinkProps> {
   };
 
   render() {
-    const { className, focusTargetRef, disableDefaultBehaviour, ...rest } = this.props;
+    const { className, focusTargetRef, disableDefaultBehaviour, href, ...rest } = this.props;
     return (
-      <a className={classNames('nhsuk-skip-link', className)} onClick={this.onClick} {...rest} />
+      <a
+        className={classNames('nhsuk-skip-link', className)}
+        onClick={this.onClick}
+        href={disableDefaultBehaviour ? undefined : href}
+        {...rest}
+      />
     );
   }
 }

src/components/skip-link/__tests__/SkipLink.test.tsx

@@ -1,5 +1,5 @@
 import React, { createRef } from 'react';
-import { shallow } from 'enzyme';
+import { shallow, mount } from 'enzyme';
 import SkipLink from '..';
 
 describe('SkipLink', () => {
@@ -17,4 +17,22 @@ describe('SkipLink', () => {
     expect(component.prop('aria-label')).toEqual('test');
     component.unmount();
   });
+
+  it('sets the href to #maincontent by default', () => {
+    const component = mount(<SkipLink />);
+    expect(component.find('.nhsuk-skip-link').prop('href')).toBe('#maincontent');
+  });
+
+  it('calls onClick callback when clicked', () => {
+    const onClick = jest.fn();
+    const component = mount(<SkipLink onClick={onClick} />);
+    component.find('.nhsuk-skip-link').simulate('click');
+    expect(onClick).toHaveBeenCalled();
+  });
+
+  it('does not set the href to #maincontent if disableDefaultBehaviour is set', () => {
+    const onClick = jest.fn();
+    const component = mount(<SkipLink disableDefaultBehaviour onClick={onClick} />);
+    expect(component.find('.nhsuk-skip-link').prop('href')).toBeUndefined();
+  });
 });