Both branches of if-then-else are identical in pkg/ruler/ruler_test.go

[chavacava]

Both branches of if-then-else are identical in

cortex/pkg/ruler/ruler_test.go

Lines 1073 to 1077 in 275a5bf

	if tc.shardingStrategy == util.ShardingStrategyShuffle {
	require.Equal(t, int32(tc.expectedClientCallCount), mockPoolClient.numberOfCalls.Load())
	} else {
	require.Equal(t, int32(tc.expectedClientCallCount), mockPoolClient.numberOfCalls.Load())
	}

Found with revive

[rahulmansharamani14]

Hi @chavacava, Is there anyone working on this? If not, I'd like to give it a try.

[chavacava]

Hi @rahulmansharamani14 , I don't know if there is someone already working on the issue. I just reported the issue.

[friedrichg]

it's yours @rahulmansharamani14