hyperledger-labs / fabric-smart-client / 35720627571

91%

Build Type

push

github

Committed by web-flow

Commit Message

fix(fabric): initialize per-channel map in AddChannelProcessor (#1868)

processorManager.AddChannelProcessor wrote into
channelProcessors[channel][ns] without creating the inner map.
NewProcessorManager initializes only the outer map, so the first call for
any channel panicked with "assignment to entry in nil map". The method is
reachable through the exported platform/fabric.ProcessorManager wrapper,
which implements driver.ProcessorManager.

Create the per-channel map on first use for a channel.

The unit test previously seeded channelProcessors["ch1"] by hand, which
masked the panic. It now calls AddChannelProcessor directly and adds a
second namespace on the same channel to confirm the existing entries
survive.

Fixes #1867

Signed-off-by: Said Altury <Said.Altury@ibm.com>

Coverage Stats

3 of 3 new or added lines in 1 file covered. (100.0%)

15 existing lines in 7 files now uncovered.

27516 of 31963 relevant lines covered (86.09%)

2346.43 hits per line