Skip to content
New issue

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

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

Already on GitHub? # to your account

Add test case for JSON resource marshaling #2537

Merged
merged 1 commit into from
Oct 27, 2022

Conversation

akshaypatil3096
Copy link
Contributor

@akshaypatil3096 akshaypatil3096 commented Oct 27, 2022

Hi @gmlewis, I have added test case for ExternalGroupMember. So, pls accept my PR for Hackoctober fest contribution.

Helps: #55.

@codecov
Copy link

codecov bot commented Oct 27, 2022

Codecov Report

Merging #2537 (0d07640) into master (cb4df26) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2537   +/-   ##
=======================================
  Coverage   97.97%   97.97%           
=======================================
  Files         123      123           
  Lines       10813    10813           
=======================================
  Hits        10594    10594           
  Misses        150      150           
  Partials       69       69           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@gmlewis gmlewis changed the title Add test cases for JSON resource marshaling Add test case for JSON resource marshaling Oct 27, 2022
Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @akshaypatil3096 !
LGTM.
Merging.

@gmlewis gmlewis merged commit f550bc8 into google:master Oct 27, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants