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

chore: cherry picked #3120 to release/0.59 #3211

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

quiet-node
Copy link
Member

@quiet-node quiet-node commented Nov 5, 2024

Description:
Cherry pick #3120

Related issue(s):

Fixes #3209

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

…3210)

* fix: added null check for nullable values in formatContractResult()

Signed-off-by: Logan Nguyen <[email protected]>

* fix: fixed unit tests

Signed-off-by: Logan Nguyen <[email protected]>

---------

Signed-off-by: Logan Nguyen <[email protected]>
@quiet-node quiet-node added the process Build, test and deployment-process related tasks label Nov 5, 2024
@quiet-node quiet-node added this to the 0.59.0 milestone Nov 5, 2024
@quiet-node quiet-node self-assigned this Nov 5, 2024
Copy link

sonarcloud bot commented Nov 5, 2024

Copy link

github-actions bot commented Nov 5, 2024

Test Results

 21 files   -   1  266 suites   - 37   38m 18s ⏱️ + 3m 35s
604 tests  -   2  596 ✅ +  2  4 💤 ±0  4 ❌  - 4 
664 runs   - 160  656 ✅  - 155  4 💤 ±0  4 ❌  - 5 

For more details on these failures, see this check.

Results for commit 119b853. ± Comparison against base commit 22bc051.

♻️ This comment has been updated with latest results.

@quiet-node quiet-node changed the title fix: added null check for nullable values in formatContractResult() chore: cherry picked #3120 to release/0.59 Nov 5, 2024
@quiet-node quiet-node merged commit b375f72 into release/0.59 Nov 5, 2024
49 of 52 checks passed
@quiet-node quiet-node deleted the cherry-pick-3210 branch November 5, 2024 04:55
Copy link

codecov bot commented Nov 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.11%. Comparing base (22bc051) to head (119b853).
Report is 1 commits behind head on release/0.59.

Additional details and impacted files
@@               Coverage Diff                @@
##           release/0.59    #3211      +/-   ##
================================================
- Coverage         84.39%   83.11%   -1.28%     
================================================
  Files                69       66       -3     
  Lines              4473     4312     -161     
  Branches            881      843      -38     
================================================
- Hits               3775     3584     -191     
- Misses              431      485      +54     
+ Partials            267      243      -24     
Flag Coverage Δ
config-service 98.14% <ø> (ø)
relay 85.23% <ø> (-0.06%) ⬇️
server 83.52% <ø> (ø)
ws-server 36.87% <ø> (ø)

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

see 16 files with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
process Build, test and deployment-process related tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants