Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added tests for resource_fhir_test.go -> ListResourceFhir #302

Merged
merged 3 commits into from
Nov 14, 2023

Conversation

mohanish2504
Copy link
Contributor

Modified resource_fhir_test.go to add test for ListResourceFhir api.

@mohanish2504 mohanish2504 changed the title Mohanish/resource_lists_tests added tests for resource_fhir_test.go -> ListResourceFhir Nov 11, 2023
@mohanish2504
Copy link
Contributor Author

just saw there are conflicts

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1ac8c3b) 11.68% compared to head (36e8207) 11.78%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #302      +/-   ##
==========================================
+ Coverage   11.68%   11.78%   +0.09%     
==========================================
  Files         262      262              
  Lines       38548    38548              
  Branches      521      521              
==========================================
+ Hits         4503     4541      +38     
+ Misses      33818    33779      -39     
- Partials      227      228       +1     
Flag Coverage Δ
unittests 11.78% <ø> (+0.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AnalogJ
Copy link
Member

AnalogJ commented Nov 14, 2023

Fantastic work! Thanks for the PR 🥳

@AnalogJ AnalogJ merged commit f2cb6a9 into fastenhealth:main Nov 14, 2023
4 of 5 checks passed
@AnalogJ AnalogJ mentioned this pull request Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants