-
Notifications
You must be signed in to change notification settings - Fork 545
[SDK] Test coverage TOOL-2241 #5826
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5826 +/- ##
==========================================
+ Coverage 54.52% 55.18% +0.66%
==========================================
Files 1105 1123 +18
Lines 59332 59611 +279
Branches 4921 5030 +109
==========================================
+ Hits 32349 32895 +546
+ Misses 26263 25996 -267
Partials 720 720
*This pull request uses carry forward flags. Click here to find out more.
|
960599a
to
9022cdd
Compare
9022cdd
to
46ad2f0
Compare
ae02146
to
7d7fec7
Compare
Merge activity
|
Holiday project. Basically trying to improve test coverage, especially in those low-haning-fruit utils files <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on enhancing documentation, improving error handling, and adding tests across various modules in the codebase. It also corrects minor typos and ensures better code readability. ### Detailed summary - Added `@internal` documentation comments to `Queue` and `convertViemChain`. - Updated test cases to use `await expect(...)` for consistency. - Fixed typos in file imports and function names. - Introduced new tests for `namehash`, `encodeLabelhash`, and `parseNFT`. - Enhanced error handling in `parseNftUri`. - Improved the `toSemver` function and its tests. - Added tests for `replaceBigInts` and `toHex` functions. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
7d7fec7
to
748edf9
Compare
Holiday project. Basically trying to improve test coverage, especially in those low-haning-fruit utils files <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on enhancing documentation, improving error handling, and adding tests across various modules in the codebase. It also corrects minor typos and ensures better code readability. ### Detailed summary - Added `@internal` documentation comments to `Queue` and `convertViemChain`. - Updated test cases to use `await expect(...)` for consistency. - Fixed typos in file imports and function names. - Introduced new tests for `namehash`, `encodeLabelhash`, and `parseNFT`. - Enhanced error handling in `parseNftUri`. - Improved the `toSemver` function and its tests. - Added tests for `replaceBigInts` and `toHex` functions. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
748edf9
to
03bf859
Compare
Holiday project.
Basically trying to improve test coverage, especially in those low-haning-fruit utils files
PR-Codex overview
This PR primarily focuses on enhancing the codebase by adding internal documentation, improving error handling in various functions, and ensuring consistency in function exports. Additionally, it includes new test cases for several utility functions.
Detailed summary
@internal
annotations toQueue
andconvertViemChain
.export
for better visibility.await expect(...)
for promise handling.namehash
,encodeLabelhash
, and other utility functions.parseNftUri
andgetContract
.isHttpUrl
,formatUniversalUrl
, andformatNativeUrl
with more test coverage.