Fixing bug with lineNumber in autocorrect mode

[petertrr]

What's done:

• Use getLineNumber everywhere
• Added test

This pull request closes #371

[codecov]

Codecov Report

Merging #400 into master will decrease coverage by 0.01%.
The diff coverage is 86.02%.

@@             Coverage Diff              @@
##             master     #400      +/-   ##
============================================
- Coverage     82.50%   82.49%   -0.02%     
- Complexity     1346     1347       +1     
============================================
  Files            65       65              
  Lines          3298     3313      +15     
  Branches       1053     1056       +3     
============================================
+ Hits           2721     2733      +12     
- Misses          178      179       +1     
- Partials        399      401       +2     

Flag	Coverage Δ	Complexity Δ
#unittests	82.49% <86.02%> (-0.02%)	1347.00 <4.00> (+1.00)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ	Complexity Δ
...tlin/org/cqfn/diktat/ruleset/utils/AstNodeUtils.kt	82.07% <84.33%> (-0.16%)	0.00 <0.00> (ø)
...at/ruleset/rules/identifiers/LocalVariablesRule.kt	91.66% <100.00%> (ø)	24.00 <4.00> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 136500f...c2f147c. Read the comment docs.

[orchestr7]

I thought we discussed it and decided that ktlint calculates everything correctly.
So why it is needed?

[petertrr]

We discussed it regarding warning messages, but we also use line number in some rules (e.g., method ASTNode.isBefore), and when AST has already been mutated by previous rules, it leads to ArrayIndexOutOfBoundsException (see #371).

[aktsay6]

lgtm