Hi Dave, On Mon, Jan 28, 2019 at 5:01 PM Dave Page <dp...@pgadmin.org> wrote:
> Hi > > On Mon, Jan 28, 2019 at 7:00 AM Aditya Toshniwal > <aditya.toshni...@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. > > -- > Dave Page > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > > EnterpriseDB UK: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > -- Thanks and Regards, Aditya Toshniwal Software Engineer | EnterpriseDB Software Solutions | Pune "Don't Complain about Heat, Plant a tree"
