From b13e13537f4180d9ff12bc0aac8215483c694736 Mon Sep 17 00:00:00 2001
From: Nick Abalov <n.abalov@2Gis.ru>
Date: Fri, 16 Sep 2016 16:40:31 +0700
Subject: [PATCH 1/2] Add clear element command support

---
 .../py-functional/tests/test_commands.py      | 33 ++++++++++++-
 .../TestApp.WindowsPhone/MainPage.xaml        |  1 +
 .../Winium.StoreApps.InnerServer/Automator.cs |  4 ++
 .../Commands/ClearCommand.cs                  | 41 +++++++++++++++++
 .../Commands/Helpers/ControlExtensions.cs     | 46 +++++++++++++++++++
 .../Commands/ValueCommand.cs                  | 35 +-------------
 .../Winium.StoreApps.InnerServer.csproj       |  2 +
 7 files changed, 126 insertions(+), 36 deletions(-)
 create mode 100644 Winium/Winium.StoreApps.InnerServer/Commands/ClearCommand.cs
 create mode 100644 Winium/Winium.StoreApps.InnerServer/Commands/Helpers/ControlExtensions.cs

diff --git a/Winium/TestApp.Test/py-functional/tests/test_commands.py b/Winium/TestApp.Test/py-functional/tests/test_commands.py
index 0e64fde..29db654 100644
--- a/Winium/TestApp.Test/py-functional/tests/test_commands.py
+++ b/Winium/TestApp.Test/py-functional/tests/test_commands.py
@@ -272,7 +272,7 @@ def test_attribute_set(self, attribute, value):
         assert str(value) == element.get_attribute(attribute)
 
 
-class TestBasicInput(WuaTestCase):
+class TestBasicInput(UsesSecondTab):
     __shared_session__ = False
 
     def test_send_keys_to_element(self):
@@ -285,6 +285,34 @@ def test_send_keys_to_element(self):
         element.send_keys(actual_input)
         assert actual_input.replace(Keys.ENTER, '\r\n') == element.text
 
+    def test_send_keys_to_number_input(self, second_tab):
+        """
+        POST /session/:sessionId/element/:id/value Send a sequence of key strokes to an element.
+        TODO: test magic keys
+        """
+        actual_input = '123'
+        element = second_tab.find_element_by_id('NumericInput')
+        element.send_keys(actual_input)
+        assert actual_input == element.text
+
+    def test_clear_number_input(self, second_tab):
+        """
+        POST /session/:sessionId/element/:id/value Send a sequence of key strokes to an element.
+        TODO: test magic keys
+        """
+        actual_input = '123'
+        element = second_tab.find_element_by_id('NumericInput')
+        element.send_keys(actual_input)
+        element.clear()
+        assert '' == element.text
+
+    def test_clear_element(self):
+        actual_input = 'Some test string' + Keys.ENTER
+        element = self.driver.find_element_by_id('MyTextBox')
+        element.send_keys(actual_input)
+        element.clear()
+        assert '' == element.text
+
     def test_send_keys_to_active_element(self):
         element = self.driver.find_element_by_id('MyTextBox')
         element.click()
@@ -308,7 +336,8 @@ def test_back(self):
     def test_click_element(self):
         element = self.driver.find_element_by_id('SetButton')
         element.click()
-        assert 'CARAMBA' == self.driver.find_element_by_id('MyTextBox').text
+        actual_value = self.driver.find_element_by_id('MyTextBox').text
+        assert 'CARAMBA' == actual_value
 
 
 @pytest.mark.skipif(True, reason="TODO")
diff --git a/Winium/TestApp/TestApp.WindowsPhone/MainPage.xaml b/Winium/TestApp/TestApp.WindowsPhone/MainPage.xaml
index 760a052..93d93eb 100644
--- a/Winium/TestApp/TestApp.WindowsPhone/MainPage.xaml
+++ b/Winium/TestApp/TestApp.WindowsPhone/MainPage.xaml
@@ -52,6 +52,7 @@
                 <Button Content="Disabled" 
                         IsEnabled="False"
                         AutomationProperties.AutomationId="DisabledBtn" />
+                <TextBox InputScope="Number" AutomationProperties.AutomationId="NumericInput"/>
             </StackPanel>
         </PivotItem>
     </Pivot>
