Hello Everyone,
The fourth week of coding Period has been completed and I am excited to provide updated on the project progress during these four weeks. I was working on fixing the existing tests which were getting skipped and had also created an epic for that. I had completed the epic but found that we are left with one more component which tests are getting skipped i.e. identifier-input component. I created another issue for this. Currently this is in progress and getting hard time in fixing these tests. This is the only component left from the epic whose PR is not ready for review yet. Other than that I have two PR which are under review. The PR Links are:-
openmrs:main ← ayush-AI:patient-registration
opened 08:10PM - 25 May 23 UTC
## Requirements
- [x] This PR has a title that briefly describes the work don… e including the ticket number. If there is a ticket, make sure your PR title includes a [conventional commit](https://o3-dev.docs.openmrs.org/#/getting_started/contributing?id=your-pr-title-should-indicate-the-type-of-change-it-is) label. See existing PR titles for inspiration.
- [ ] My work conforms to the [**OpenMRS 3.0 Styleguide**](https://om.rs/styleguide) and [**design documentation**](https://zeroheight.com/23a080e38/p/880723-introduction).
- [ ] I checked for feature overlap with [**existing widgets**](https://om.rs/directory).
## Summary
In this PR, I fixed all the failing test cases for [patient-registation.test.tsx](https://github.com/openmrs/openmrs-esm-patient-management/blob/c9e9e6b6dea780a43e08b9c2ff61f56cdc702a49/packages/esm-patient-registration-app/src/patient-registration/patient-registration.test.tsx) file. There were multiple tests failing due to multiple reasons. I addressed those issues and now all the tests are passing except [this one](https://github.com/ayush-AI/openmrs-esm-patient-management/blob/9294feaa15f4eb110db5766f20b76b2fe7c9465e/packages/esm-patient-registration-app/src/patient-registration/patient-registration.test.tsx#LL268C6-L268C6). I have added necessary comments why the test is failing and also commented the approaches that I took in attempt to fix the issue, but the test is still failing.
<!--
Required.
Please describe what problems your PR addresses.
-->
## Screenshots
*None.*
<!--
Optional.
If possible, please insert any screenshots/videos of your changes here.
Don't forget to remove the *None.* above if you do fill this section.
-->
## Related Issue
https://issues.openmrs.org/browse/O3-2082
<!--
Required if applicable.
If present, please link any related issue here, e.g. "https://issues.openmrs.org/browse/123").
Don't forget to remove the *None.* above if you do fill this section.
-->
## Other
Please look into the failing test and suggest some ways to fix it.
*None.*
<!--
Optional.
Anything else that isn't covered by one of the sections above.
Don't forget to remove the *None.* above if you do fill this section.
-->
openmrs:main ← ayush-AI:death-info-section
opened 08:05AM - 26 Apr 23 UTC
## Requirements
- [x] This PR has a title that briefly describes the work don… e including the ticket number. If there is a ticket, make sure your PR title includes a [conventional commit](https://o3-dev.docs.openmrs.org/#/getting_started/contributing?id=your-pr-title-should-indicate-the-type-of-change-it-is) label. See existing PR titles for inspiration.
- [ ] My work conforms to the [**OpenMRS 3.0 Styleguide**](https://om.rs/styleguide) and [**design documentation**](https://zeroheight.com/23a080e38/p/880723-introduction).
- [ ] I checked for feature overlap with [**existing widgets**](https://om.rs/directory).
## Summary
I have updated the tests for the death-info-section component in this PR. I noticed that the tests were failing due to the missing isDead context, causing them to be skipped. I fixed this issue and the tests now pass successfully.
<!--
Required.
Please describe what problems your PR addresses.
-->
## Screenshots
*None.*
<!--
Optional.
If possible, please insert any screenshots/videos of your changes here.
Don't forget to remove the *None.* above if you do fill this section.
-->
## Related Issue
https://issues.openmrs.org/browse/O3-2073
<!--
Required if applicable.
If present, please link any related issue here, e.g. "https://issues.openmrs.org/browse/123").
Don't forget to remove the *None.* above if you do fill this section.
-->
## Other
*None.*
<!--
Optional.
Anything else that isn't covered by one of the sections above.
Don't forget to remove the *None.* above if you do fill this section.
-->
One PR I got merged for fixing the patient-registration-validation component. Test for this component was integration tests. But after the suggestion of @piumal1999 I changed it to unit tests. This was all for the update for fixing the skipped tests. Let’s move to the next step i.e. the MSW setup.
We are not going to use the MSW you can check this conversation for the reasoning.
Now the next step is to write the new tests. For this I have decided to go by each micro-frontend one by one. I have started working on writing the new tests for the esm-patient-registration-app and created the epic for this. You can check the epic here . I have written tests for the one of the component named field.component.tsx and written all the tests but one of them is failing due to some missing mock values. I have created draft PR for this also. PR link:-
openmrs:main ← ayush-AI:field
opened 05:02PM - 24 Jun 23 UTC
## Requirements
- [x] This PR has a title that briefly describes the work don… e including the ticket number. If there is a ticket, make sure your PR title includes a [conventional commit](https://o3-dev.docs.openmrs.org/#/getting_started/contributing?id=your-pr-title-should-indicate-the-type-of-change-it-is) label. See existing PR titles for inspiration.
- [ ] My work conforms to the [**OpenMRS 3.0 Styleguide**](https://om.rs/styleguide) and [**design documentation**](https://zeroheight.com/23a080e38/p/880723-introduction).
- [ ] I checked for feature overlap with [**existing widgets**](https://om.rs/directory).
## Summary
In this PR I have added the Tests for field.component.tsx of esm-patient-registration-app.
## Screenshots
<img width="963" alt="Screenshot 2023-06-24 at 10 30 08 PM" src="https://github.com/openmrs/openmrs-esm-patient-management/assets/54752747/d313afe7-8abb-470d-afcd-3d9357181d2e">
## Related Issue
https://issues.openmrs.org/browse/O3-2215
<!--
Required if applicable.
If present, please link any related issue here, e.g. "https://issues.openmrs.org/browse/123").
Don't forget to remove the *None.* above if you do fill this section.
-->
## Other
*None.*
<!--
Optional.
Anything else that isn't covered by one of the sections above.
Don't forget to remove the *None.* above if you do fill this section.
-->
I will work on writing new tests for esm-patient-registration-app throughtout this week.
Thank you for your support. Keep supporting and keep suggesting.