Attached is the updated patch. I have changed fill_input_by_field_name function to use javascript to set values of text boxes and the selector is changed from xpath to CSS with a :not(:disabled) clause. With this, test case will fail if tried to set the value of a disabled input.
On Mon, Jan 28, 2019 at 5:39 PM Dave Page <dpage@pgadmin.org> wrote:
On Mon, Jan 28, 2019 at 12:05 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote: > > Hi Dave, > > On Mon, Jan 28, 2019 at 5:01 PM Dave Page <dpage@pgadmin.org> wrote: >> >> Hi >> >> On Mon, Jan 28, 2019 at 7:00 AM Aditya Toshniwal >> <aditya.toshniwal@enterprisedb.com> wrote: >> > >> > Hi Hackers, >> > >> > Attached is the patch to fix failing SQL editor related feature tests. >> > The failure is occurring mostly because of send_keys function of selenium which sometimes does not input the complete text. The change will use Codemirror javascript object to set the text. >> > >> > I have also added new args flag "--modules" which can be used after --pkg where you can test a specific module file under the pkg. This is not so useful for build but definitely helpful for the developers. >> >> One of the most common failures I see is failure to fill in the >> details on the Add Server dialogue (it might type half of an entry, >> then just hang). Should we get rid of all uses of send_keys? > > Setting the text using javascript executor can be done. send_keys simulate an actual person, so if you try to set a disabled text box using send_keys, it won't. Places where disabled fields doesn't matter, send_keys can be removed.