diff --git a/Winium/Winium.StoreApps.InnerServer/Automator.cs b/Winium/Winium.StoreApps.InnerServer/Automator.cs
index c784a16..dee1eaf 100644
--- a/Winium/Winium.StoreApps.InnerServer/Automator.cs
+++ b/Winium/Winium.StoreApps.InnerServer/Automator.cs
@@ -105,6 +105,10 @@ public string ProcessCommand(string content)
 
                 commandToExecute = new ValueCommand { ElementId = elementId, KeyString = value };
             }
+            else if (command.Equals(DriverCommand.ClearElement))
+            {
+                commandToExecute = new ClearCommand { ElementId = elementId };
+            }
             else if (command.Equals(DriverCommand.GetElementText))
             {
                 commandToExecute = new TextCommand { ElementId = elementId };
diff --git a/Winium/Winium.StoreApps.InnerServer/Commands/ClearCommand.cs b/Winium/Winium.StoreApps.InnerServer/Commands/ClearCommand.cs
new file mode 100644
index 0000000..0de9ce3
--- /dev/null
+++ b/Winium/Winium.StoreApps.InnerServer/Commands/ClearCommand.cs
@@ -0,0 +1,41 @@
+namespace Winium.StoreApps.InnerServer.Commands
+{
+    #region
+
+    using Windows.UI.Xaml;
+    using Windows.UI.Xaml.Automation.Peers;
+    using Windows.UI.Xaml.Automation.Provider;
+    using Windows.UI.Xaml.Controls;
+
+    using Winium.StoreApps.Common;
+    using Winium.StoreApps.Common.Exceptions;
+    using Winium.StoreApps.InnerServer.Commands.Helpers;
+
+    #endregion
+
+    internal class ClearCommand : CommandBase
+    {
+        #region Public Properties
+
+        public string ElementId { get; set; }
+
+        #endregion
+
+        #region Public Methods and Operators
+
+        protected override string DoImpl()
+        {
+            var element = this.Automator.ElementsRegistry.GetRegisteredElement(this.ElementId);
+            var control = element.Element as Control;
+            if (control == null)
+            {
+                throw new AutomationException("Element referenced is not of control type.", ResponseStatus.UnknownError);
+            }
+
+            control.TrySetText(string.Empty);
+            return this.JsonResponse();
+        }
+
+        #endregion
+    }
+}
diff --git a/Winium/Winium.StoreApps.InnerServer/Commands/Helpers/ControlExtensions.cs b/Winium/Winium.StoreApps.InnerServer/Commands/Helpers/ControlExtensions.cs
new file mode 100644
index 0000000..2502e45
--- /dev/null
+++ b/Winium/Winium.StoreApps.InnerServer/Commands/Helpers/ControlExtensions.cs
@@ -0,0 +1,46 @@
+namespace Winium.StoreApps.InnerServer.Commands.Helpers
+{
+    #region
+
+    using Windows.UI.Xaml;
+    using Windows.UI.Xaml.Automation.Peers;
+    using Windows.UI.Xaml.Automation.Provider;
+    using Windows.UI.Xaml.Controls;
+
+    using Winium.StoreApps.Common;
+    using Winium.StoreApps.Common.Exceptions;
+
+    #endregion
+
+    internal static class ContorlExtensions
+    {
+        internal static void TrySetText(this Control element, string text)
+        {
+            // TODO Research why TextBox does not support IValueProvider
+            var provider = element.GetProviderOrDefault<IValueProvider>(PatternInterface.Value);
+
+            if (provider != null)
+            {
+                provider.SetValue(text);
+            }
+            else if (element is TextBox)
+            {
+                var textBox = element as TextBox;
+                textBox.Text = text;
+                textBox.SelectionStart = text.Length;
+            }
+            else if (element is PasswordBox)
+            {
+                var passwordBox = element as PasswordBox;
+                passwordBox.Password = text;
+            }
+            else
+            {
+                throw new AutomationException("Element does not support SendKeys.", ResponseStatus.UnknownError);
+            }
+
+            // TODO: new parameter - FocusState
+            element.Focus(FocusState.Pointer);
+        }
+    }
+}
diff --git a/Winium/Winium.StoreApps.InnerServer/Commands/ValueCommand.cs b/Winium/Winium.StoreApps.InnerServer/Commands/ValueCommand.cs
index 887236e..c56d297 100644
--- a/Winium/Winium.StoreApps.InnerServer/Commands/ValueCommand.cs
+++ b/Winium/Winium.StoreApps.InnerServer/Commands/ValueCommand.cs
@@ -34,43 +34,10 @@ protected override string DoImpl()
                 throw new AutomationException("Element referenced is not of control type.", ResponseStatus.UnknownError);
             }
 
