Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

nidi3/code-assert

Repository files navigation

code-assert

Build Status codecov License Maven Central

Assert that the source code of a project satisfies certain rules.

Nobody follows rules that are not checked. If they are only checked periodically / manually by an "architect", it's often too late because there are already too many violations.
A better way is to define coding rules in JUnit tests. This way, they are asserted automatically and regularly. Violations of rules break the build and therefore, one is forced to either adjust the code to comply with the rules or to adapt the rules in a reasonable way.

code-assert supports rules on the package structure and the test coverage. It also integrates several static code analysis tools.

Language independent checks

Java checks

Kotlin checks

Other


Dependency

This is based on code from JDepend. It checks if the package structure contains cycles and/or follows the defined rules.

public class DependencyTest {
 // Analyze all sources in src/main/java
 private final AnalyzerConfig config = AnalyzerConfig.maven().main();
 @Test
 public void noCycles() {
 assertThat(new DependencyAnalyzer(config).analyze(), hasNoCycles());
 }
 @Test
 public void dependency() {
 // Defines the dependency rules for package org.proj
 class OrgProj extends DependencyRuler {
 // Rules for org.proj.dep, org.proj.model, org.proj.util
 DependencyRule dep, model, util;
 @Override
 public void defineRules() {
 base().mayUse(util, dep.allSubOf()); //org.proj may use org.proj.util and all subpackages of org.proj.dep
 dep.andAllSub().mustUse(model); //org.proj.dep and all subpackages thereof must use org.proj.model
 model.mayUse(util).mustNotUse(base()); //org.proj.model may use org.proj.util but not org.proj
 }
 }
 // All dependencies are forbidden, except the ones defined in OrgProj
 // java, org, net packages may be used freely
 DependencyRules rules = DependencyRules.denyAll()
 .withRelativeRules(new OrgProj())
 .withExternals("java.*", "org.*", "net.*");
 DependencyResult result = new DependencyAnalyzer(config).rules(rules).analyze();
 assertThat(result, matchesRulesExactly());
 }
}

Test coverage

Maven Central

To verify the test coverage of a project, JaCoCo can be used. The following steps are needed:

  • Add this to the <build><plugins> section of pom.xml:
<plugin>
 <groupId>guru.nidi</groupId>
 <artifactId>code-assert-maven-plugin</artifactId>
 <version>0.9.14</version>
 <executions>
 <execution>
 <goals>
 <goal>prepare</goal>
 <goal>assert</goal>
 </goals>
 </execution>
 </executions>
</plugin>
  • prepare sets up the surefire plugin to run the tests with the JaCoCo agent which collects coverage data.
  • assert generates a coverage report and runs a coverage test (default is src/test/java/CodeCoverage.java, configurable through the testClass property).
  • Write a code coverage test:
public class CodeCoverage {
 @Test
 public void coverage() {
 // Coverage of branches must be at least 70%, lines 80% and methods 90%
 // This is checked globally and for all packages except for entities.
 JacocoAnalyzer analyzer = new JacocoAnalyzer(new CoverageCollector(BRANCH, LINE, METHOD)
 .just(For.global().setMinima(70, 80, 90))
 .just(For.allPackages().setMinima(70, 80, 90))
 .just(For.thePackage("org.proj.entity.*").setNoMinima()));
 assertThat(analyzer.analyze(), hasEnoughCoverage());
 }
}

FindBugs

Runs FindBugs on the code and finds questionable constructs.

public class FindBugsTest {
 @Test
 public void findBugs() {
 // Analyze all sources in src/main/java
 AnalyzerConfig config = AnalyzerConfig.maven().main();
 // Only treat bugs with rank < 17 and with NORMAL_PRIORITY or higher
 // Ignore the given bug types in the given classes / methods.
 BugCollector collector = new BugCollector().maxRank(17).minPriority(Priorities.NORMAL_PRIORITY)
 .just(In.everywhere().ignore("UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR"))
 .because("It's checked and OK like this",
 In.classes(DependencyRules.class, PmdRuleset.class).ignore("DP_DO_INSIDE_DO_PRIVILEGED"),
 In.classes("*Test", "Rulesets")
 .and(In.classes("ClassFileParser").withMethods("doParse"))
 .ignore("URF_UNREAD_FIELD"));
 FindBugsResult result = new FindBugsAnalyzer(config, collector).analyze();
 assertThat(result, hasNoBugs());
 }
}

Checkstyle

Runs checkstyle on the code and finds questionable constructs.

public class CheckstyleTest {
 @Test
 public void checkstyle() {
 // Analyze all sources in src/main/java
 AnalyzerConfig config = AnalyzerConfig.maven().main();
 // Only treat issues with severity WARNING or higher
 StyleEventCollector collector = new StyleEventCollector().severity(SeverityLevel.WARNING)
 .just(In.everywhere().ignore("import.avoidStar", "javadoc.missing"))
 .because("in tests, long lines are ok", In.classes("*Test").ignore("maxLineLen"));
 //use google checks, but adjust max line length
 final StyleChecks checks = StyleChecks.google().maxLineLen(120);
 CheckstyleResult result = new CheckstyleAnalyzer(config, checks, collector).analyze();
 assertThat(result, hasNoCheckstyleIssues());
 }
}

PMD

Runs PMD on the code and finds questionable constructs and code duplications.

