From 472433f614cdf4db1456542432abf66947335354 Mon Sep 17 00:00:00 2001
From: Alexander Kedrik <alkedr@yandex-team.ru>
Date: Wed, 23 Jul 2014 21:18:22 +0400
Subject: [PATCH 1/4] fix issue #65

---
 .../qatools/htmlelements/element/Form.java     | 18 ++++++++++++++----
 .../htmlelements/testelements/MockedForm.java  |  1 +
 2 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/htmlelements-java/src/main/java/ru/yandex/qatools/htmlelements/element/Form.java b/htmlelements-java/src/main/java/ru/yandex/qatools/htmlelements/element/Form.java
index 450f8e9d..ba6a4a8a 100644
--- a/htmlelements-java/src/main/java/ru/yandex/qatools/htmlelements/element/Form.java
+++ b/htmlelements-java/src/main/java/ru/yandex/qatools/htmlelements/element/Form.java
@@ -1,6 +1,8 @@
 package ru.yandex.qatools.htmlelements.element;
 
+import org.apache.commons.lang3.StringUtils;
 import org.openqa.selenium.By;
+import org.openqa.selenium.Keys;
 import org.openqa.selenium.WebElement;
 
 import java.util.List;
@@ -68,23 +70,31 @@ protected void fillElement(WebElement element, Object value) {
 
         if (isInput(element)) {
             String inputType = element.getAttribute("type");
-            if (inputType == null || inputType.equals(TEXT_INPUT_TYPE) || inputType.equals(PASSWORD_INPUT_TYPE)) {
-                element.sendKeys(value.toString());
-            } else if (inputType.equals(CHECKBOX_TYPE)) {
+            if (inputType.equals(CHECKBOX_TYPE)) {
                 CheckBox checkBox = new CheckBox(element);
                 checkBox.set(Boolean.parseBoolean(value.toString()));
             } else if (inputType.equals(RADIO_TYPE)) {
                 Radio radio = new Radio(element);
                 radio.selectByValue(value.toString());
+            } else {
+                element.sendKeys(getClearTextInputElementCharSequence(element) + value.toString());
             }
         } else if (isSelect(element)) {
             Select select = new Select(element);
             select.selectByValue(value.toString());
         } else if (isTextArea(element)) {
-            element.sendKeys(value.toString());
+            element.sendKeys(getClearTextInputElementCharSequence(element) + value.toString());
         }
     }
 
+    // Returns sequence of backspaces and deletes that will clear element
+    // element.clear() can't be used because clear() generates separate onchange event
+    // element must be <textarea> or <input> with type text, password, email etc
+    // See https://github.com/yandex-qatools/htmlelements/issues/65
+    private static String getClearTextInputElementCharSequence(WebElement element) {
+        return StringUtils.repeat(Keys.DELETE.toString() + Keys.BACK_SPACE, element.getText().length());
+    }
+
     private boolean isInput(WebElement element) {
         return "input".equals(element.getTagName());
     }
diff --git a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
index 495ff034..ff036952 100644
--- a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
+++ b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
@@ -47,6 +47,7 @@ private WebElement mockInputWithNameAndType(String name, String type) {
         when(element.getTagName()).thenReturn("input");
         when(element.getAttribute("name")).thenReturn(name);
         when(element.getAttribute("type")).thenReturn(type);
+        when(element.getText()).thenReturn("");
         return element;
     }
 

From 3ca92340b65583a0d75e410723f8e0ae5b38c5b3 Mon Sep 17 00:00:00 2001
From: Alexander Kedrik <alkedr@yandex-team.ru>
Date: Wed, 23 Jul 2014 21:20:54 +0400
Subject: [PATCH 2/4] fix FormFillingTest for textarea

---
 .../ru/yandex/qatools/htmlelements/testelements/MockedForm.java  | 1 +
 1 file changed, 1 insertion(+)

diff --git a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
index ff036952..817b4f43 100644
--- a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
+++ b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
@@ -90,6 +90,7 @@ private WebElement mockTextArea() {
         when(getWrappedElement().findElements(By.name(TEXT_AREA_NAME))).thenReturn(Arrays.asList(textArea));
         when(textArea.getTagName()).thenReturn("textarea");
         when(textArea.getAttribute("name")).thenReturn(TEXT_AREA_NAME);
+        when(textArea.getText()).thenReturn("");
         return textArea;
     }
 

From 6b342a6b1c9efefb40e3b182c7dbf5a4ddc6b414 Mon Sep 17 00:00:00 2001
From: Alexander Kedrik <alkedr@yandex-team.ru>
Date: Wed, 6 Aug 2014 23:10:26 +0400
Subject: [PATCH 3/4] add tests for filling text inputs and text areas when
 they already contain some text

---
 .../ru/yandex/qatools/htmlelements/FormFillingTest.java     | 6 ++++--
 .../qatools/htmlelements/testelements/MockedForm.java       | 4 ++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/FormFillingTest.java b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/FormFillingTest.java
index 166524ba..f74179bb 100644
--- a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/FormFillingTest.java
+++ b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/FormFillingTest.java
@@ -2,6 +2,7 @@
 
 import org.junit.Before;
 import org.junit.Test;
+import org.openqa.selenium.Keys;
 import ru.yandex.qatools.htmlelements.testelements.MockedForm;
 
 import java.util.HashMap;
@@ -15,6 +16,7 @@
  */
 public class FormFillingTest {
     private static final String INPUT_TEXT_TO_SEND = "text";
+    private static final String INPUT_KEYS_TO_SEND = Keys.DELETE.toString() + Keys.BACK_SPACE.toString() + "text";
     private static final boolean CHECKBOX_VALUE_TO_SET = true;
 
     private final MockedForm form = new MockedForm();
@@ -35,10 +37,10 @@ public void fillForm() {
 
     @Test
     public void formFieldsShouldBeFilledCorrectly() {
-        verify(form.getTextInput()).sendKeys(INPUT_TEXT_TO_SEND);
+        verify(form.getTextInput()).sendKeys(INPUT_KEYS_TO_SEND);
         verify(form.getCheckBox()).click();
         verify(form.getRadioButton()).click();
         verify(form.getSelectOption()).click();
-        verify(form.getTextArea()).sendKeys(INPUT_TEXT_TO_SEND);
+        verify(form.getTextArea()).sendKeys(INPUT_KEYS_TO_SEND);
     }
 }
diff --git a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
index 817b4f43..91746f04 100644
--- a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
+++ b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
@@ -47,13 +47,13 @@ private WebElement mockInputWithNameAndType(String name, String type) {
         when(element.getTagName()).thenReturn("input");
         when(element.getAttribute("name")).thenReturn(name);
         when(element.getAttribute("type")).thenReturn(type);
-        when(element.getText()).thenReturn("");
         return element;
     }
 
     private WebElement mockTextInput() {
         WebElement textInput = mockInputWithNameAndType(TEXT_INPUT_NAME, "text");
         when(getWrappedElement().findElements(By.name(TEXT_INPUT_NAME))).thenReturn(Arrays.asList(textInput));
+        when(textInput.getText()).thenReturn("a");
         return textInput;
     }
 
@@ -90,7 +90,7 @@ private WebElement mockTextArea() {
         when(getWrappedElement().findElements(By.name(TEXT_AREA_NAME))).thenReturn(Arrays.asList(textArea));
         when(textArea.getTagName()).thenReturn("textarea");
         when(textArea.getAttribute("name")).thenReturn(TEXT_AREA_NAME);
-        when(textArea.getText()).thenReturn("");
+        when(textArea.getText()).thenReturn("a");
         return textArea;
     }
 

From 7b1ec96a12172512645ed4cd6dfaf4310a5c41da Mon Sep 17 00:00:00 2001
From: Alexander Kedrik <alexander.kedrik@gmail.com>
Date: Thu, 7 Aug 2014 21:50:46 +0400
Subject: [PATCH 4/4] test both filling empty and not empty text inputs

---
 .../qatools/htmlelements/FormFillingTest.java | 10 ++++--
 .../htmlelements/testelements/MockedForm.java | 34 +++++++++++++------
 2 files changed, 31 insertions(+), 13 deletions(-)

diff --git a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/FormFillingTest.java b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/FormFillingTest.java
index f74179bb..a43dc0b7 100644
--- a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/FormFillingTest.java
+++ b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/FormFillingTest.java
@@ -16,7 +16,7 @@
  */
 public class FormFillingTest {
     private static final String INPUT_TEXT_TO_SEND = "text";
-    private static final String INPUT_KEYS_TO_SEND = Keys.DELETE.toString() + Keys.BACK_SPACE.toString() + "text";
+    private static final String INPUT_WITH_TEXT_KEYS_TO_SEND = Keys.DELETE.toString() + Keys.BACK_SPACE.toString() + "text";
     private static final boolean CHECKBOX_VALUE_TO_SET = true;
 
     private final MockedForm form = new MockedForm();
@@ -26,10 +26,12 @@ public void fillForm() {
         // Prepare data to fill form with
         Map<String, Object> data = new HashMap<String, Object>();
         data.put(MockedForm.TEXT_INPUT_NAME, INPUT_TEXT_TO_SEND);
+        data.put(MockedForm.TEXT_INPUT_WITH_TEXT_NAME, INPUT_TEXT_TO_SEND);
         data.put(MockedForm.CHECKBOX_NAME, CHECKBOX_VALUE_TO_SET);
         data.put(MockedForm.RADIO_NAME, MockedForm.RADIO_BUTTON_VALUE);
         data.put(MockedForm.SELECT_NAME, MockedForm.SELECT_OPTION_VALUE);
         data.put(MockedForm.TEXT_AREA_NAME, INPUT_TEXT_TO_SEND);
+        data.put(MockedForm.TEXT_AREA_WITH_TEXT_NAME, INPUT_TEXT_TO_SEND);
 
         // Fill form
         form.fill(data);
@@ -37,10 +39,12 @@ public void fillForm() {
 
     @Test
     public void formFieldsShouldBeFilledCorrectly() {
-        verify(form.getTextInput()).sendKeys(INPUT_KEYS_TO_SEND);
+        verify(form.getTextInput()).sendKeys(INPUT_TEXT_TO_SEND);
+        verify(form.getTextInputWithText()).sendKeys(INPUT_WITH_TEXT_KEYS_TO_SEND);
         verify(form.getCheckBox()).click();
         verify(form.getRadioButton()).click();
         verify(form.getSelectOption()).click();
-        verify(form.getTextArea()).sendKeys(INPUT_KEYS_TO_SEND);
+        verify(form.getTextArea()).sendKeys(INPUT_TEXT_TO_SEND);
+        verify(form.getTextAreaWithText()).sendKeys(INPUT_WITH_TEXT_KEYS_TO_SEND);
     }
 }
diff --git a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
index 91746f04..67dd0f0f 100644
--- a/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
+++ b/htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/testelements/MockedForm.java
@@ -15,19 +15,23 @@
  */
 public class MockedForm extends Form {
     public static final String TEXT_INPUT_NAME = "textinput";
+    public static final String TEXT_INPUT_WITH_TEXT_NAME = "textinput_with_text";
     public static final String CHECKBOX_NAME = "checkbox";
     public static final String SELECT_NAME = "select";
     public static final String RADIO_NAME = "radio";
     public static final String TEXT_AREA_NAME = "textarea";
+    public static final String TEXT_AREA_WITH_TEXT_NAME = "textarea_with_text";
 
     public static final String RADIO_BUTTON_VALUE = "value";
     public static final String SELECT_OPTION_VALUE = "value";
 
     private final WebElement textInput;
+    private final WebElement textInputWithText;
     private final WebElement checkBox;
     private final WebElement selectOption;
     private final WebElement radioButton;
     private final WebElement textArea;
+    private final WebElement textAreaWithText;
 
     public MockedForm() {
         this(mock(WebElement.class));
@@ -35,11 +39,13 @@ public MockedForm() {
 
     private MockedForm(WebElement wrappedElement) {
         super(wrappedElement);
-        textInput = mockTextInput();
+        textInput = mockTextInput(TEXT_INPUT_NAME, "");
+        textInputWithText = mockTextInput(TEXT_INPUT_WITH_TEXT_NAME, "a");
         checkBox = mockCheckBox();
         selectOption = mockSelectOption();
         radioButton = mockRadioButton();
-        textArea = mockTextArea();
+        textArea = mockTextArea(TEXT_AREA_NAME, "");
+        textAreaWithText = mockTextArea(TEXT_AREA_WITH_TEXT_NAME, "a");
     }
 
     private WebElement mockInputWithNameAndType(String name, String type) {
@@ -50,10 +56,10 @@ private WebElement mockInputWithNameAndType(String name, String type) {
         return element;
     }
 
-    private WebElement mockTextInput() {
-        WebElement textInput = mockInputWithNameAndType(TEXT_INPUT_NAME, "text");
-        when(getWrappedElement().findElements(By.name(TEXT_INPUT_NAME))).thenReturn(Arrays.asList(textInput));
-        when(textInput.getText()).thenReturn("a");
+    private WebElement mockTextInput(String name, String text) {
+        WebElement textInput = mockInputWithNameAndType(name, "text");
+        when(getWrappedElement().findElements(By.name(name))).thenReturn(Arrays.asList(textInput));
+        when(textInput.getText()).thenReturn(text);
         return textInput;
     }
 
@@ -85,12 +91,12 @@ private WebElement mockSelectOption() {
         return selectOption;
     }
 
-    private WebElement mockTextArea() {
+    private WebElement mockTextArea(String name, String text) {
         WebElement textArea = mock(WebElement.class);
-        when(getWrappedElement().findElements(By.name(TEXT_AREA_NAME))).thenReturn(Arrays.asList(textArea));
+        when(getWrappedElement().findElements(By.name(name))).thenReturn(Arrays.asList(textArea));
         when(textArea.getTagName()).thenReturn("textarea");
-        when(textArea.getAttribute("name")).thenReturn(TEXT_AREA_NAME);
-        when(textArea.getText()).thenReturn("a");
+        when(textArea.getAttribute("name")).thenReturn(name);
+        when(textArea.getText()).thenReturn(text);
         return textArea;
     }
 
@@ -98,6 +104,10 @@ public WebElement getTextInput() {
         return textInput;
     }
 
+    public WebElement getTextInputWithText() {
+        return textInputWithText;
+    }
+
     public WebElement getCheckBox() {
         return checkBox;
     }
@@ -113,4 +123,8 @@ public WebElement getRadioButton() {
     public WebElement getTextArea() {
         return textArea;
     }
+
+    public WebElement getTextAreaWithText() {
+        return textAreaWithText;
+    }
 }