Recactoring tests in the <LoginComponent />

wipkanban · Nov 10, 2018 · 3235ace · 3235ace
1 parent 35a1209
commit 3235ace
Show file tree

Hide file tree

Showing 3 changed files with 50 additions and 3 deletions.
diff --git a/src/client/components/Login/FormLogin.test.js b/src/client/components/Login/FormLogin.test.js
@@ -70,7 +70,8 @@ describe("Login User", () => {
     let expectedState = {
       email: "email@example.com",
       password: "mypassword",
-      requiredFields: false
+      emailFieldEmpty: false,
+      passwordFieldEmpty: false
     };
 
     //password field

diff --git a/src/client/components/Login/__snapshots__/FormLogin.test.js.snap b/src/client/components/Login/__snapshots__/FormLogin.test.js.snap
@@ -123,8 +123,11 @@ exports[`Login User does not should call function onLogin when click button Logi
                 }
                 className="Login-text-4"
                 error={true}
+                helperText="The email field is empty"
+                inputRef={[Function]}
                 label="Type your email"
                 onChange={[Function]}
+                onKeyUp={[Function]}
                 required={false}
                 select={false}
                 variant="standard"
@@ -140,14 +143,22 @@ exports[`Login User does not should call function onLogin when click button Logi
                   }
                 }
                 className="Login-text-4"
-                error={true}
+                error={false}
+                helperText=""
                 label="Password"
                 onChange={[Function]}
+                onKeyUp={[Function]}
                 required={false}
                 select={false}
                 type="password"
                 variant="standard"
               />
+              <WithStyles(Typography)
+                align="center"
+                className="Login-textError-10"
+                gutterBottom={true}
+                variant="subtitle2"
+              />
               <div
                 style={
                   Object {
@@ -357,8 +368,11 @@ exports[`Login User should call function onLogin when click button Login with fi
                 }
                 className="Login-text-4"
                 error={false}
+                helperText=""
+                inputRef={[Function]}
                 label="Type your email"
                 onChange={[Function]}
+                onKeyUp={[Function]}
                 required={false}
                 select={false}
                 variant="standard"
@@ -375,13 +389,21 @@ exports[`Login User should call function onLogin when click button Login with fi
                 }
                 className="Login-text-4"
                 error={false}
+                helperText=""
                 label="Password"
                 onChange={[Function]}
+                onKeyUp={[Function]}
                 required={false}
                 select={false}
                 type="password"
                 variant="standard"
               />
+              <WithStyles(Typography)
+                align="center"
+                className="Login-textError-10"
+                gutterBottom={true}
+                variant="subtitle2"
+              />
               <div
                 style={
                   Object {
@@ -591,8 +613,11 @@ exports[`Login User should render 2 inputs and a Button 1`] = `
                 }
                 className="Login-text-4"
                 error={false}
+                helperText=""
+                inputRef={[Function]}
                 label="Type your email"
                 onChange={[Function]}
+                onKeyUp={[Function]}
                 required={false}
                 select={false}
                 variant="standard"
@@ -609,13 +634,21 @@ exports[`Login User should render 2 inputs and a Button 1`] = `
                 }
                 className="Login-text-4"
                 error={false}
+                helperText=""
                 label="Password"
                 onChange={[Function]}
+                onKeyUp={[Function]}
                 required={false}
                 select={false}
                 type="password"
                 variant="standard"
               />
+              <WithStyles(Typography)
+                align="center"
+                className="Login-textError-10"
+                gutterBottom={true}
+                variant="subtitle2"
+              />
               <div
                 style={
                   Object {
@@ -825,8 +858,11 @@ exports[`Login User should render 2 inputs and a Button 2`] = `
                 }
                 className="Login-text-4"
                 error={false}
+                helperText=""
+                inputRef={[Function]}
                 label="Type your email"
                 onChange={[Function]}
+                onKeyUp={[Function]}
                 required={false}
                 select={false}
                 variant="standard"
@@ -843,13 +879,21 @@ exports[`Login User should render 2 inputs and a Button 2`] = `
                 }
                 className="Login-text-4"
                 error={false}
+                helperText=""
                 label="Password"
                 onChange={[Function]}
+                onKeyUp={[Function]}
                 required={false}
                 select={false}
                 type="password"
                 variant="standard"
               />
+              <WithStyles(Typography)
+                align="center"
+                className="Login-textError-10"
+                gutterBottom={true}
+                variant="subtitle2"
+              />
               <div
                 style={
                   Object {

diff --git a/src/client/components/Login/index.js b/src/client/components/Login/index.js
@@ -98,7 +98,9 @@ class Login extends React.Component<Props, State> {
   }
 
   componentDidMount() {
-    this._inputEmail.focus();
+    if (this._inputEmail !== undefined) {
+      this._inputEmail.focus();
+    }
   }
 
   _onLogin() {