Skip to content

Conversation

@peterlimg
Copy link
Contributor

Fixes

Tests

Tasks to complete before merging PR:

  • Ensure system tests are passing. If not Run them manually to check for any regressions 📋
  • Do any new system tests need added to test this change? do any existing system tests need updated? If so create a PR at 0chain/system_test
  • Merge your system tests PR to master AFTER merging this PR
@codecov-commenter
Copy link

codecov-commenter commented Jun 19, 2022

Codecov Report

Merging #733 (b8d5e21) into staging (cbe3d88) will increase coverage by 0.06%.
The diff coverage is n/a.

@@ Coverage Diff @@ ## staging #733 +/- ## =========================================== + Coverage 24.13% 24.19% +0.06%  =========================================== Files 69 69 Lines 7898 7898 =========================================== + Hits 1906 1911 +5  + Misses 5708 5703 -5  Partials 284 284 
Flag Coverage Δ
Unit-Tests 24.19% <ø> (+0.06%) ⬆️

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

Impacted Files Coverage Δ
code/go/0chain.net/core/lock/lock.go 100.00% <0.00%> (+13.51%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cbe3d88...b8d5e21. Read the comment docs.

@service-0chain
Copy link
Collaborator

Manual system tests [success] with the following config

config value
tests skipped FALSE
system-tests fix/getUsersPool
gosdk NOT OVERRIDDEN
zboxcli fix/getUsersPool
zwalletcli fix/getUsersPool
0chain fix/getUserPools
blobber fix/getUsersPool
authorizer staging
0box staging
0block staging
0dns staging
explorer master
0proxy staging
0search staging
@service-0chain
Copy link
Collaborator

Manual system tests [success] with the following config

config value
tests skipped FALSE
system-tests fix/getUsersPool
gosdk fix/getUsersPool
zboxcli fix/getUsersPool
zwalletcli fix/getUsersPool
0chain fix/getUserPools
blobber fix/getUsersPool
authorizer staging
0box staging
0block staging
0dns staging
explorer staging
0proxy staging
0search staging
@service-0chain
Copy link
Collaborator

Manual system tests [failure] with the following config

config value
tests skipped FALSE
system-tests fix/getUsersPool
gosdk fix/getUsersPool
zboxcli fix/getUsersPool
zwalletcli fix/getUsersPool
0chain fix/getUserPools
blobber fix/getUsersPool
authorizer staging
0box staging
0block staging
0dns staging
explorer staging
0proxy staging
0search staging
Copy link
Contributor

@dabasov dabasov left a comment

Choose a reason for hiding this comment

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

lgtm

@service-0chain
Copy link
Collaborator

Manual system tests [failure] with the following config

config value
tests skipped FALSE
system-tests fix/getUsersPool
gosdk NOT OVERRIDDEN
zboxcli fix/getUsersPool
zwalletcli fix/getUsersPool
0chain fix/getUserPools
blobber fix/getUsersPool
authorizer staging
0box staging
0block staging
0dns staging
explorer master
0proxy staging
0search staging
@peterlimg peterlimg merged commit 8ef0832 into staging Jun 22, 2022
@cnlangzi cnlangzi deleted the fix/getUsersPool branch January 29, 2023 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants