Feat openemr 9877 ccda tests ci 1#10
Open
adunsulag wants to merge 3 commits into
Open
Conversation
Implemented a phpunit test for verifying certification data didn't change. I went with Alice who is the most comprehensive test patient for certification. I could do Happy Child and Jeremy but I think this is sufficient without us going and implementing the entire ETT test suite for onc certification. If someone doesn't have the private files test suite and attempts to run the certification suite it just skips the test. I also implemented a much faster test that verifies the node serveccda.js without requiring that we work on the certification test data set. It uses the OpenEMR xml generated CCDA file that is sent to the serveccda which formats it to the ccda specification format. This verifies that the node server gets started and that it properly generates the correct xml file. Fixes openemr#9877
The serveccda.js was seg faulting due to invalid node version 20 when version 22 is now required for package dependencies. I updated the version. I left it at 8.3 which is what inferno has been running on. We could bump it up to 8.4 if we wanted to in the future.
❌ 1 Tests Failed:
View the top 1 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Attempting to test ci changes.