-            TrySetText(control, this.KeyString);
+            control.TrySetText(this.KeyString);
             return this.JsonResponse();
         }
 
         #endregion
-
-        #region Methods
-
-        private static void TrySetText(Control element, string text)
-        {
-            // TODO Research why TextBox does not support IValueProvider
-            var provider = element.GetProviderOrDefault<IValueProvider>(PatternInterface.Value);
-
-            if (provider != null)
-            {
-                provider.SetValue(text);
-            }
-            else if (element is TextBox)
-            {
-                var textBox = element as TextBox;
-                textBox.Text = text;
-                textBox.SelectionStart = text.Length;
-            }
-            else if (element is PasswordBox)
-            {
-                var passwordBox = element as PasswordBox;
-                passwordBox.Password = text;
-            }
-            else
-            {
-                throw new AutomationException("Element does not support SendKeys.", ResponseStatus.UnknownError);
-            }
-
-            // TODO: new parameter - FocusState
-            element.Focus(FocusState.Pointer);
-        }
-
-        #endregion
     }
 }
diff --git a/Winium/Winium.StoreApps.InnerServer/Winium.StoreApps.InnerServer.csproj b/Winium/Winium.StoreApps.InnerServer/Winium.StoreApps.InnerServer.csproj
index 117f724..432f4e3 100644
--- a/Winium/Winium.StoreApps.InnerServer/Winium.StoreApps.InnerServer.csproj
+++ b/Winium/Winium.StoreApps.InnerServer/Winium.StoreApps.InnerServer.csproj
@@ -46,8 +46,10 @@
     <Compile Include="AcceptedRequest.cs" />
     <Compile Include="AutomationServer.cs" />
     <Compile Include="Automator.cs" />
+    <Compile Include="Commands\ClearCommand.cs" />
     <Compile Include="Commands\GetElementRectCommand.cs" />
     <Compile Include="Commands\GetElementSizeCommand.cs" />
+    <Compile Include="Commands\Helpers\ControlExtensions.cs" />
     <Compile Include="Commands\Helpers\IDictionaryExtensions.cs" />
     <Compile Include="Commands\IsElementEnabledCommand.cs" />
     <Compile Include="ElementsRegistry.cs" />

From edde0e8dd8c1ff716c389922f0729305e069769b Mon Sep 17 00:00:00 2001
From: Nick Abalov <n.abalov@2Gis.ru>
Date: Fri, 16 Sep 2016 16:40:57 +0700
Subject: [PATCH 2/2] Fix listener crashing on empty incoming request

---
 Winium/Winium.StoreApps.Driver/Listener.cs | 28 ++++++++++++++--------
 1 file changed, 18 insertions(+), 10 deletions(-)

diff --git a/Winium/Winium.StoreApps.Driver/Listener.cs b/Winium/Winium.StoreApps.Driver/Listener.cs
index c205e48..f885415 100644
--- a/Winium/Winium.StoreApps.Driver/Listener.cs
+++ b/Winium/Winium.StoreApps.Driver/Listener.cs
@@ -91,19 +91,27 @@ public void StartListening()
                     using (var stream = client.GetStream())
                     {
                         var acceptedRequest = HttpRequest.ReadFromStreamWithoutClosing(stream);
-                        Logger.Debug("ACCEPTED REQUEST {0}", acceptedRequest.StartingLine);
 
-                        var response = this.HandleRequest(acceptedRequest);
-                        using (var writer = new StreamWriter(stream))
+                        if (string.IsNullOrWhiteSpace(acceptedRequest.StartingLine))
                         {
-                            try
-                            {
-                                writer.Write(response);
-                                writer.Flush();
-                            }
-                            catch (IOException ex)
+                            Logger.Warn("ACCEPTED EMPTY REQUEST");
+                        }
+                        else
+                        {
+                            Logger.Debug("ACCEPTED REQUEST {0}", acceptedRequest.StartingLine);
+
+                            var response = this.HandleRequest(acceptedRequest);
+                            using (var writer = new StreamWriter(stream))
                             {
-                                Logger.Error("Error occured while writing response: {0}", ex);
+                                try
+                                {
+                                    writer.Write(response);
+                                    writer.Flush();
+                                }
+                                catch (IOException ex)
+                                {
+                                    Logger.Error("Error occured while writing response: {0}", ex);
+                                }
                             }
                         }