Skip to content

Commit dc9d4c4

Browse files
authored
Merge pull request #1384 from 0chain/hotfix/challenges
Fix challenge success
2 parents d1f396d + 9532164 commit dc9d4c4

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

.github/workflows/system_tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ on:
1313
required: true
1414
run_smoke_tests:
1515
description: 'Run subset of system tests (smoke tests) for faster feedback (NOT FOR PRS POINTED TO STAGING)'
16-
default: 'false'
16+
default: 'true'
1717
required: false
1818
test_file_filter:
1919
description: 'Comma separated list of test files to run (eg. zwalletcli_register_wallet_test.go, zwalletcli_send_and_balance_test.go). If supplied, the PR will NOT be notified of the test result'

code/go/0chain.net/blobbercore/challenge/protocol.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,12 @@ func (cr *ChallengeEntity) LoadValidationTickets(ctx context.Context) error {
303303
}
304304
updateMapAndSlice(validatorID, i, &validationTicket)
305305

306-
numSuccess++
306+
if validationTicket.Result {
307+
numSuccess++
308+
} else {
309+
numFailed++
310+
}
311+
307312
}(url, validator.ID, i)
308313
}
309314

0 commit comments

Comments
 (0)