-
Notifications
You must be signed in to change notification settings - Fork 39
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
Smoke tests in diktat #1534
Smoke tests in diktat #1534
Conversation
### What's done: * moved smoke tests to diktat-ruleset
diktat-ruleset/src/test/kotlin/org/cqfn/diktat/ruleset/smoke/DiktatSmokeTestBase.kt
Fixed
Show fixed
Hide fixed
Codecov Report
@@ Coverage Diff @@
## master #1534 +/- ##
============================================
- Coverage 82.98% 82.09% -0.89%
+ Complexity 2363 2306 -57
============================================
Files 112 112
Lines 7777 7793 +16
Branches 2011 2011
============================================
- Hits 6454 6398 -56
- Misses 442 481 +39
- Partials 881 914 +33
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ktlint found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
diktat-ruleset/src/test/kotlin/org/cqfn/diktat/ruleset/smoke/DiktatSmokeTest.kt
Fixed
Show fixed
Hide fixed
diktat-ruleset/src/test/kotlin/org/cqfn/diktat/ruleset/smoke/DiktatSmokeTest.kt
Fixed
Show fixed
Hide fixed
diktat-test-framework/src/main/kotlin/org/cqfn/diktat/test/framework/util/TestUtils.kt
Fixed
Show fixed
Hide fixed
* Bugfix for fix mode of AVOID_NULL_CHECK * Added a check for assignment operator when deciding if to put run block or not * Added tests (#1293)
diktat-ruleset/src/test/kotlin/org/cqfn/diktat/ruleset/smoke/DiktatSaveSmokeTest.kt
Outdated
Show resolved
Hide resolved
@nulls, I repeatedly receive the following exception when trying to build your branch locally:
The above is a Windows build. I also receive the same error when buildsing using a local TeamCity instance (WSL/Linux): Let's investigate that, too, before you merge. |
What's done:
it's part of #1518