Fix dataclump analysis for clumps in non consecutive methods

[le-santos]

This PR fixes issue #1798.

What changes?

1. DataClump detector now creates all combinations between methods to analyse for possible clumps, instead of grouping only the consecutive methods.

def a(x,y); end
def b(x,y); end
def c(z)  ; end
def d(x,y); end

# Before
# candidate_methods.each_cons(3)
# => [[a, b, c], [b, c, d]]

# Now
# candidate_methods.combination(3)
# => [[a, b, c], [a, b, d], [a, c, d], [b, c, d]]

2. After this fix, the codebase quality tests pointed a new DataClump smell in Reek::AST::Node class. For that reason, it was necessary to adjust some params in protected methods of this class.

[leandronsp]

superb!

[troessner]

Very nice @le-santos! @mvz PR looks good to me, wdyt about merging this one and releasing a new version?

[mvz]

@troessner yes, looks good to me too!

[troessner]

Releasing 6.4 now, thanks @le-santos !