Merge pull request #12 from NHSDigital/lp-fix-disable-default-behaviour-error

sp-fix Removed disableDefaultBehaviour from the list of rendered attr…

Author: Tomdango

src/components/skip-link/SkipLink.tsx

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

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

@@ -0,0 +1,20 @@
+import React, { createRef } from 'react';
+import { shallow } from 'enzyme';
+import SkipLink from '..';
+
+describe('SkipLink', () => {
+  it('matches snapshot', () => {
+    const component = shallow(<SkipLink />);
+    expect(component).toMatchSnapshot('SkipLink');
+    component.unmount();
+  });
+
+  it('does not add focusTargetRef or disableDefaultBehaviour as attributes when rendered', () => {
+    const ref = createRef<HTMLElement>();
+    const component = shallow(<SkipLink focusTargetRef={ref} disableDefaultBehaviour aria-label="test" />);
+    expect(component.prop('focusTargetRef')).toBeUndefined();
+    expect(component.prop('disableDefaultBehaviour')).toBeUndefined();
+    expect(component.prop('aria-label')).toEqual('test');
+    component.unmount();
+  });
+});

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

@@ -0,0 +1,11 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`SkipLink matches snapshot: SkipLink 1`] = `
+<a
+  className="nhsuk-skip-link"
+  href="#maincontent"
+  onClick={[Function]}
+>
+  Skip to main content
+</a>
+`;