Fix: return attestation err result to dispatcher #1011

ian-shim · 2024-12-17T00:20:57Z

Why are these changes needed?

Makes sure sigChan receives exactly len(operators) results.
Expand logging.

I've made sure the tests are passing. Note that there might be a few flaky tests, in that case, please comment that they are not relevant.
I've checked the new test coverage and the coverage percentage didn't drop.
Testing Strategy
- Unit tests
- Integration tests
- This PR is not tested :(

disperser/controller/dispatcher.go

ian-shim requested review from dmanc and cody-littley December 17, 2024 00:20

ian-shim force-pushed the dispatcher-return-err-to-sig-chan branch from e1b1b0f to 46955b1 Compare December 17, 2024 00:22

dmanc reviewed Dec 17, 2024

View reviewed changes

disperser/controller/dispatcher.go Outdated Show resolved Hide resolved

ian-shim force-pushed the dispatcher-return-err-to-sig-chan branch 6 times, most recently from ebde2d5 to 39e1552 Compare December 17, 2024 01:27

return attestation err result to dispatcher

d394332

ian-shim force-pushed the dispatcher-return-err-to-sig-chan branch from 39e1552 to d394332 Compare December 17, 2024 01:42

ian-shim marked this pull request as ready for review December 17, 2024 01:57

dmanc approved these changes Dec 17, 2024

View reviewed changes

ian-shim merged commit fc5dff1 into Layr-Labs:master Dec 17, 2024
8 checks passed