Fix early returning iterator

[danwt]

Problem

interchain-security/x/ccv/provider/keeper/relay.go

Lines 356 to 357 in c8a762c

	// continue to iterate through all consumers
	return true

I think it should return false, rather than true.

interchain-security/x/ccv/provider/keeper/keeper.go

Lines 200 to 202 in c8a762c

	if cb(ctx, channelID, chainID) {
	break
	}

Closing criteria

Fix the iteration if necessary, and add a regression test

TODOs

• Labels have been added for issue
• Issue has been added to the ICS project

[mpoke]

Nice catch @danwt. IMO the fix entails changing the if condition to

if !cb(ctx, channelID, chainID)

in order to be consistent with the other iterators.

[mpoke]

Could be worthwhile to also create a UT for EndBlockCCR to catch these types of errors. The e2e tests have only one consumer chain.