Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rule 6.1.6 abstract classes should have at least one abstract method #457

Merged
merged 6 commits into from
Oct 26, 2020

Conversation

aktsay6
Copy link
Collaborator

@aktsay6 aktsay6 commented Oct 26, 2020

What's done:

  • Rule logic made
  • Added fix and warn tests

This pull request closes #442

Actions checklist

  • Implemented Rule, added Warnings
  • Added tests on checks
  • Added tests on fixers
  • Updated diktat-analysis.yml
  • Updated available-rules.md

### What's done:
  * Rule logic made
  * Added fix and warn tests
@aktsay6 aktsay6 added the enhancement New feature or request label Oct 26, 2020
@aktsay6 aktsay6 requested review from petertrr and kentr0w October 26, 2020 10:59
@codecov
Copy link

codecov bot commented Oct 26, 2020

Codecov Report

Merging #457 into master will decrease coverage by 0.00%.
The diff coverage is 80.76%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #457      +/-   ##
============================================
- Coverage     81.95%   81.94%   -0.01%     
- Complexity     1435     1445      +10     
============================================
  Files            69       70       +1     
  Lines          3591     3617      +26     
  Branches       1147     1155       +8     
============================================
+ Hits           2943     2964      +21     
- Misses          201      202       +1     
- Partials        447      451       +4     
Flag Coverage Δ Complexity Δ
#unittests 81.94% <80.76%> (-0.01%) 1445.00 <10.00> (+10.00) ⬇️

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

Impacted Files Coverage Δ Complexity Δ
...at-rules/src/main/kotlin/generated/WarningNames.kt 0.00% <ø> (ø) 0.00 <0.00> (ø)
...iktat/ruleset/rules/classes/AbstractClassesRule.kt 79.16% <79.16%> (ø) 10.00 <10.00> (?)
...tlin/org/cqfn/diktat/ruleset/constants/Warnings.kt 97.27% <100.00%> (+0.02%) 9.00 <0.00> (ø)
...cqfn/diktat/ruleset/rules/DiktatRuleSetProvider.kt 96.92% <100.00%> (+0.04%) 8.00 <0.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 e271109...0b8caa2. Read the comment docs.

### What's done:
  * Fixed bugs
Copy link
Member

@petertrr petertrr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add test with other class modifiers

### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
@aktsay6 aktsay6 requested a review from petertrr October 26, 2020 15:03
Copy link
Member

@petertrr petertrr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@aktsay6 aktsay6 merged commit f28dfb9 into master Oct 26, 2020
@aktsay6 aktsay6 deleted the feature/rule-6.1.6-abstract-classes branch October 26, 2020 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rule 6.1.6: Abstract class should have at least one abstract method
2 participants