Compare commits
47 commits
Author | SHA1 | Date | |
---|---|---|---|
9ea433edce | |||
86ea747ba2 | |||
9e8f6c0497 | |||
a15218eb1d | |||
7ae1650677 | |||
d304b931f9 | |||
355cdf770a | |||
4feba23b90 | |||
91c9556bd9 | |||
67ce6546fe | |||
cfbeeeb5c7 | |||
fb0acda205 | |||
e28496ca16 | |||
a2d37055c6 | |||
dca8baf3ad | |||
af25b2f507 | |||
25bce33323 | |||
e7fac3130b | |||
a1d0b30968 | |||
bc60dcbcf8 | |||
71253717bf | |||
28f9d29d9a | |||
682c50e949 | |||
6882490430 | |||
9184979e2f | |||
e7b366eaf0 | |||
dfdc311607 | |||
586e4a0e19 | |||
21edab586b | |||
f58d4ac052 | |||
c7e5731d9e | |||
4af11d862b | |||
5c33fd2a15 | |||
f46d75c2fe | |||
592c0c514d | |||
3b0550add2 | |||
22b7ab9220 | |||
a986add8ee | |||
d20f3e41b8 | |||
47c0b14ce9 | |||
176929cd29 | |||
3e18345efe | |||
2b6b4a5e82 | |||
e63c1a636e | |||
fec530d26e | |||
9cb3191dd4 | |||
a253342bd6 |
16 changed files with 608 additions and 135 deletions
19
.github/workflows/bld.yml
vendored
19
.github/workflows/bld.yml
vendored
|
@ -1,14 +1,16 @@
|
|||
name: bld-ci
|
||||
|
||||
on: [push, pull_request, workflow_dispatch]
|
||||
on: [ push, pull_request, workflow_dispatch ]
|
||||
|
||||
jobs:
|
||||
build-bld-project:
|
||||
runs-on: ubuntu-latest
|
||||
|
||||
strategy:
|
||||
matrix:
|
||||
java-version: [17, 21, 22]
|
||||
java-version: [ 17, 21, 24 ]
|
||||
kotlin-version: [ 1.9.25, 2.0.21, 2.1.20 ]
|
||||
os: [ ubuntu-latest, windows-latest, macos-latest ]
|
||||
|
||||
runs-on: ${{ matrix.os }}
|
||||
|
||||
steps:
|
||||
- name: Checkout source repository
|
||||
|
@ -22,11 +24,8 @@ jobs:
|
|||
distribution: "zulu"
|
||||
java-version: ${{ matrix.java-version }}
|
||||
|
||||
- name: Grant execute permission for bld
|
||||
run: chmod +x bld
|
||||
|
||||
- name: Download the dependencies
|
||||
- name: Download dependencies
|
||||
run: ./bld download
|
||||
|
||||
- name: Run tests with bld
|
||||
run: ./bld compile test
|
||||
- name: Run tests
|
||||
run: ./bld compile test
|
4
.github/workflows/pages.yml
vendored
4
.github/workflows/pages.yml
vendored
|
@ -47,11 +47,11 @@ jobs:
|
|||
uses: actions/configure-pages@v3
|
||||
|
||||
- name: Upload artifact
|
||||
uses: actions/upload-pages-artifact@v1
|
||||
uses: actions/upload-pages-artifact@v3
|
||||
with:
|
||||
# Upload generated Javadocs repository
|
||||
path: "build/javadoc/"
|
||||
|
||||
- name: Deploy to GitHub Pages
|
||||
id: deployment
|
||||
uses: actions/deploy-pages@v1
|
||||
uses: actions/deploy-pages@v4
|
||||
|
|
6
.idea/bld.xml
generated
Normal file
6
.idea/bld.xml
generated
Normal file
|
@ -0,0 +1,6 @@
|
|||
<?xml version="1.0" encoding="UTF-8"?>
|
||||
<project version="4">
|
||||
<component name="BldConfiguration">
|
||||
<events />
|
||||
</component>
|
||||
</project>
|
13
.idea/icon.svg
generated
Normal file
13
.idea/icon.svg
generated
Normal file
|
@ -0,0 +1,13 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd">
|
||||
<svg width="100%" height="100%" viewBox="0 0 179 108" version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" xml:space="preserve" xmlns:serif="http://www.serif.com/" style="fill-rule:evenodd;clip-rule:evenodd;stroke-linejoin:round;stroke-miterlimit:2;">
|
||||
<g transform="matrix(1,0,0,1,-210.511,-96.3382)">
|
||||
<g transform="matrix(1,0,0,1,-23.3386,-649.816)">
|
||||
<g transform="matrix(0.221288,0,0,0.24,73.9536,390.254)">
|
||||
<path d="M722.568,1482.92L722.568,1921.1L808.968,1921.1L808.968,1888.7C822.168,1907.9 846.168,1930.1 893.568,1930.1C933.168,1930.1 961.968,1917.5 985.368,1893.5C1012.97,1865.9 1027.37,1827.5 1027.37,1786.1C1027.37,1741.7 1011.17,1705.1 985.368,1680.5C961.968,1658.3 928.968,1644.5 892.368,1644.5C862.968,1644.5 830.568,1654.1 808.968,1683.5L808.968,1482.92L722.568,1482.92ZM871.368,1718.9C888.768,1718.9 903.768,1723.7 917.568,1736.9C930.168,1748.9 938.568,1766.3 938.568,1787.9C938.568,1807.7 930.168,1825.1 917.568,1837.1C904.368,1849.7 887.568,1855.7 872.568,1855.7C856.368,1855.7 837.168,1849.1 823.368,1835.9C813.168,1826.3 803.568,1810.1 803.568,1787.9C803.568,1765.1 812.568,1749.5 822.768,1738.7C836.568,1724.3 852.768,1718.9 871.368,1718.9Z" style="fill:rgb(35,146,255);fill-rule:nonzero;"/>
|
||||
<rect x="1083.77" y="1482.92" width="86.4" height="438.182" style="fill:rgb(250,144,82);fill-rule:nonzero;"/>
|
||||
<path d="M1531.37,1482.92L1444.97,1482.92L1444.97,1683.5C1423.37,1654.1 1390.97,1644.5 1361.57,1644.5C1324.97,1644.5 1291.97,1658.3 1268.57,1680.5C1242.77,1705.1 1226.57,1741.7 1226.57,1786.1C1226.57,1827.5 1240.97,1865.9 1268.57,1893.5C1291.97,1917.5 1320.77,1930.1 1360.37,1930.1C1407.77,1930.1 1431.77,1907.9 1444.97,1888.7L1444.97,1921.1L1531.37,1921.1L1531.37,1482.92ZM1382.57,1718.9C1401.17,1718.9 1417.37,1724.3 1431.17,1738.7C1441.37,1749.5 1450.37,1765.1 1450.37,1787.9C1450.37,1810.1 1440.77,1826.3 1430.57,1835.9C1416.77,1849.1 1397.57,1855.7 1381.37,1855.7C1366.37,1855.7 1349.57,1849.7 1336.37,1837.1C1323.77,1825.1 1315.37,1807.7 1315.37,1787.9C1315.37,1766.3 1323.77,1748.9 1336.37,1736.9C1350.17,1723.7 1365.17,1718.9 1382.57,1718.9Z" style="fill:rgb(35,146,255);fill-rule:nonzero;"/>
|
||||
</g>
|
||||
</g>
|
||||
</g>
|
||||
</svg>
|
After Width: | Height: | Size: 2.4 KiB |
4
.idea/libraries/bld.xml
generated
4
.idea/libraries/bld.xml
generated
|
@ -2,12 +2,12 @@
|
|||
<library name="bld">
|
||||
<CLASSES>
|
||||
<root url="file://$PROJECT_DIR$/lib/bld" />
|
||||
<root url="jar://$USER_HOME$/.bld/dist/bld-1.9.1.jar!/" />
|
||||
<root url="jar://$USER_HOME$/.bld/dist/bld-2.2.1.jar!/" />
|
||||
</CLASSES>
|
||||
<JAVADOC />
|
||||
<SOURCES>
|
||||
<root url="file://$PROJECT_DIR$/lib/bld" />
|
||||
<root url="jar://$USER_HOME$/.bld/dist/bld-1.9.1-sources.jar!/" />
|
||||
<root url="jar://$USER_HOME$/.bld/dist/bld-2.2.1-sources.jar!/" />
|
||||
</SOURCES>
|
||||
<excluded>
|
||||
<root url="jar://$PROJECT_DIR$/lib/bld/bld-wrapper.jar!/" />
|
||||
|
|
4
.idea/libraries/compile.xml
generated
4
.idea/libraries/compile.xml
generated
|
@ -7,7 +7,7 @@
|
|||
<SOURCES>
|
||||
<root url="file://$PROJECT_DIR$/lib/compile" />
|
||||
</SOURCES>
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/compile" recursive="false" />
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/compile" recursive="false" type="SOURCES" />
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/compile" recursive="true" />
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/compile" recursive="true" type="SOURCES" />
|
||||
</library>
|
||||
</component>
|
4
.idea/libraries/runtime.xml
generated
4
.idea/libraries/runtime.xml
generated
|
@ -7,7 +7,7 @@
|
|||
<SOURCES>
|
||||
<root url="file://$PROJECT_DIR$/lib/runtime" />
|
||||
</SOURCES>
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/runtime" recursive="false" />
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/runtime" recursive="false" type="SOURCES" />
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/runtime" recursive="true" />
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/runtime" recursive="true" type="SOURCES" />
|
||||
</library>
|
||||
</component>
|
4
.idea/libraries/test.xml
generated
4
.idea/libraries/test.xml
generated
|
@ -7,7 +7,7 @@
|
|||
<SOURCES>
|
||||
<root url="file://$PROJECT_DIR$/lib/test" />
|
||||
</SOURCES>
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/test" recursive="false" />
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/test" recursive="false" type="SOURCES" />
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/test" recursive="true" />
|
||||
<jarDirectory url="file://$PROJECT_DIR$/lib/test" recursive="true" type="SOURCES" />
|
||||
</library>
|
||||
</component>
|
2
.vscode/settings.json
vendored
2
.vscode/settings.json
vendored
|
@ -9,7 +9,7 @@
|
|||
],
|
||||
"java.configuration.updateBuildConfiguration": "automatic",
|
||||
"java.project.referencedLibraries": [
|
||||
"${HOME}/.bld/dist/bld-1.9.1.jar",
|
||||
"${HOME}/.bld/dist/bld-2.2.1.jar",
|
||||
"lib/**/*.jar"
|
||||
]
|
||||
}
|
||||
|
|
14
README.md
14
README.md
|
@ -2,14 +2,22 @@
|
|||
|
||||
[](https://opensource.org/licenses/Apache-2.0)
|
||||
[](https://www.oracle.com/java/technologies/javase/jdk17-archive-downloads.html)
|
||||
[](https://rife2.com/bld)
|
||||
[](https://rife2.com/bld)
|
||||
[](https://repo.rife2.com/#/releases/com/uwyn/rife2/bld-pmd)
|
||||
[](https://repo.rife2.com/#/snapshots/com/uwyn/rife2/bld-pmd)
|
||||
[](https://github.com/rife2/bld-pmd/actions/workflows/bld.yml)
|
||||
|
||||
To install, please refer to the [extensions documentation](https://github.com/rife2/bld/wiki/Extensions).
|
||||
To install the latest version, add the following to the `lib/bld/bld-wrapper.properties` file:
|
||||
|
||||
To check all source code using the [Java Quickstart](https://docs.pmd-code.org/latest/pmd_rules_java.html) configuration, add the following to your build file:
|
||||
```properties
|
||||
bld.extension-pmd=com.uwyn.rife2:bld-pmd
|
||||
```
|
||||
|
||||
For more information, please refer to the [extensions](https://github.com/rife2/bld/wiki/Extensions) documentation.
|
||||
|
||||
## Check Source with PMD
|
||||
|
||||
To check all source Codecode using the [Java Quickstart](https://docs.pmd-code.org/latest/pmd_rules_java.html) configuration, add the following to your build file:
|
||||
|
||||
```java
|
||||
@BuildCommand(summary = "Checks source code with PMD")
|
||||
|
|
|
@ -7,9 +7,9 @@
|
|||
<!-- BEST PRACTICES -->
|
||||
<rule ref="category/java/bestpractices.xml">
|
||||
<exclude name="AvoidPrintStackTrace"/>
|
||||
<exclude name="JUnit4TestShouldUseTestAnnotation"/>
|
||||
<exclude name="JUnitTestContainsTooManyAsserts"/>
|
||||
<exclude name="GuardLogStatement"/>
|
||||
<exclude name="UnitTestContainsTooManyAsserts"/>
|
||||
<exclude name="UnitTestShouldUseTestAnnotation"/>
|
||||
</rule>
|
||||
|
||||
<rule ref="category/java/bestpractices.xml/MissingOverride">
|
||||
|
|
Binary file not shown.
|
@ -2,4 +2,4 @@ bld.downloadExtensionJavadoc=false
|
|||
bld.downloadExtensionSources=true
|
||||
bld.downloadLocation=
|
||||
bld.repositories=MAVEN_LOCAL,MAVEN_CENTRAL,RIFE2_SNAPSHOTS,RIFE2_RELEASES
|
||||
bld.version=1.9.1
|
||||
bld.version=2.2.1
|
||||
|
|
|
@ -1,5 +1,5 @@
|
|||
/*
|
||||
* Copyright 2023-2024 the original author or authors.
|
||||
* Copyright 2023-2025 the original author or authors.
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
|
@ -23,8 +23,7 @@ import rife.bld.publish.PublishScm;
|
|||
|
||||
import java.util.List;
|
||||
|
||||
import static rife.bld.dependencies.Repository.MAVEN_CENTRAL;
|
||||
import static rife.bld.dependencies.Repository.RIFE2_RELEASES;
|
||||
import static rife.bld.dependencies.Repository.*;
|
||||
import static rife.bld.dependencies.Scope.*;
|
||||
import static rife.bld.operations.JavadocOptions.DocLinkOption.NO_MISSING;
|
||||
|
||||
|
@ -32,25 +31,25 @@ public class PmdOperationBuild extends Project {
|
|||
public PmdOperationBuild() {
|
||||
pkg = "rife.bld.extension";
|
||||
name = "bld-pmd";
|
||||
version = version(1, 1, 2);
|
||||
version = version(1, 2, 3);
|
||||
|
||||
javaRelease = 17;
|
||||
|
||||
downloadSources = true;
|
||||
autoDownloadPurge = true;
|
||||
repositories = List.of(MAVEN_CENTRAL, RIFE2_RELEASES);
|
||||
|
||||
repositories = List.of(MAVEN_LOCAL, MAVEN_CENTRAL, RIFE2_RELEASES, RIFE2_SNAPSHOTS);
|
||||
|
||||
var pmd = version(7, 3, 0);
|
||||
var pmd = version(7, 13, 0);
|
||||
scope(compile)
|
||||
.include(dependency("com.uwyn.rife2", "bld", version(1, 9, 1)))
|
||||
.include(dependency("com.uwyn.rife2", "bld", version(2, 2, 1)))
|
||||
.include(dependency("net.sourceforge.pmd", "pmd-java", pmd));
|
||||
scope(runtime)
|
||||
.include(dependency("net.sourceforge.pmd", "pmd-java", pmd))
|
||||
.include(dependency("org.slf4j", "slf4j-simple", version(2, 0, 13)));
|
||||
.include(dependency("org.slf4j", "slf4j-simple", version(2, 0, 17)));
|
||||
scope(test)
|
||||
.include(dependency("org.junit.jupiter", "junit-jupiter", version(5, 10, 3)))
|
||||
.include(dependency("org.junit.platform", "junit-platform-console-standalone", version(1, 10, 3)))
|
||||
.include(dependency("org.assertj", "assertj-core", version(3, 26, 0)));
|
||||
.include(dependency("org.junit.jupiter", "junit-jupiter", version(5, 12, 2)))
|
||||
.include(dependency("org.junit.platform", "junit-platform-console-standalone", version(1, 12, 2)))
|
||||
.include(dependency("org.assertj", "assertj-core", version(3, 27, 3)));
|
||||
|
||||
javadocOperation()
|
||||
.javadocOptions()
|
||||
|
@ -61,29 +60,26 @@ public class PmdOperationBuild extends Project {
|
|||
.link("https://javadoc.io/doc/net.sourceforge.pmd/pmd-core/latest/");
|
||||
|
||||
publishOperation()
|
||||
// .repository(MAVEN_LOCAL)
|
||||
.repository(version.isSnapshot() ? repository("rife2-snapshot") : repository("rife2"))
|
||||
.repository(repository("github"))
|
||||
.info()
|
||||
.groupId("com.uwyn.rife2")
|
||||
.artifactId("bld-pmd")
|
||||
.description("bld Extension to Perform Static Code Analysis with PMD")
|
||||
.url("https://github.com/rife2/bld-pmd")
|
||||
.developer(
|
||||
new PublishDeveloper()
|
||||
.id("ethauvin").name("Erik C. Thauvin")
|
||||
.email("erik@thauvin.net")
|
||||
.url("https://erik.thauvin.net/")
|
||||
.developer(new PublishDeveloper()
|
||||
.id("ethauvin").name("Erik C. Thauvin")
|
||||
.email("erik@thauvin.net")
|
||||
.url("https://erik.thauvin.net/")
|
||||
)
|
||||
.license(
|
||||
new PublishLicense()
|
||||
.name("The Apache License, Version 2.0")
|
||||
.url("https://www.apache.org/licenses/LICENSE-2.0.txt")
|
||||
.license(new PublishLicense()
|
||||
.name("The Apache License, Version 2.0")
|
||||
.url("https://www.apache.org/licenses/LICENSE-2.0.txt")
|
||||
)
|
||||
.scm(
|
||||
new PublishScm()
|
||||
.connection("scm:git:https://github.com/rife2/bld-pmd.git")
|
||||
.developerConnection("scm:git:git@github.com:rife2/bld-pmd.git")
|
||||
.url("https://github.com/rife2/bld-pmd")
|
||||
.scm(new PublishScm()
|
||||
.connection("scm:git:https://github.com/rife2/bld-pmd.git")
|
||||
.developerConnection("scm:git:git@github.com:rife2/bld-pmd.git")
|
||||
.url("https://github.com/rife2/bld-pmd")
|
||||
)
|
||||
.signKey(property("sign.key"))
|
||||
.signPassphrase(property("sign.passphrase"));
|
||||
|
|
|
@ -1,5 +1,5 @@
|
|||
/*
|
||||
* Copyright 2023-2024 the original author or authors.
|
||||
* Copyright 2023-2025 the original author or authors.
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
|
@ -49,6 +49,10 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
public static final String RULE_SET_DEFAULT = "rulesets/java/quickstart.xml";
|
||||
private static final Logger LOGGER = Logger.getLogger(PmdOperation.class.getName());
|
||||
private static final String PMD_DIR = "pmd";
|
||||
/**
|
||||
* The list of paths to exclude.
|
||||
*/
|
||||
private final Collection<Path> excludes_ = new ArrayList<>();
|
||||
/**
|
||||
* The input paths (source) list.
|
||||
*/
|
||||
|
@ -101,6 +105,10 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* The default language version(s).
|
||||
*/
|
||||
private Collection<LanguageVersion> languageVersions_ = new ArrayList<>();
|
||||
/**
|
||||
* The classpath to prepend.
|
||||
*/
|
||||
private String prependClasspath_;
|
||||
/**
|
||||
* The project reference.
|
||||
*/
|
||||
|
@ -134,6 +142,79 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
*/
|
||||
private int threads_ = 1;
|
||||
|
||||
/**
|
||||
* Adds paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes one or more paths to exclude
|
||||
* @return this operation
|
||||
* @see #excludes(Path...)
|
||||
* @since 1.2.0
|
||||
*/
|
||||
public PmdOperation addExcludes(Path... excludes) {
|
||||
return addExcludes(List.of(excludes));
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes paths to exclude
|
||||
* @return this operation
|
||||
* @see #excludes(Collection)
|
||||
* @since 1.2.0
|
||||
*/
|
||||
public PmdOperation addExcludes(Collection<Path> excludes) {
|
||||
excludes_.addAll(excludes);
|
||||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes one or more paths to exclude
|
||||
* @return this operation
|
||||
* @see #excludesFiles(Collection)
|
||||
* @since 1.2.0
|
||||
*/
|
||||
public PmdOperation addExcludesFiles(Collection<File> excludes) {
|
||||
return addExcludes(excludes.stream().map(File::toPath).toList());
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes one or more paths to exclude
|
||||
* @return this operation
|
||||
* @see #excludesFiles(File...)
|
||||
* @since 1.2.0
|
||||
*/
|
||||
public PmdOperation addExcludesFiles(File... excludes) {
|
||||
return addExcludesFiles(List.of(excludes));
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes one or more paths to exclude
|
||||
* @return this operation
|
||||
* @see #excludesStrings(Collection)
|
||||
* @since 1.2.0
|
||||
*/
|
||||
public PmdOperation addExcludesStrings(Collection<String> excludes) {
|
||||
return addExcludes(excludes.stream().map(Paths::get).toList());
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes one or more paths to exclude
|
||||
* @return this operation
|
||||
* @see #excludesStrings(String...)
|
||||
* @since 1.2.0
|
||||
*/
|
||||
public PmdOperation addExcludesStrings(String... excludes) {
|
||||
return addExcludesStrings(List.of(excludes));
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds paths to source files, or directories containing source files to analyze.\
|
||||
*
|
||||
|
@ -142,8 +223,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @see #inputPaths(Path...)
|
||||
*/
|
||||
public PmdOperation addInputPaths(Path... inputPath) {
|
||||
inputPaths_.addAll(List.of(inputPath));
|
||||
return this;
|
||||
return inputPaths(List.of(inputPath));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -154,8 +234,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @see #inputPaths(File...)
|
||||
*/
|
||||
public PmdOperation addInputPaths(File... inputPath) {
|
||||
inputPaths_.addAll(Arrays.stream(inputPath).map(File::toPath).toList());
|
||||
return this;
|
||||
return addInputPathsFiles(List.of(inputPath));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -163,11 +242,10 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
*
|
||||
* @param inputPath one or more paths
|
||||
* @return this operation
|
||||
* @see #addInputPaths(String...)
|
||||
* @see #inputPaths(String...)
|
||||
*/
|
||||
public PmdOperation addInputPaths(String... inputPath) {
|
||||
inputPaths_.addAll(Arrays.stream(inputPath).map(Paths::get).toList());
|
||||
return this;
|
||||
return addInputPathsStrings(List.of(inputPath));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -182,6 +260,28 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds paths to source files, or directories containing source files to analyze.
|
||||
*
|
||||
* @param inputPath a collection of input paths
|
||||
* @return this operation
|
||||
* @see #inputPathsFiles(Collection)
|
||||
*/
|
||||
public PmdOperation addInputPathsFiles(Collection<File> inputPath) {
|
||||
return addInputPaths(inputPath.stream().map(File::toPath).toList());
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds paths to source files, or directories containing source files to analyze.
|
||||
*
|
||||
* @param inputPath a collection of input paths
|
||||
* @return this operation
|
||||
* @see #inputPathsStrings(Collection)
|
||||
*/
|
||||
public PmdOperation addInputPathsStrings(Collection<String> inputPath) {
|
||||
return addInputPaths(inputPath.stream().map(Paths::get).toList());
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds new rule set paths.
|
||||
* <p>
|
||||
|
@ -203,8 +303,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @see #ruleSets(String...)
|
||||
*/
|
||||
public PmdOperation addRuleSet(String... ruleSet) {
|
||||
ruleSets_.addAll(List.of(ruleSet));
|
||||
return this;
|
||||
return addRuleSet(List.of(ruleSet));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -225,7 +324,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
*
|
||||
* @param ruleSet a collection of rule set paths
|
||||
* @return this operation
|
||||
* @see #ruleSets(Collection
|
||||
* @see #ruleSets(Collection)
|
||||
*/
|
||||
public PmdOperation addRuleSet(Collection<String> ruleSet) {
|
||||
ruleSets_.addAll(ruleSet);
|
||||
|
@ -240,6 +339,20 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets the location of the cache file for incremental analysis.
|
||||
*/
|
||||
public PmdOperation cache(File cache) {
|
||||
return cache(cache.toPath());
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets the location of the cache file for incremental analysis.
|
||||
*/
|
||||
public PmdOperation cache(String cache) {
|
||||
return cache(Path.of(cache));
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets the default language version to be used for all input files.
|
||||
*
|
||||
|
@ -247,8 +360,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @return this operation
|
||||
*/
|
||||
public PmdOperation defaultLanguageVersions(LanguageVersion... languageVersion) {
|
||||
languageVersions_.addAll(List.of(languageVersion));
|
||||
return this;
|
||||
return languageVersions(List.of(languageVersion));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -268,8 +380,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* <p>The valid values are the standard character sets of {@link java.nio.charset.Charset Charset}.</p>
|
||||
*/
|
||||
public PmdOperation encoding(String encoding) {
|
||||
encoding_ = Charset.forName(encoding);
|
||||
return this;
|
||||
return encoding(Charset.forName(encoding));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -283,6 +394,90 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes one or more paths to exclude
|
||||
* @return this operation
|
||||
* @see #addExcludes(Path...)
|
||||
*/
|
||||
public PmdOperation excludes(Path... excludes) {
|
||||
excludes(List.of(excludes));
|
||||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes paths to exclude
|
||||
* @return this operation
|
||||
* @see #addExcludes(Collection)
|
||||
*/
|
||||
public PmdOperation excludes(Collection<Path> excludes) {
|
||||
excludes_.clear();
|
||||
excludes_.addAll(excludes);
|
||||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns the paths to exclude from the analysis.
|
||||
*
|
||||
* @return the exclude paths
|
||||
*/
|
||||
public Collection<Path> excludes() {
|
||||
return excludes_;
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes one or more paths to exclude
|
||||
* @return this operation
|
||||
* @see #excludesFiles(Collection)
|
||||
* @since 1.2.0
|
||||
*/
|
||||
public PmdOperation excludesFiles(Collection<File> excludes) {
|
||||
excludes(excludes.stream().map(File::toPath).toList());
|
||||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes one or more paths to exclude
|
||||
* @return this operation
|
||||
* @see #excludesFiles(File...)
|
||||
* @since 1.2.0
|
||||
*/
|
||||
public PmdOperation excludesFiles(File... excludes) {
|
||||
return excludesFiles(List.of(excludes));
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes one or more paths to exclude
|
||||
* @return this operation
|
||||
* @see #excludesStrings(Collection)
|
||||
* @since 1.2.0
|
||||
*/
|
||||
public PmdOperation excludesStrings(Collection<String> excludes) {
|
||||
excludes(excludes.stream().map(Paths::get).toList());
|
||||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets paths to exclude from the analysis.
|
||||
*
|
||||
* @param excludes one or more paths to exclude
|
||||
* @return this operation
|
||||
* @see #excludesStrings(String...)
|
||||
* @since 1.2.0
|
||||
*/
|
||||
public PmdOperation excludesStrings(String... excludes) {
|
||||
return excludesStrings(List.of(excludes));
|
||||
}
|
||||
|
||||
/**
|
||||
* Performs the PMD code analysis operation.
|
||||
*/
|
||||
|
@ -385,8 +580,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @return this operation
|
||||
*/
|
||||
public PmdOperation ignoreFile(File ignoreFile) {
|
||||
ignoreFile_ = ignoreFile.toPath();
|
||||
return this;
|
||||
return ignoreFile(ignoreFile.toPath());
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -396,8 +590,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @return this operation
|
||||
*/
|
||||
public PmdOperation ignoreFile(String ignoreFile) {
|
||||
ignoreFile_ = Paths.get(ignoreFile);
|
||||
return this;
|
||||
return ignoreFile(Path.of(ignoreFile));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -427,8 +620,17 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @return this operation
|
||||
*/
|
||||
public PMDConfiguration initConfiguration(String commandName) {
|
||||
PMDConfiguration config = new PMDConfiguration();
|
||||
var config = new PMDConfiguration();
|
||||
|
||||
// addRelativizeRoots
|
||||
config.addRelativizeRoots(relativizeRoots_.stream().toList());
|
||||
|
||||
// prependAuxClasspath
|
||||
if (prependClasspath_ != null) {
|
||||
config.prependAuxClasspath(prependClasspath_);
|
||||
}
|
||||
|
||||
// setAnalysisCacheLocation
|
||||
if (cache_ == null && project_ != null && incrementalAnalysis_) {
|
||||
config.setAnalysisCacheLocation(
|
||||
Paths.get(project_.buildDirectory().getPath(), PMD_DIR, PMD_DIR + "-cache").toFile().getAbsolutePath());
|
||||
|
@ -436,38 +638,50 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
config.setAnalysisCacheLocation(cache_.toFile().getAbsolutePath());
|
||||
}
|
||||
|
||||
config.setFailOnError(failOnError_);
|
||||
config.setFailOnViolation(failOnViolation_);
|
||||
|
||||
// setDefaultLanguageVersions
|
||||
if (languageVersions_ != null) {
|
||||
config.setDefaultLanguageVersions(languageVersions_.stream().toList());
|
||||
}
|
||||
|
||||
// setExcludes
|
||||
if (!excludes_.isEmpty()) {
|
||||
config.setExcludes(excludes_.stream().toList());
|
||||
}
|
||||
|
||||
// setFailOnError
|
||||
config.setFailOnError(failOnError_);
|
||||
// setFailOnViolation
|
||||
config.setFailOnViolation(failOnViolation_);
|
||||
|
||||
// setForceLanguageVersion
|
||||
if (forcedLanguageVersion_ != null) {
|
||||
config.setForceLanguageVersion(forcedLanguageVersion_);
|
||||
}
|
||||
|
||||
// setIgnoreFilePath
|
||||
if (ignoreFile_ != null) {
|
||||
config.setIgnoreFilePath(ignoreFile_);
|
||||
}
|
||||
|
||||
// setIgnoreIncrementalAnalysis
|
||||
config.setIgnoreIncrementalAnalysis(!incrementalAnalysis_);
|
||||
|
||||
// setInputPathList
|
||||
if (inputPaths_.isEmpty()) {
|
||||
throw new IllegalArgumentException(commandName + ": InputPaths required.");
|
||||
} else {
|
||||
config.setInputPathList(inputPaths_.stream().toList());
|
||||
}
|
||||
if (reportProperties_ != null) {
|
||||
config.setReportProperties(reportProperties_);
|
||||
}
|
||||
|
||||
// setInputUri
|
||||
if (inputUri_ != null) {
|
||||
config.setInputUri(inputUri_);
|
||||
}
|
||||
|
||||
// setMinimumPriority
|
||||
config.setMinimumPriority(rulePriority_);
|
||||
|
||||
// setReportFile
|
||||
if (project_ != null) {
|
||||
config.setReportFile(Objects.requireNonNullElseGet(reportFile_,
|
||||
() -> Paths.get(project_.buildDirectory().getPath(),
|
||||
|
@ -476,12 +690,25 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
config.setReportFile(reportFile_);
|
||||
}
|
||||
|
||||
config.addRelativizeRoots(relativizeRoots_.stream().toList());
|
||||
// setReportFormat
|
||||
config.setReportFormat(reportFormat_);
|
||||
|
||||
// setReportProperties
|
||||
if (reportProperties_ != null) {
|
||||
config.setReportProperties(reportProperties_);
|
||||
}
|
||||
|
||||
// setRuleSets
|
||||
config.setRuleSets(ruleSets_.stream().toList());
|
||||
|
||||
// setShowSuppressedViolations
|
||||
config.setShowSuppressedViolations(showSuppressed_);
|
||||
// setSourceEncoding
|
||||
config.setSourceEncoding(encoding_);
|
||||
// setSuppressMarker
|
||||
config.setSuppressMarker(suppressedMarker_);
|
||||
|
||||
// setThreads
|
||||
config.setThreads(threads_);
|
||||
|
||||
return config;
|
||||
|
@ -495,9 +722,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @see #addInputPaths(Path...)
|
||||
*/
|
||||
public PmdOperation inputPaths(Path... inputPath) {
|
||||
inputPaths_.clear();
|
||||
inputPaths_.addAll(List.of(inputPath));
|
||||
return this;
|
||||
return inputPaths(List.of(inputPath));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -510,9 +735,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @see #addInputPaths(File...)
|
||||
*/
|
||||
public PmdOperation inputPaths(File... inputPath) {
|
||||
inputPaths_.clear();
|
||||
inputPaths_.addAll(Arrays.stream(inputPath).map(File::toPath).toList());
|
||||
return this;
|
||||
return inputPathsFiles(List.of(inputPath));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -525,9 +748,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @see #addInputPaths(String...)
|
||||
*/
|
||||
public PmdOperation inputPaths(String... inputPath) {
|
||||
inputPaths_.clear();
|
||||
inputPaths_.addAll(Arrays.stream(inputPath).map(Paths::get).toList());
|
||||
return this;
|
||||
return inputPathsStrings(List.of(inputPath));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -554,6 +775,32 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
return inputPaths_;
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets paths to source files, or directories containing source files to analyze.
|
||||
* <p>
|
||||
* Previous entries are disregarded.
|
||||
*
|
||||
* @param inputPath a collection of input paths
|
||||
* @return this operation
|
||||
* @see #addInputPathsFiles(Collection)
|
||||
*/
|
||||
public PmdOperation inputPathsFiles(Collection<File> inputPath) {
|
||||
return inputPaths(inputPath.stream().map(File::toPath).toList());
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets paths to source files, or directories containing source files to analyze.
|
||||
* <p>
|
||||
* Previous entries are disregarded.
|
||||
*
|
||||
* @param inputPath a collection of input paths
|
||||
* @return this operation
|
||||
* @see #addInputPathsStrings(Collection)
|
||||
*/
|
||||
public PmdOperation inputPathsStrings(Collection<String> inputPath) {
|
||||
return inputPaths(inputPath.stream().map(Paths::get).toList());
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets the default language versions.
|
||||
*
|
||||
|
@ -561,8 +808,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @return this operation
|
||||
*/
|
||||
public PmdOperation languageVersions(LanguageVersion... languageVersion) {
|
||||
languageVersions_.addAll(List.of(languageVersion));
|
||||
return this;
|
||||
return languageVersions(List.of(languageVersion));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -662,44 +908,70 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
return numViolations;
|
||||
}
|
||||
|
||||
/**
|
||||
* Prepend the specified classpath like string to the current ClassLoader of the configuration. If no ClassLoader
|
||||
* is currently configured, the ClassLoader used to load the PMDConfiguration class will be used as the parent
|
||||
* ClassLoader of the created ClassLoader.
|
||||
* <p>
|
||||
* If the classpath String looks like a URL to a file (i.e. starts with {@code file://}) the file will be read with
|
||||
* each line representing an entry on the classpath.
|
||||
*
|
||||
* @param classpath one or more classpath
|
||||
* @return this operation
|
||||
*/
|
||||
public PmdOperation prependAuxClasspath(String... classpath) {
|
||||
prependClasspath_ = String.join(File.pathSeparator, classpath);
|
||||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns the prepended classpath.
|
||||
*
|
||||
* @return the classpath
|
||||
*/
|
||||
public String prependAuxClasspath() {
|
||||
return prependClasspath_;
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds several paths to shorten paths that are output in the report.
|
||||
*
|
||||
* @param relativeRoot one or more relative root paths
|
||||
* @return this operations
|
||||
* @return this operation
|
||||
* @see #relativizeRoots(Collection)
|
||||
*/
|
||||
public PmdOperation relativizeRoots(Path... relativeRoot) {
|
||||
relativizeRoots_.addAll(List.of(relativeRoot));
|
||||
return this;
|
||||
return relativizeRoots(List.of(relativeRoot));
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds several paths to shorten paths that are output in the report.
|
||||
*
|
||||
* @param relativeRoot one or more relative root paths
|
||||
* @return this operations
|
||||
* @return this operation
|
||||
* @see #relativizeRootsFiles(Collection)
|
||||
*/
|
||||
public PmdOperation relativizeRoots(File... relativeRoot) {
|
||||
relativizeRoots_.addAll(Arrays.stream(relativeRoot).map(File::toPath).toList());
|
||||
return this;
|
||||
return relativizeRootsFiles(List.of(relativeRoot));
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds several paths to shorten paths that are output in the report.
|
||||
*
|
||||
* @param relativeRoot one or more relative root paths
|
||||
* @return this operations
|
||||
* @return this operation
|
||||
* @see #relativizeRootsStrings(Collection)
|
||||
*/
|
||||
public PmdOperation relativizeRoots(String... relativeRoot) {
|
||||
relativizeRoots_.addAll(Arrays.stream(relativeRoot).map(Paths::get).toList());
|
||||
return this;
|
||||
return relativizeRootsStrings(List.of(relativeRoot));
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds several paths to shorten paths that are output in the report.
|
||||
*
|
||||
* @param relativeRoot a collection of relative root paths
|
||||
* @return this operations
|
||||
* @return this operation
|
||||
* @see #relativizeRoots(Path...)
|
||||
*/
|
||||
public PmdOperation relativizeRoots(Collection<Path> relativeRoot) {
|
||||
relativizeRoots_.addAll(relativeRoot);
|
||||
|
@ -715,6 +987,28 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
return relativizeRoots_;
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds several paths to shorten paths that are output in the report.
|
||||
*
|
||||
* @param relativeRoot a collection of relative root paths
|
||||
* @return this operation
|
||||
* @see #relativizeRoots(File...)
|
||||
*/
|
||||
public PmdOperation relativizeRootsFiles(Collection<File> relativeRoot) {
|
||||
return relativizeRoots(relativeRoot.stream().map(File::toPath).toList());
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds several paths to shorten paths that are output in the report.
|
||||
*
|
||||
* @param relativeRoot a collection of relative root paths
|
||||
* @return this operation
|
||||
* @see #relativizeRoots(String...)
|
||||
*/
|
||||
public PmdOperation relativizeRootsStrings(Collection<String> relativeRoot) {
|
||||
return relativizeRoots(relativeRoot.stream().map(Paths::get).toList());
|
||||
}
|
||||
|
||||
/**
|
||||
* Sets the path to the report page.
|
||||
*
|
||||
|
@ -733,8 +1027,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @return this operation
|
||||
*/
|
||||
public PmdOperation reportFile(File reportFile) {
|
||||
reportFile_ = reportFile.toPath();
|
||||
return this;
|
||||
return reportFile(reportFile.toPath());
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -744,8 +1037,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @return this operation
|
||||
*/
|
||||
public PmdOperation reportFile(String reportFile) {
|
||||
reportFile_ = Paths.get(reportFile);
|
||||
return this;
|
||||
return reportFile(Paths.get(reportFile));
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -800,9 +1092,7 @@ public class PmdOperation extends AbstractOperation<PmdOperation> {
|
|||
* @see #addRuleSet(String...)
|
||||
*/
|
||||
public PmdOperation ruleSets(String... ruleSet) {
|
||||
ruleSets_.clear();
|
||||
ruleSets_.addAll(List.of(ruleSet));
|
||||
return this;
|
||||
return ruleSets(List.of(ruleSet));
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -1,5 +1,5 @@
|
|||
/*
|
||||
* Copyright 2023-2024 the original author or authors.
|
||||
* Copyright 2023-2025 the original author or authors.
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
|
@ -19,6 +19,7 @@ package rife.bld.extension;
|
|||
import net.sourceforge.pmd.PMDConfiguration;
|
||||
import net.sourceforge.pmd.lang.LanguageRegistry;
|
||||
import net.sourceforge.pmd.lang.rule.RulePriority;
|
||||
import org.assertj.core.api.AutoCloseableSoftAssertions;
|
||||
import org.junit.jupiter.api.Test;
|
||||
import rife.bld.BaseProject;
|
||||
import rife.bld.operations.exceptions.ExitStatusException;
|
||||
|
@ -46,6 +47,7 @@ import static org.assertj.core.api.Assertions.assertThatCode;
|
|||
* @since 1.0
|
||||
*/
|
||||
class PmdOperationTest {
|
||||
static final String BAR = "bar";
|
||||
static final String CATEGORY_FOO = "category/foo.xml";
|
||||
static final Path CODE_STYLE_SAMPLE = Path.of("src/test/resources/java/CodeStyle.java");
|
||||
static final String CODE_STYLE_XML = "category/java/codestyle.xml";
|
||||
|
@ -54,6 +56,11 @@ class PmdOperationTest {
|
|||
static final String DOCUMENTATION_XML = "category/java/documentation.xml";
|
||||
static final Path ERROR_PRONE_SAMPLE = Path.of("src/test/resources/java/ErrorProne.java");
|
||||
static final String ERROR_PRONE_XML = "category/java/errorprone.xml";
|
||||
static final File FILE_BAR = new File(BAR);
|
||||
static final String FOO = "foo";
|
||||
static final File FILE_FOO = new File(FOO);
|
||||
static final Path PATH_BAR = Path.of(BAR);
|
||||
static final Path PATH_FOO = Path.of(FOO);
|
||||
static final String PERFORMANCE_XML = "category/java/performance.xml";
|
||||
static final String SECURITY_XML = "category/java/security.xml";
|
||||
static final String TEST = "test";
|
||||
|
@ -61,13 +68,46 @@ class PmdOperationTest {
|
|||
PmdOperation newPmdOperation() {
|
||||
return new PmdOperation()
|
||||
.inputPaths(Path.of("src/main"), Path.of("src/test"))
|
||||
.cache(Paths.get("build", COMMAND_NAME, "pmd-cache"))
|
||||
.cache("build/" + COMMAND_NAME + "/pmd-cache")
|
||||
.failOnViolation(false)
|
||||
.reportFile(Paths.get("build", COMMAND_NAME, "pmd-test-report.txt"));
|
||||
}
|
||||
|
||||
@Test
|
||||
void testAddInputPath() throws ExitStatusException {
|
||||
void testAddExcludes() {
|
||||
var pmd = newPmdOperation().ruleSets(CATEGORY_FOO).addExcludes(PATH_FOO);
|
||||
var config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getExcludes()).containsExactly(PATH_FOO);
|
||||
|
||||
pmd = pmd.addExcludes(PATH_BAR);
|
||||
config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getExcludes()).containsExactly(PATH_FOO, PATH_BAR);
|
||||
}
|
||||
|
||||
@Test
|
||||
void testAddExcludesFiles() {
|
||||
var pmd = newPmdOperation().ruleSets(CATEGORY_FOO).addExcludesFiles(FILE_FOO);
|
||||
var config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getExcludes()).containsExactly(FILE_FOO.toPath());
|
||||
|
||||
pmd = pmd.addExcludesFiles(FILE_BAR);
|
||||
config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getExcludes()).containsExactly(FILE_FOO.toPath(), FILE_BAR.toPath());
|
||||
}
|
||||
|
||||
@Test
|
||||
void testAddExcludesStrings() {
|
||||
var pmd = newPmdOperation().ruleSets(CATEGORY_FOO).addExcludesStrings(FOO);
|
||||
var config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getExcludes()).containsExactly(PATH_FOO);
|
||||
|
||||
pmd = pmd.addExcludesStrings(BAR);
|
||||
config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getExcludes()).containsExactly(PATH_FOO, PATH_BAR);
|
||||
}
|
||||
|
||||
@Test
|
||||
void testAddInputPaths() throws ExitStatusException {
|
||||
var project = new BaseProject();
|
||||
var pmd = new PmdOperation().fromProject(project);
|
||||
|
||||
|
@ -77,22 +117,42 @@ class PmdOperationTest {
|
|||
var err = pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME));
|
||||
|
||||
pmd.inputPaths().clear();
|
||||
pmd.addInputPaths(project.srcMainDirectory());
|
||||
pmd = pmd.addInputPaths(project.srcMainDirectory(), project.srcTestDirectory());
|
||||
|
||||
assertThat(pmd.inputPaths()).as("main").containsExactly(project.srcMainDirectory().toPath());
|
||||
|
||||
pmd.inputPaths().clear();
|
||||
pmd.addInputPaths(project.srcMainDirectory().toPath(), project.srcTestDirectory().toPath());
|
||||
|
||||
assertThat(pmd.inputPaths()).as("toPath(main, test)").containsExactly(project.srcMainDirectory().toPath(),
|
||||
assertThat(pmd.inputPaths()).as("File...").containsExactly(project.srcMainDirectory().toPath(),
|
||||
project.srcTestDirectory().toPath());
|
||||
|
||||
pmd.inputPaths().clear();
|
||||
pmd.addInputPaths(List.of(project.srcMainDirectory().toPath(), project.srcTestDirectory().toPath()));
|
||||
pmd = pmd.addInputPathsFiles(List.of(project.srcMainDirectory(), project.srcTestDirectory()));
|
||||
|
||||
assertThat(pmd.inputPaths()).as("List(main, test)").containsExactly(
|
||||
project.srcMainDirectory().toPath(),
|
||||
project.srcTestDirectory().toPath());
|
||||
assertThat(pmd.inputPaths()).as("List(File...)")
|
||||
.containsExactly(project.srcMainDirectory().toPath(), project.srcTestDirectory().toPath());
|
||||
|
||||
pmd.inputPaths().clear();
|
||||
pmd = pmd.addInputPaths(project.srcMainDirectory().getAbsolutePath(),
|
||||
project.srcTestDirectory().getAbsolutePath());
|
||||
|
||||
assertThat(pmd.inputPaths()).as("String...")
|
||||
.containsExactly(project.srcMainDirectory().toPath(), project.srcTestDirectory().toPath());
|
||||
|
||||
pmd.inputPaths().clear();
|
||||
pmd = pmd.addInputPathsStrings(
|
||||
List.of(project.srcMainDirectory().getAbsolutePath(), project.srcTestDirectory().getAbsolutePath()));
|
||||
|
||||
assertThat(pmd.inputPaths()).as("List(String...)")
|
||||
.containsExactly(project.srcMainDirectory().toPath(), project.srcTestDirectory().toPath());
|
||||
|
||||
pmd.inputPaths().clear();
|
||||
pmd = pmd.addInputPaths(project.srcMainDirectory().toPath(), project.srcTestDirectory().toPath());
|
||||
|
||||
assertThat(pmd.inputPaths()).as("Path...")
|
||||
.containsExactly(project.srcMainDirectory().toPath(), project.srcTestDirectory().toPath());
|
||||
|
||||
pmd.inputPaths().clear();
|
||||
pmd = pmd.addInputPaths(List.of(project.srcMainDirectory().toPath(), project.srcTestDirectory().toPath()));
|
||||
|
||||
assertThat(pmd.inputPaths()).as("List(Path)")
|
||||
.containsExactly(project.srcMainDirectory().toPath(), project.srcTestDirectory().toPath());
|
||||
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME)))
|
||||
.isGreaterThan(0).isEqualTo(err);
|
||||
|
@ -124,7 +184,10 @@ class PmdOperationTest {
|
|||
@Test
|
||||
void testCache() throws ExitStatusException {
|
||||
var cache = Path.of("build/pmd/temp-cache");
|
||||
var pmd = newPmdOperation().ruleSets(CODE_STYLE_XML).inputPaths(List.of(CODE_STYLE_SAMPLE)).cache(cache);
|
||||
var pmd = newPmdOperation()
|
||||
.ruleSets(CODE_STYLE_XML)
|
||||
.inputPaths(List.of(CODE_STYLE_SAMPLE))
|
||||
.cache(cache);
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME))).isGreaterThan(0);
|
||||
var f = cache.toFile();
|
||||
assertThat(f.exists()).as("file exits").isTrue();
|
||||
|
@ -142,6 +205,59 @@ class PmdOperationTest {
|
|||
assertThat(config.getSourceEncoding()).as("ISO_8859").isEqualTo(StandardCharsets.ISO_8859_1);
|
||||
}
|
||||
|
||||
@Test
|
||||
void testExcludes() {
|
||||
var foz = Path.of("foz/baz");
|
||||
var baz = Path.of("baz/foz");
|
||||
|
||||
var pmd = newPmdOperation().ruleSets(CATEGORY_FOO).excludes(PATH_FOO, PATH_BAR);
|
||||
var config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(pmd.excludes()).containsExactly(List.of(PATH_FOO, PATH_BAR).toArray(new Path[0]));
|
||||
assertThat(config.getExcludes()).containsExactly(List.of(PATH_FOO, PATH_BAR).toArray(new Path[0]));
|
||||
|
||||
var excludes = List.of(List.of(PATH_FOO, PATH_BAR), List.of(foz, baz));
|
||||
for (var exclude : excludes) {
|
||||
pmd = newPmdOperation().ruleSets(CATEGORY_FOO).excludes(exclude);
|
||||
config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getExcludes()).containsExactly(exclude.toArray(new Path[0]));
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
void testExcludesFiles() {
|
||||
var foz = new File("foz");
|
||||
var baz = new File("baz");
|
||||
|
||||
var pmd = newPmdOperation().ruleSets(CATEGORY_FOO).excludesFiles(FILE_FOO, FILE_BAR);
|
||||
var config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getExcludes()).containsExactly(FILE_FOO.toPath(), FILE_BAR.toPath());
|
||||
|
||||
var excludes = List.of(List.of(FILE_FOO, FILE_BAR), List.of(foz, baz));
|
||||
for (var exclude : excludes) {
|
||||
pmd = newPmdOperation().ruleSets(CATEGORY_FOO).excludesFiles(exclude);
|
||||
config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getExcludes()).containsExactly(exclude.stream().map(File::toPath).toArray(Path[]::new));
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
void testExcludesStrings() {
|
||||
var foz = "foz";
|
||||
var baz = "baz";
|
||||
|
||||
var pmd = newPmdOperation().ruleSets(CATEGORY_FOO).excludesStrings(FOO, BAR);
|
||||
var config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(pmd.excludes()).containsExactly(PATH_FOO, PATH_BAR);
|
||||
assertThat(config.getExcludes()).containsExactly(PATH_FOO, PATH_BAR);
|
||||
|
||||
var excludes = List.of(List.of(FOO, BAR), List.of(foz, baz));
|
||||
for (var exclude : excludes) {
|
||||
pmd = newPmdOperation().ruleSets(CATEGORY_FOO).excludesStrings(exclude);
|
||||
config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getExcludes()).containsExactly(exclude.stream().map(Paths::get).toArray(Path[]::new));
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
void testExecute() throws ExitStatusException {
|
||||
var pmd = new PmdOperation().fromProject(new BaseProject());
|
||||
|
@ -213,7 +329,31 @@ class PmdOperationTest {
|
|||
var pmd = newPmdOperation()
|
||||
.ruleSets(PmdOperation.RULE_SET_DEFAULT, CODE_STYLE_XML)
|
||||
.inputPaths(ERROR_PRONE_SAMPLE, CODE_STYLE_SAMPLE);
|
||||
assertThat(pmd.inputPaths()).contains(ERROR_PRONE_SAMPLE, CODE_STYLE_SAMPLE);
|
||||
assertThat(pmd.inputPaths()).as("Path....").containsExactly(ERROR_PRONE_SAMPLE, CODE_STYLE_SAMPLE);
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME))).isGreaterThan(0);
|
||||
|
||||
pmd = newPmdOperation()
|
||||
.ruleSets(PmdOperation.RULE_SET_DEFAULT, CODE_STYLE_XML)
|
||||
.inputPaths(ERROR_PRONE_SAMPLE.toFile(), CODE_STYLE_SAMPLE.toFile());
|
||||
assertThat(pmd.inputPaths()).as("File...").containsExactly(ERROR_PRONE_SAMPLE, CODE_STYLE_SAMPLE);
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME))).isGreaterThan(0);
|
||||
|
||||
pmd = newPmdOperation()
|
||||
.ruleSets(PmdOperation.RULE_SET_DEFAULT, CODE_STYLE_XML)
|
||||
.inputPaths(ERROR_PRONE_SAMPLE.toString(), CODE_STYLE_SAMPLE.toString());
|
||||
assertThat(pmd.inputPaths()).as("String...").containsExactly(ERROR_PRONE_SAMPLE, CODE_STYLE_SAMPLE);
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME))).isGreaterThan(0);
|
||||
|
||||
pmd = newPmdOperation()
|
||||
.ruleSets(PmdOperation.RULE_SET_DEFAULT, CODE_STYLE_XML)
|
||||
.inputPathsFiles(List.of(ERROR_PRONE_SAMPLE.toFile(), CODE_STYLE_SAMPLE.toFile()));
|
||||
assertThat(pmd.inputPaths()).as("List(Path...)").containsExactly(ERROR_PRONE_SAMPLE, CODE_STYLE_SAMPLE);
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME))).isGreaterThan(0);
|
||||
|
||||
pmd = newPmdOperation()
|
||||
.ruleSets(PmdOperation.RULE_SET_DEFAULT, CODE_STYLE_XML)
|
||||
.inputPathsStrings(List.of(ERROR_PRONE_SAMPLE.toString(), CODE_STYLE_SAMPLE.toString()));
|
||||
assertThat(pmd.inputPaths()).as("List(String...)").containsExactly(ERROR_PRONE_SAMPLE, CODE_STYLE_SAMPLE);
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME))).isGreaterThan(0);
|
||||
}
|
||||
|
||||
|
@ -235,6 +375,7 @@ class PmdOperationTest {
|
|||
var pmd = newPmdOperation().ruleSets(CODE_STYLE_XML).inputPaths(CODE_STYLE_SAMPLE);
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME)))
|
||||
.as("code style").isGreaterThan(0);
|
||||
|
||||
pmd = pmd.ruleSets(ERROR_PRONE_XML).inputPaths(ERROR_PRONE_SAMPLE);
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME)))
|
||||
.as("code style + error prone").isGreaterThan(0);
|
||||
|
@ -245,7 +386,7 @@ class PmdOperationTest {
|
|||
var pmd = newPmdOperation()
|
||||
.ruleSets(DESIGN_XML)
|
||||
.inputPaths("src/test/resources/java/Design.java")
|
||||
.cache(Path.of("build/pmd/design-cache"));
|
||||
.cache(new File("build/pmd/design-cache"));
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME))).isGreaterThan(0);
|
||||
}
|
||||
|
||||
|
@ -301,7 +442,7 @@ class PmdOperationTest {
|
|||
var pmd = newPmdOperation()
|
||||
.forceLanguageVersion(language.getLatestVersion())
|
||||
.defaultLanguageVersions(language.getVersions())
|
||||
.languageVersions(language.getVersion("22"))
|
||||
.languageVersions(language.getDefaultVersion())
|
||||
.ruleSets(PmdOperation.RULE_SET_DEFAULT);
|
||||
assertThat(pmd.languageVersions()).contains(language.getDefaultVersion());
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME))).isGreaterThan(0);
|
||||
|
@ -319,6 +460,12 @@ class PmdOperationTest {
|
|||
assertThat(pmd).isEqualTo(0);
|
||||
}
|
||||
|
||||
@Test
|
||||
void testPrependAuxClasspath() {
|
||||
var pmd = newPmdOperation().ruleSets(CATEGORY_FOO).prependAuxClasspath(FOO, BAR);
|
||||
assertThat(pmd.prependAuxClasspath()).isEqualTo(FOO + File.pathSeparator + BAR);
|
||||
}
|
||||
|
||||
@Test
|
||||
void testPriority() throws ExitStatusException {
|
||||
var pmd = newPmdOperation().inputPaths(CODE_STYLE_SAMPLE).minimumPriority(RulePriority.HIGH);
|
||||
|
@ -327,15 +474,26 @@ class PmdOperationTest {
|
|||
|
||||
@Test
|
||||
void testRelativizeRoots() {
|
||||
var foo = Path.of("foo/bar");
|
||||
var bar = Path.of("bar/foo");
|
||||
var baz = Path.of("baz/foz");
|
||||
|
||||
var pmd = newPmdOperation().ruleSets(List.of(CATEGORY_FOO)).relativizeRoots(foo).relativizeRoots(bar.toFile())
|
||||
.relativizeRoots(baz.toString()).relativizeRoots(List.of(foo, bar, baz));
|
||||
var pmd = newPmdOperation().ruleSets(List.of(CATEGORY_FOO)).relativizeRoots(PATH_FOO).
|
||||
relativizeRoots(PATH_BAR.toFile()).relativizeRoots(baz.toString())
|
||||
.relativizeRoots(List.of(PATH_FOO, PATH_BAR, baz));
|
||||
var config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getRelativizeRoots()).isEqualTo(pmd.relativizeRoots());
|
||||
assertThat(config.getRelativizeRoots()).containsExactly(foo, bar, baz, foo, bar, baz);
|
||||
assertThat(config.getRelativizeRoots()).isEqualTo(pmd.relativizeRoots())
|
||||
.containsExactly(PATH_FOO, PATH_BAR, baz, PATH_FOO, PATH_BAR, baz);
|
||||
|
||||
pmd = newPmdOperation().ruleSets(List.of(CATEGORY_FOO))
|
||||
.relativizeRootsFiles(List.of(PATH_FOO.toFile(), PATH_BAR.toFile(), baz.toFile()));
|
||||
config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getRelativizeRoots()).as("List(File...)").isEqualTo(pmd.relativizeRoots())
|
||||
.containsExactly(PATH_FOO, PATH_BAR, baz);
|
||||
|
||||
pmd = newPmdOperation().ruleSets(List.of(CATEGORY_FOO))
|
||||
.relativizeRootsStrings(List.of(PATH_FOO.toString(), PATH_BAR.toString(), baz.toString()));
|
||||
config = pmd.initConfiguration(COMMAND_NAME);
|
||||
assertThat(config.getRelativizeRoots()).as("List(String....)").isEqualTo(pmd.relativizeRoots())
|
||||
.containsExactly(PATH_FOO, PATH_BAR, baz);
|
||||
}
|
||||
|
||||
@Test
|
||||
|
@ -359,8 +517,11 @@ class PmdOperationTest {
|
|||
void testReportFormat() throws IOException, ExitStatusException {
|
||||
var pmd = newPmdOperation().ruleSets(ERROR_PRONE_XML).reportFormat("xml").inputPaths(ERROR_PRONE_SAMPLE);
|
||||
assertThat(pmd.performPmdAnalysis(TEST, pmd.initConfiguration(COMMAND_NAME))).isGreaterThan(0);
|
||||
try (var br = Files.newBufferedReader(pmd.reportFile())) {
|
||||
assertThat(br.readLine()).as("xml report").startsWith("<?xml version=\"1.0\" encoding=\"UTF-8\"?>");
|
||||
try (var softly = new AutoCloseableSoftAssertions()) {
|
||||
try (var br = Files.newBufferedReader(pmd.reportFile())) {
|
||||
softly.assertThat(br.readLine()).as("xml report")
|
||||
.startsWith("<?xml version=\"1.0\" encoding=\"UTF-8\"?>");
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue