-
Notifications
You must be signed in to change notification settings - Fork 187
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clarify correct behavior for uploading a folder via "Element Send Keys" #1742
Comments
Also note step 8.5:
Chromedriver will not raise such an exception because it auto-populates the list with files as found in this folder. |
@sadym-chromium or @nechaev-chromium could you please have a look at this issue? When we get started to implement the file upload for BiDi we should have it spec compliant. Thanks. CC @OrKoN |
Ping @sadym-chromium or @nechaev-chromium |
The Browser Testing and Tools Working Group just discussed
The full IRC log of that discussion<AutomatedTester> Topic: Clarify correct behavior for uploading a folder via "Element Send Keys"<AutomatedTester> github: https://github.com//issues/1742 <AutomatedTester> whimboo: I have noticed this for a issue against for Firefox recently. Firefox doesnt do anything but ChromeDriver uploads all the files in the directory <AutomatedTester> ... so what should we be doing as we could break backwards compat here <AutomatedTester> q? <AutomatedTester> automatedtester: I kinda think that we should follow what chromedriver is doing here for everyone <AutomatedTester> whimboo: This will potentially upload files that people are not expecting here... so we might want to clarify things for users <AutomatedTester> q? <jgraham_> q? <MaksimSadym> q+ <sadym> q+ <shs96c> https://irc.w3.org <Randolf__> q+ <sadym> q- <AutomatedTester_> ack MaksimSadym <Randolf__> q- <MaksimSadym> q- <orkon___> q+ <AutomatedTester_> ack next <Randolf__> q+ <AutomatedTester_> orkon___: I think we should probably move chromedriver towards to firefox. We should raise a bug and see what happens on the file. <AutomatedTester_> ... and with regard to bidi, the validation is left to the client <AutomatedTester_> q? <Randolf__> q- <Randolf__> q+ <AutomatedTester_> ACTION: Chromedriver folks to see what happens on a web page when and possible move ChromeDriver to Firefox implementation <AutomatedTester_> ack next <Randolf__> q- <AutomatedTester_> Randolf__: this feels like it is OS territory here. I think we should just do something simple and move CHromedriver to just do files and not folders |
@sadym-chromium or @nechaev-chromium I wanted to ask if you already had a chance to check the action item from the November W3C meeting? Thanks. |
ping @nechaev-chromium |
I'm on it |
ChromeDriver just allows setting the directory path to : source code |
Thanks @nechaev-chromium. Would you also mind to give feedback to the 2nd part of the action item which was:
As per current classic specification using a folder is not allowed. And I think that we also don't allow that in BiDi. |
Quite some time ago we get a report for geckodriver that uploading a folder doesn't work. Hereby the relevant code didn't specify a list of files as required by the WebDriver specification for
Element Send Keys
, but just the folder name.As also mentioned chromedriver supports that method and as such adds extra behavior on top of the WebDriver specification which other drivers like at least geckodriver doesn't do.
For a consistent behavior across browsers we should most likely support a unique behavior.
CC @sadym-chromium, @foolip, @gsnedders, @jgraham
The text was updated successfully, but these errors were encountered: