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

test: dict.Data.String and dict.NewFromIfoPath #31

Merged
merged 2 commits into from
Nov 4, 2024
Merged

Conversation

ianlewis
Copy link
Owner

@ianlewis ianlewis commented Nov 4, 2024

Description:

  • Add test for dict.Data.String
  • Add test for dict.NewFromIfoPath

Related Issues:

N/A

Checklist:

  • Review the CONTRIBUTING.md documentation.
  • Add a reference to a related issue in the repository.
  • Add a description of the changes proposed in the pull request.
  • Add unit tests if applicable.
  • Update documentation if applicable.
  • Add a note in the CHANGELOG.md if applicable.
  • Sign the Google CLA.

Signed-off-by: Ian Lewis <ianmlewis@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented Nov 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.89%. Comparing base (05e878b) to head (6e11369).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
+ Coverage   35.76%   41.89%   +6.13%     
==========================================
  Files          11       11              
  Lines         685      685              
==========================================
+ Hits          245      287      +42     
+ Misses        407      361      -46     
- Partials       33       37       +4     

see 1 file with indirect coverage changes

Signed-off-by: Ian Lewis <ianmlewis@gmail.com>
@ianlewis ianlewis merged commit 8fb3bc1 into main Nov 4, 2024
13 checks passed
@ianlewis ianlewis deleted the dict-units branch February 5, 2025 08:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants