Error on Uitests qa branch using Selenium Testing

1

Hello Friends, Have been working on quality assurance working on this ticket, and we have been encounting same error as seen here, and some other pull requests here as i was trying to debug this happened to some other friends of mine who are working on quality assurance tickets facing the same error which almost got us blocked for some while :slightly_smiling_face:

Procedures to follow to re produce same error can be found following this link QAFramework Technical documentation(Cucumber-Selenium&Cypress Set up) - Documentation - OpenMRS Wiki, making sure you try to ignore any single test, since this is community priority quality assurance work , your emergency help will be appreciated and keep us moving on well thanks cc @k.joseph @dkayiwa @ibacher @raff

2

is there anyone facing this among the qa contributors; @kdaud ? etc

You need to be sure travis is running the qa branch @sharif

1 Like
3

Sorry for late reply, sure am running on qa branch as seen here , and btw seems this happens even when there is no change that has been made

4

follow the travis config to will realise the parent project isn’t compiled anywhere above

5

Locally the build plan passes. However, when triggering a commit on the upstream qa branch the ci plan fails. Have just made a pr in respect to the this ticket

6

I understand this phrase as: When making a pr, ensure that the commit is made against qa branch of the distro. I am positive for correction in respect to the interpretation

7

Sorry basically parent project is master branch of uitest framework or whole distro refapp! some clarification here thanks

8

@k.joseph As you can see it here with @kdaud builds its the same error

9

Going through the error log, it seems the noise is coming from the method clickOnLast(org.openqa.selenium.By) present at org.openmrs.reference.page.ClinicianFacingPatientDashboardPag. Ideally this method was introduced on qa branch and is not present on master But this should not be the cause as I may think

10

@k.joseph this error continues to appear on qa branch with clickOnLast method, can we try to use any other method incase of clickOnLast ,may be it that might also release us

11

Have gone through @herbert24 commits against qa branch module in respect to qa technical work and his ci plans are passing. I think we may be missing something obvious. @herbert24 do you have something to share with us on this point.

12

well am re testing a couple of things and will get back here as requested cc @kdaud @sharif

13

How fur with testing make sure please you are qa branch thanks

14

Do you mind sharing pull request that is passing

15

Have just picked one of them here. Hope @herbert24 will take a look at the review have made in respect to the commit

16

May be as @herbert24 is looking into the comments on pull requests , i guess some pull requests may pass on travis only and only when that work being done is not related with Page class which will be rare at the moment since we are working with pages and feature files .

17

I think the ci plan will fail no matter where the work is done as long as a commit is triggered against the qa branch. I am investigating time into this issue to find out the root cause and ASAP will share my opinion. I think @k.joseph is looking into this as well

18

FYI debugging considers to work on single line of error log per log , if you jump a single line it may be tricky to track unless some one is more experienced in debugging , so i think if that error is handled first we may land on other error log or getting ci to pass

19

What is your recommendation then

20

Suggesting that, Since clickOn() have the same functionality with clickOnLast() under the hood , would suggest that we first use clickOn method since it passes both locally and travis, though still wondering why that method failed travis even if its introduced on master branch , hopefully in todays call @k.joseph will clarify on this point, otherwise this blocker is making work not progressing