Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@sowmyav24
Copy link
Contributor

Fixes #3177

  • Added check to ensure Pair is used within the callExpression before reporting
  • Added test to create pair using Pair and to within a function

@codecov
Copy link

codecov bot commented Oct 27, 2020

Codecov Report

Merging #3178 into master will decrease coverage by 0.01%.
The diff coverage is 25.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3178      +/-   ##
============================================
- Coverage     79.58%   79.56%   -0.02%     
- Complexity     2610     2613       +3     
============================================
  Files           439      439              
  Lines          7935     7938       +3     
  Branches       1511     1512       +1     
============================================
+ Hits           6315     6316       +1     
  Misses          820      820              
- Partials        800      802       +2     
Impacted Files Coverage Δ Complexity Δ
...ekt/rules/style/optional/PreferToOverPairSyntax.kt 70.58% <25.00%> (-7.99%) 8.00 <2.00> (+3.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 b5a423f...f35f079. Read the comment docs.

@sowmyav24
Copy link
Contributor Author

The code coverage for the class, although it fails in CI, when run locally gives 100%
coverage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

False Positive PreferToOverPairSyntax

5 participants