From 588a129496f987c7b8b361e737bcd1a5d279e9ba Mon Sep 17 00:00:00 2001
From: Tatu Aalto <aalto.tatu@gmail.com>
Date: Mon, 24 Jun 2019 23:17:25 +0300
Subject: [PATCH] Choose File keyword should not fail if file_path is not file

---
 atest/acceptance/keywords/forms_and_buttons.robot | 12 +++++++++++-
 src/SeleniumLibrary/keywords/formelement.py       | 11 ++++++-----
 2 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/atest/acceptance/keywords/forms_and_buttons.robot b/atest/acceptance/keywords/forms_and_buttons.robot
index 89a1b72f7..bf6ff0905 100644
--- a/atest/acceptance/keywords/forms_and_buttons.robot
+++ b/atest/acceptance/keywords/forms_and_buttons.robot
@@ -51,7 +51,6 @@ Click button created with <button> by tag content
     Verify Location Is "${FORM SUBMITTED}"
 
 Choose File
-    [Tags]    Known Issue Internet Explorer    Known Issue Safari
     [Setup]    Navigate To File Upload Form And Create Temp File To Upload
     Choose File    file_to_upload    ${CURDIR}${/}temp.txt
     # Not sure do you get "C:\fakepath\" prefix with all browsers but at least
@@ -59,6 +58,17 @@ Choose File
     Textfield Value Should Be    file_to_upload    C:\\fakepath\\temp.txt
     [Teardown]    Remove File    ${CURDIR}${/}temp.txt
 
+Choose File And File Does Not Exist
+    [Setup]    Go To Page "forms/file_upload_form.html"
+    Run Keyword And Expect Error
+    ...     	InvalidArgumentException: Message:*
+    ...    Choose File    file_to_upload    ${CURDIR}${/}NotHere.txt
+
+Choose File And Folder
+    [Setup]    Go To Page "forms/file_upload_form.html"
+    Choose File    file_to_upload    ${CURDIR}
+    Textfield Value Should Be    file_to_upload    C:\\fakepath\\keywords
+
 Click Image With Submit Type Images
     [Setup]    Go To Page "forms/form_with_image_submit.html"
     Click Image    robot.bmp
diff --git a/src/SeleniumLibrary/keywords/formelement.py b/src/SeleniumLibrary/keywords/formelement.py
index f723390be..f91710d1f 100644
--- a/src/SeleniumLibrary/keywords/formelement.py
+++ b/src/SeleniumLibrary/keywords/formelement.py
@@ -196,20 +196,21 @@ def choose_file(self, locator, file_path):
         """Inputs the ``file_path`` into file input field ``locator``.
 
         This keyword is most often used to input files into upload forms.
-        The file specified with ``file_path`` must be available on machine
-        where tests are executed. When using Selenium Grid, Seleniun will,
+        Keyword  does not check ``file_path`` is the file or folder
+        available on machine where tests are executed. If the ``file_path``
+        point sot a file and when using Selenium Grid, Selenium will,
         [https://seleniumhq.github.io/selenium/docs/api/py/webdriver_remote/selenium.webdriver.remote.command.html?highlight=upload#selenium.webdriver.remote.command.Command.UPLOAD_FILE|magically],
         transfer the file from the machine where test are executed
         to the Selenium Grid node where the browser is running. 
         Then Selenium will send the file path, from to node file 
         system, to the browser.
 
+        ``file_path`` is not checked is new in SeleniumLibrary 4.0
+
         Example:
         | `Choose File` | my_upload_field | ${CURDIR}/trades.csv |
         """
-        if not os.path.isfile(file_path):
-            raise ValueError("File '%s' does not exist on the local file "
-                             "system." % file_path)
+        self.info('Sending %s to browser.' % os.path.abspath(file_path))
         self.find_element(locator).send_keys(file_path)
 
     @keyword