public class PmdTest {
 // Analyze all sources in src/main/java
 private final AnalyzerConfig config = AnalyzerConfig.maven().main();
 @Test
 public void pmd() {
 // Only treat violations with MEDIUM priority or higher
 // Ignore the given violations in the given classes / methods
 PmdViolationCollector collector = new PmdViolationCollector().minPriority(RulePriority.MEDIUM)
 .because("It's not severe and occurs very often",
 In.everywhere().ignore("MethodArgumentCouldBeFinal"),
 In.locs("JavaClassBuilder#from", "FindBugsMatchers").ignore("AvoidInstantiatingObjectsInLoops"))
 .because("it'a an enum",
 In.classes("SignatureParser").ignore("SwitchStmtsShouldHaveDefault"))
 .just(In.classes("*Test").ignore("TooManyStaticImports"));
 // Define and configure the rule sets to be used
 PmdAnalyzer analyzer = new PmdAnalyzer(config, collector).withRulesets(
 basic(), braces(), design(), empty(), optimizations(),
 codesize().excessiveMethodLength(40).tooManyMethods(30));
 assertThat(analyzer.analyze(), hasNoPmdViolations());
 }
 @Test
 public void cpd() {
 // Ignore duplications in the given classes
 CpdMatchCollector collector = new CpdMatchCollector()
 .because("equals",
 In.everywhere().ignore("public boolean equals(Object o) {"))
 .just(
 In.classes(DependencyRule.class, Dependencies.class).ignoreAll(),
 In.classes("SignatureParser").ignoreAll());
 // Only treat duplications with at least 20 tokens
 CpdAnalyzer analyzer = new CpdAnalyzer(config, 20, collector);
 assertThat(analyzer.analyze(), hasNoCodeDuplications());
 }
}

ktlint

Runs ktlint, a kotlin linter.

public class KtlintTest {
 @Test
 public void analyze() {
 // Analyze all sources in src/main/kotlin
 AnalyzerConfig config = AnalyzerConfig.maven(KOTLIN).main();
 KtlintCollector collector = new KtlintCollector()
 .just(In.classes("Linker").ignore("no-semi"));
 KtlintResult result = new KtlintAnalyzer(config, collector).analyze();
 assertThat(result, hasNoKtlintIssues());
 }
}

detekt

Runs detekt, a static code analysis tool for kotlin.

public class DetektTest {
 @Test
 public void analyze() {
 // Analyze all sources in src/main/kotlin
 AnalyzerConfig config = AnalyzerConfig.maven(KOTLIN).main();
 DetektCollector collector = new DetektCollector()
 .just(In.classes("Linker").ignore("MaxLineLength"));
 DetektResult result = new DetektAnalyzer(config, collector).analyze();
 assertThat(result, hasNoDetektIssues());
 }
}

Configuration reuse

Collector configurations can be defined separately and thus reused. Some configurations are defined in PredefConfig.

private final CollectorTemplate<Ignore> pmdTestCollector = CollectorTemplate.forA(PmdViolationCollector.class)
 .because("It's a test", In.classes("*Test")
 .ignore("JUnitSpelling", "AvoidDuplicateLiterals", "SignatureDeclareThrowsException"))
 .because("It's compiler generated code", In.languages(KOTLIN)
 .ignore("BC_BAD_CAST_TO_ABSTRACT_COLLECTION"));
@Test
public void pmd() {
 PmdViolationCollector collector = new PmdViolationCollector().minPriority(RulePriority.MEDIUM)
 .apply(pmdTestCollector)
 .because("It's not severe and occurs often", In.everywhere().ignore("MethodArgumentCouldBeFinal"));
 PmdAnalyzer analyzer = new PmdAnalyzer(config, collector).withRulesets(rules);
 assertThat(analyzer.analyze(), hasNoPmdViolations());
}

Standard tests

A test can inherit from CodeAssertTest. It should override one or more analyzeXXX methods. If it does so, these standard checks will be executed:

  • dependency rules
  • circular dependencies
  • PMD
  • PMD - unused actions
  • CPD
  • CPD - unused actions
  • FindBugs
  • FindBugs - unused actions
  • Checkstyle
  • Checkstyle - unused actions
//extend CodeAssertTest if you still use JUnit 4
public class CodeTest extends CodeAssertJunit5Test {
 private static final AnalyzerConfig CONFIG = AnalyzerConfig.maven().main();
 @Override
 protected DependencyResult analyzeDependencies() {
 class MyProject extends DependencyRuler {
 DependencyRule packages;
 @Override
 public void defineRules() {
 //TODO
 }
 }
 final DependencyRules rules = denyAll().withExternals("java.*").withRelativeRules(new MyProject());
 return new DependencyAnalyzer(CONFIG).rules(rules).analyze();
 }
 @Override
 protected FindBugsResult analyzeFindBugs() {
 final BugCollector bugCollector = new BugCollector().just(
 In.classes("*Exception").ignore("SE_BAD_FIELD"));
 return new FindBugsAnalyzer(CONFIG, bugCollector).analyze();
 }
 @Override
 protected CheckstyleResult analyzeCheckstyle() {
 final StyleEventCollector bugCollector = new StyleEventCollector().just(
 In.everywhere().ignore("javadoc.missing"));
 return new CheckstyleAnalyzer(CONFIG, StyleChecks.google(), bugCollector).analyze();
 }
 @Override
 protected PmdResult analyzePmd() {
 final PmdViolationCollector collector = new PmdViolationCollector().just(
 In.everywhere().ignore("MethodArgumentCouldBeFinal"));
 return new PmdAnalyzer(CONFIG, collector).withRulesets(basic(), braces()).analyze();
 }
}

About

Assert that the java code of a project satisfies certain checks.

Topics

Resources

License

Stars

Watchers

Forks

Sponsor this project

Packages

No packages published

Contributors 4

Languages

AltStyle によって変換されたページ (->オリジナル) /