kdaud
(Kakumirizi Daud)
November 1, 2021, 9:31am
1
The hard coded Request Appointment - Appointment Types
aren’t rendered on the UI for the test to select a type on the latest refapp module. The Appointment Type is hard coded and the test is set to enter one of them within the system for the save button to be active.
The test is passing well when run against demo.openmrs.org however, we set our automated tests to run against the latest refapp SNAPSHOT. Most likely there might be a merge on the module that renders Request Appointment - Appointment Type on the UI(which am not certain the exact module).
The test passes on demo.openmrs.org server
The test fails on qa-refapp.openmrs.org (this server uses the latest modules of refapp)
any clue on how we can have this bug sorted to have the test pass?
cc: @sharif @ibacher @dkayiwa
jwnasambu
(Juliet Wamalwa)
November 1, 2021, 10:05am
2
@kdaud You are right that is a bug which was fixed on this ticket by @cosmin and I tested it works perfect and I was to fix the css part and he also tests then its good to go. I promise it will be done by 5pm EAT. Am sorry I was caught-up yesterday and other weekends.
kdaud
(Kakumirizi Daud)
November 1, 2021, 10:20am
3
Thanks @jwnasambu for the information.
I anxiously look forward to your fixing this !
dkayiwa
(Daniel Kayiwa)
November 1, 2021, 11:45am
4
This was caused by some one who changed qa-refapp from platform 2.4.0 to 2.5.0-alpha, leading to the failure of the REST call that fetches appointment types. openmrs-module-appointmentscheduling/AppointmentTypeResource1_9.java at master · openmrs/openmrs-module-appointmentscheduling · GitHub
@kdaud i used the page you referenced above to see the failure from Chrome’s developer tools network tab.
2 Likes
kdaud
(Kakumirizi Daud)
November 1, 2021, 11:56am
5
Thanks @dkayiwa for the hint.
Lesson well taught and the student has fully grasped the concept
jwnasambu
(Juliet Wamalwa)
November 1, 2021, 3:10pm
6
@dkayiwa thanks for the clarification on cause of the test failure. @kdaud am sorry am unable to beat the set time because of my poor internet. Nevertheless am grateful the bug is not related to the ticket I shared above.
kdaud
(Kakumirizi Daud)
November 1, 2021, 4:25pm
7
@jwnasambu the bug was caught by the test and fixed by @dkayiwa . See more info on the subject at
kdaud
(Kakumirizi Daud)
November 4, 2021, 9:58am
8
@dkayiwa @tendomart seems like https://qa-refapp.openmrs.org/openmrs/ has been switched from platform 2.4.0 to 2.5.0 aplha3 and the REST call that fetches appointment types has not updated its resources to include support for platform 2.5.0. Am noticing this from the clinical visit test workflow run failing to pass a user story of Booking an appointment
dkayiwa
(Daniel Kayiwa)
November 4, 2021, 10:08am
9
@kdaud do you mind raising a pull request for the appointmentscheduling module?
tendomart
(tendo kiiza Martyn)
November 4, 2021, 10:56am
10
It’s 2.5.0-SNAPSHOT though am not stil sure wether this PR could have triggered the changes
dkayiwa
(Daniel Kayiwa)
November 4, 2021, 11:05am
11
Do not worry @tendomart . It was done by dkayiwa
1 Like
tendomart
(tendo kiiza Martyn)
November 4, 2021, 11:07am
12
anyways , got scared because my local spin is also 2.5.0-SNAPSHOT…
But i have doen this update , core is still failing locally regardless.
openmrs:master
← openmrs:tendomart-patch-1
opened 10:59AM - 04 Nov 21 UTC
## Description of what I changed
## Issue I worked on
see https:… //issues.openmrs.org/browse/TRUNK-
## Checklist: I completed these to help reviewers :)
- [ ] My IDE is configured to follow the [**code style**](https://wiki.openmrs.org/display/docs/Java+Conventions) of this project.
No? Unsure? -> [configure your IDE](https://wiki.openmrs.org/display/docs/How-To+Setup+And+Use+Your+IDE), format the code and add the changes with `git add . && git commit --amend`
- [ ] I have **added tests** to cover my changes. (If you refactored
existing code that was well tested you do not have to add tests)
No? -> write tests and add them to this commit `git add . && git commit --amend`
- [ ] I ran `mvn clean package` right before creating this pull request and
added all formatting changes to my commit.
No? -> execute above command
- [ ] All new and existing **tests passed**.
No? -> figure out why and add the fix to your commit. It is your responsibility to make sure your code works.
- [ ] My pull request is **based on the latest changes** of the master branch.
No? Unsure? -> execute command `git pull --rebase upstream master`
kdaud
(Kakumirizi Daud)
November 4, 2021, 4:29pm
13
The pull request below has the best answer to the question
openmrs:master
← kdaud:RATEST
opened 01:46PM - 04 Nov 21 UTC
Ticket [ID](https://issues.openmrs.org/browse/AM-223)
Description -> Update i… ts resources to include support for platform 2.5.0
dkayiwa
(Daniel Kayiwa)
November 4, 2021, 6:18pm
14
@kdaud merged with thanks! Did you go ahead and confirm that qa-refapp is now happy?
kdaud
(Kakumirizi Daud)
November 4, 2021, 6:40pm
15
@dkayiwa the trusted report shows qa-refapp is happy and satisfied with the changes.