From 46e3d75c76413f807d41bd24ef712eee23b77ce8 Mon Sep 17 00:00:00 2001
From: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Date: Tue, 25 Aug 2020 19:29:39 +0200
Subject: [PATCH] fix(client): select only required props (Login) (#39461)

---
 client/src/components/Header/components/Login.js | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/client/src/components/Header/components/Login.js b/client/src/components/Header/components/Login.js
index f8c990ba71..b2a00d671e 100644
--- a/client/src/components/Header/components/Login.js
+++ b/client/src/components/Header/components/Login.js
@@ -19,15 +19,20 @@ const mapStateToProps = createSelector(
 );
 
 function Login(props) {
-  const { children, isSignedIn, ...restProps } = props;
+  const {
+    block,
+    'data-test-label': dataTestLabel,
+    children,
+    isSignedIn
+  } = props;
   const href = isSignedIn ? '/learn' : `${apiLocation}/signin`;
   return (
     <Button
       bsStyle='default'
-      className={(restProps.block ? 'btn-cta-big' : '') + ' signup-btn btn-cta'}
+      className={(block ? 'btn-cta-big' : '') + ' signup-btn btn-cta'}
+      data-test-label={dataTestLabel}
       href={href}
       onClick={() => gtagReportConversion()}
-      {...restProps}
     >
       {children || 'Sign In'}
     </Button>
@@ -36,7 +41,9 @@ function Login(props) {
 
 Login.displayName = 'Login';
 Login.propTypes = {
+  block: PropTypes.bool,
   children: PropTypes.any,
+  'data-test-label': PropTypes.string,
   isSignedIn: PropTypes.bool
 };