diff --git a/multi/multi__spring_cloud_contract_stub_runner.html b/multi/multi__spring_cloud_contract_stub_runner.html
index 4b7720c6f5..f60b2bd4e2 100644
--- a/multi/multi__spring_cloud_contract_stub_runner.html
+++ b/multi/multi__spring_cloud_contract_stub_runner.html
@@ -294,19 +294,20 @@ Check their import org.springframework.cloud.contract.spec.Contract;
public interface StubFinder extends StubTrigger {
+
/**
- * For the given groupId and artifactId tries to find the matching
- * URL of the running stub.
- *
- * @param groupId - might be null. In that case a search only via artifactId takes place
+ * For the given groupId and artifactId tries to find the matching URL of the running
+ * stub.
+ * @param groupId - might be null. In that case a search only via artifactId takes
+ * place
* @return URL of a running stub or throws exception if not found
*/
URL findStubUrl(String groupId, String artifactId) throws StubNotFoundException;
/**
- * For the given Ivy notation {@code [groupId]:artifactId:[version]:[classifier]} tries to
- * find the matching URL of the running stub. You can also pass only {@code artifactId}.
- *
+ * For the given Ivy notation {@code [groupId]:artifactId:[version]:[classifier]}
+ * tries to find the matching URL of the running stub. You can also pass only
+ * {@code artifactId}.
* @param ivyNotation - Ivy representation of the Maven artifact
* @return URL of a running stub or throws exception if not found
*/
@@ -321,6 +322,7 @@ Check their
Map<StubConfiguration, Collection<Contract>> getContracts();
+
} Example of usage in Spock tests:@ClassRule @Shared StubRunnerRule rule = new StubRunnerRule()
.stubsMode(StubRunnerProperties.StubsMode.REMOTE)
.repoRoot(StubRunnerRuleSpec.getResource("/m2repo/repository").toURI().toString())
diff --git a/multi/multi__spring_cloud_contract_wiremock.html b/multi/multi__spring_cloud_contract_wiremock.html
index ab7fe3047c..5ba211e421 100644
--- a/multi/multi__spring_cloud_contract_wiremock.html
+++ b/multi/multi__spring_cloud_contract_wiremock.html
@@ -12,21 +12,23 @@ your test. The following code shows an example:
< @SpringBootTest(webEnvironment = WebEnvironment.RANDOM_PORT) @AutoConfigureWireMock(port = 0) public class WiremockForDocsTests { + // A service that calls out over HTTP - @Autowired private Service service; + @Autowired + private Service service; // Using the WireMock APIs in the normal way: @Test public void contextLoads() throws Exception { // Stubbing WireMock - stubFor(get(urlEqualTo("/resource")) - .willReturn(aResponse().withHeader("Content-Type", "text/plain").withBody("Hello World!"))); + stubFor(get(urlEqualTo("/resource")).willReturn(aResponse() + .withHeader("Content-Type", "text/plain").withBody("Hello World!"))); // We're asserting if WireMock responded properly assertThat(this.service.go()).isEqualTo("Hello World!"); } } -//end::wiremock_test2[]
To start the stub server on a different port use (for example), +// end::wiremock_test2[]
To start the stub server on a different port use (for example),
@AutoConfigureWireMock(port=9999). For a random port, use a value of 0. The stub
server port can be bound in the test application context with the "wiremock.server.port"
property. Using @AutoConfigureWireMock adds a bean of type WiremockConfiguration to
@@ -72,6 +74,7 @@ instance, as shown in the following example:
@ClassRule public static WireMockClassRule wiremock = new WireMockClassRule( WireMockSpring.options().dynamicPort()); + // A service that calls out over HTTP to localhost:${wiremock.port} @Autowired private Service service; @@ -80,14 +83,14 @@ instance, as shown in the following example:@Test public void contextLoads() throws Exception { // Stubbing WireMock - wiremock.stubFor(get(urlEqualTo("/resource")) - .willReturn(aResponse().withHeader("Content-Type", "text/plain").withBody("Hello World!"))); + wiremock.stubFor(get(urlEqualTo("/resource")).willReturn(aResponse() + .withHeader("Content-Type", "text/plain").withBody("Hello World!"))); // We're asserting if WireMock responded properly assertThat(this.service.go()).isEqualTo("Hello World!"); } } -//end::wiremock_test2[]The
@ClassRulemeans that the server shuts down after all the methods in this class +// end::wiremock_test2[]
The @ClassRule means that the server shuts down after all the methods in this class
have been run.
WireMock lets you stub a "secure" server with an "https" URL protocol. If your
application wants to contact that stub server in an integration test, it will find that
the SSL certificates are not valid (the usual problem with self-installed certificates).
@@ -136,6 +139,7 @@ a Spring MockRestServiceServer. The following code
assertThat(this.service.go()).isEqualTo("Hello World");
server.verify();
}
+
}
The baseUrl value is prepended to all mock calls, and the stubs() method takes a stub
path resource pattern as an argument. In the preceding example, the stub defined at
/stubs/resource.json is loaded into the mock server. If the RestTemplate is asked to
@@ -148,9 +152,11 @@ Spring Boot embedded servers, and Wiremock itself has "native" support for a par
version of Jetty (currently 9.2). To use the native Jetty, you need to add the native
Wiremock dependencies and exclude the Spring Boot container (if there is one).
You can register a bean of org.springframework.cloud.contract.wiremock.WireMockConfigurationCustomizer type
in order to customize the WireMock configuration (e.g. add custom transformers).
-Example:
@Bean WireMockConfigurationCustomizer optionsCustomizer() { +Example:@Bean + WireMockConfigurationCustomizer optionsCustomizer() { return new WireMockConfigurationCustomizer() { - @Override public void customize(WireMockConfiguration options) { + @Override + public void customize(WireMockConfiguration options) { // perform your customization here } }; @@ -264,16 +270,14 @@ about a situation in which they would like to move to DSL-based contract definit but they already have a lot of Spring MVC tests. Using this feature lets you generate the contract files that you can later modify and move to folders (defined in your configuration) so that the plugin finds them.
Tip You might wonder why this functionality is in the WireMock module. The functionality -is there because it makes sense to generate both the contracts and the stubs.
Consider the following test:
this.mockMvc.perform(post("/foo") - .accept(MediaType.APPLICATION_PDF) - .accept(MediaType.APPLICATION_JSON) - .contentType(MediaType.APPLICATION_JSON) - .content("{\"foo\": 23, \"bar\" : \"baz\" }")) - .andExpect(status().isOk()) - .andExpect(content().string("bar")) +is there because it makes sense to generate both the contracts and the stubs.
Consider the following test:
this.mockMvc + .perform(post("/foo").accept(MediaType.APPLICATION_PDF) + .accept(MediaType.APPLICATION_JSON) + .contentType(MediaType.APPLICATION_JSON) + .content("{\"foo\": 23, \"bar\" : \"baz\" }")) + .andExpect(status().isOk()).andExpect(content().string("bar")) // first WireMock - .andDo(WireMockRestDocs.verify() - .jsonPath("$[?(@.foo >= 20)]") + .andDo(WireMockRestDocs.verify().jsonPath("$[?(@.foo >= 20)]") .jsonPath("$[?(@.bar in ['baz','bazz','bazzz'])]") .contentType(MediaType.valueOf("application/json")) .stub("shouldGrantABeerIfOldEnough")) diff --git a/multi/multi_stub-runner-for-messaging.html b/multi/multi_stub-runner-for-messaging.html index edf0586ed6..2dd19c55af 100644 --- a/multi/multi_stub-runner-for-messaging.html +++ b/multi/multi_stub-runner-for-messaging.html @@ -12,10 +12,10 @@ That way the only remaining framework is Spring AMQP.public interface StubTrigger { /** - * Triggers an event by a given label for a given {@code groupid:artifactid} notation. You can use only {@code artifactId} too. + * Triggers an event by a given label for a given {@code groupid:artifactid} notation. + * You can use only {@code artifactId} too. * * Feature related to messaging. - * * @return true - if managed to run a trigger */ boolean trigger(String ivyNotation, String labelName); @@ -24,7 +24,6 @@ That way the only remaining framework is Spring AMQP. boolean trigger(String labelName); @@ -33,7 +32,6 @@ That way the only remaining framework is Spring AMQP. boolean trigger(); @@ -44,6 +42,7 @@ That way the only remaining framework is Spring AMQP. Map<String, Collection<String>> labels(); + }
For convenience, the StubFinder interface extends StubTrigger, so you only need one
or the other in your tests.
StubTrigger gives you the following options to trigger a message:
stubFinder.trigger('org.springframework.cloud.contract.verifier.stubs:streamService', 'return_book_1')
Spring Cloud Contract Verifier Stub Runner’s messaging module gives you an easy way to
integrate with Spring Integration. For the provided artifacts, it automatically downloads
@@ -270,20 +269,20 @@ to disable them explicitly by setting the stubrunner.stre
follows:
stubTrigger.trigger("contract-test.person.created.event")The message has a destination of contract-test.exchange, so the Spring AMQP stub runner
integration looks for bindings related to this exchange.
@Bean public Binding binding() { - return BindingBuilder.bind(new Queue("test.queue")).to(new DirectExchange("contract-test.exchange")).with("#"); + return BindingBuilder.bind(new Queue("test.queue")) + .to(new DirectExchange("contract-test.exchange")).with("#"); }
The binding definition binds the queue test.queue. As a result, the following listener
definition is matched and invoked with the contract message.
@Bean -public SimpleMessageListenerContainer simpleMessageListenerContainer(ConnectionFactory connectionFactory, - MessageListenerAdapter listenerAdapter) { +public SimpleMessageListenerContainer simpleMessageListenerContainer( + ConnectionFactory connectionFactory, + MessageListenerAdapter listenerAdapter) { SimpleMessageListenerContainer container = new SimpleMessageListenerContainer(); container.setConnectionFactory(connectionFactory); container.setQueueNames("test.queue"); container.setMessageListener(listenerAdapter); return container; -}
Also, the following annotated listener matches and is invoked:
@RabbitListener(bindings = @QueueBinding(
- value = @Queue(value = "test.queue"),
- exchange = @Exchange(value = "contract-test.exchange", ignoreDeclarationExceptions = "true")))
+}Also, the following annotated listener matches and is invoked:
@RabbitListener(bindings = @QueueBinding(value = @Queue(value = "test.queue"), exchange = @Exchange(value = "contract-test.exchange", ignoreDeclarationExceptions = "true"))) public void handlePerson(Person person) { this.person = person; }
For convenience, the StubFinder interface extends StubTrigger, so you only need one
or the other in your tests.
StubTrigger gives you the following options to trigger a message:
stubFinder.trigger('org.springframework.cloud.contract.verifier.stubs:streamService', 'return_book_1')
Spring Cloud Contract Verifier Stub Runner’s messaging module gives you an easy way to
integrate with Spring Integration. For the provided artifacts, it automatically downloads
@@ -3329,20 +3330,20 @@ to disable them explicitly by setting the stubrunner.stre
follows:
stubTrigger.trigger("contract-test.person.created.event")The message has a destination of contract-test.exchange, so the Spring AMQP stub runner
integration looks for bindings related to this exchange.
@Bean public Binding binding() { - return BindingBuilder.bind(new Queue("test.queue")).to(new DirectExchange("contract-test.exchange")).with("#"); + return BindingBuilder.bind(new Queue("test.queue")) + .to(new DirectExchange("contract-test.exchange")).with("#"); }
The binding definition binds the queue test.queue. As a result, the following listener
definition is matched and invoked with the contract message.
@Bean -public SimpleMessageListenerContainer simpleMessageListenerContainer(ConnectionFactory connectionFactory, - MessageListenerAdapter listenerAdapter) { +public SimpleMessageListenerContainer simpleMessageListenerContainer( + ConnectionFactory connectionFactory, + MessageListenerAdapter listenerAdapter) { SimpleMessageListenerContainer container = new SimpleMessageListenerContainer(); container.setConnectionFactory(connectionFactory); container.setQueueNames("test.queue"); container.setMessageListener(listenerAdapter); return container; -}
Also, the following annotated listener matches and is invoked:
@RabbitListener(bindings = @QueueBinding(
- value = @Queue(value = "test.queue"),
- exchange = @Exchange(value = "contract-test.exchange", ignoreDeclarationExceptions = "true")))
+}Also, the following annotated listener matches and is invoked:
@RabbitListener(bindings = @QueueBinding(value = @Queue(value = "test.queue"), exchange = @Exchange(value = "contract-test.exchange", ignoreDeclarationExceptions = "true"))) public void handlePerson(Person person) { this.person = person; }
![]() | Note | ||
|---|---|---|---|
The message is directly handed over to the < @SpringBootTest(webEnvironment = WebEnvironment.RANDOM_PORT) @AutoConfigureWireMock(port = 0) public class WiremockForDocsTests { + // A service that calls out over HTTP - @Autowired private Service service; + @Autowired + private Service service; // Using the WireMock APIs in the normal way: @Test public void contextLoads() throws Exception { // Stubbing WireMock - stubFor(get(urlEqualTo("/resource")) - .willReturn(aResponse().withHeader("Content-Type", "text/plain").withBody("Hello World!"))); + stubFor(get(urlEqualTo("/resource")).willReturn(aResponse() + .withHeader("Content-Type", "text/plain").withBody("Hello World!"))); // We're asserting if WireMock responded properly assertThat(this.service.go()).isEqualTo("Hello World!"); } } -//end::wiremock_test2[] To start the stub server on a different port use (for example), +// end::wiremock_test2[] To start the stub server on a different port use (for example),
@ClassRule public static WireMockClassRule wiremock = new WireMockClassRule( WireMockSpring.options().dynamicPort()); + // A service that calls out over HTTP to localhost:${wiremock.port} @Autowired private Service service; @@ -6059,14 +6063,14 @@ instance, as shown in the following example:@Test public void contextLoads() throws Exception { // Stubbing WireMock - wiremock.stubFor(get(urlEqualTo("/resource")) - .willReturn(aResponse().withHeader("Content-Type", "text/plain").withBody("Hello World!"))); + wiremock.stubFor(get(urlEqualTo("/resource")).willReturn(aResponse() + .withHeader("Content-Type", "text/plain").withBody("Hello World!"))); // We're asserting if WireMock responded properly assertThat(this.service.go()).isEqualTo("Hello World!"); } } -//end::wiremock_test2[] The WireMock lets you stub a "secure" server with an "https" URL protocol. If your
application wants to contact that stub server in an integration test, it will find that
the SSL certificates are not valid (the usual problem with self-installed certificates).
@@ -6115,6 +6119,7 @@ a Spring The You can register a bean of @Bean WireMockConfigurationCustomizer optionsCustomizer() { +Example:@Bean + WireMockConfigurationCustomizer optionsCustomizer() { return new WireMockConfigurationCustomizer() { - @Override public void customize(WireMockConfiguration options) { + @Override + public void customize(WireMockConfiguration options) { // perform your customization here } }; @@ -6243,16 +6250,14 @@ about a situation in which they would like to move to DSL-based contract definit but they already have a lot of Spring MVC tests. Using this feature lets you generate the contract files that you can later modify and move to folders (defined in your configuration) so that the plugin finds them. |
Consider the following test:
this.mockMvc + .perform(post("/foo").accept(MediaType.APPLICATION_PDF) + .accept(MediaType.APPLICATION_JSON) + .contentType(MediaType.APPLICATION_JSON) + .content("{\"foo\": 23, \"bar\" : \"baz\" }")) + .andExpect(status().isOk()).andExpect(content().string("bar")) // first WireMock - .andDo(WireMockRestDocs.verify() - .jsonPath("$[?(@.foo >= 20)]") + .andDo(WireMockRestDocs.verify().jsonPath("$[?(@.foo >= 20)]") .jsonPath("$[?(@.bar in ['baz','bazz','bazzz'])]") .contentType(MediaType.valueOf("application/json")) .stub("shouldGrantABeerIfOldEnough")) diff --git a/spring-cloud-contract-maven-plugin/checkstyle.html b/spring-cloud-contract-maven-plugin/checkstyle.html index 31b6f60d58..88ed24d9e3 100644 --- a/spring-cloud-contract-maven-plugin/checkstyle.html +++ b/spring-cloud-contract-maven-plugin/checkstyle.html @@ -285,7 +285,7 @@12 0 0 -545
| RightCurly | -4 | +12 | ||
| coding | @@ -435,7 +435,7 @@||||
| sizes | LineLength | -216 | +231 | |
| @@ -451,16 +451,6 @@ | NoWhitespaceAfter | 1 | -||
| - | OperatorWrap | -10 | -||
| - | WhitespaceAround | -1 |
| Severity | Category | Rule | Message | Line | |||
|---|---|---|---|---|---|---|---|
| sizes | LineLength | Line is longer than 80 characters (found 81). | 40 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 86). | +42 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 120). | -42 | ||||
| whitespace | -FileTabCharacter | -File contains tab characters (this is the first instance). | +Line is longer than 80 characters (found 102). | 45 | |||
| sizes | LineLength | Line is longer than 80 characters (found 84). | 48 | ||||
| whitespace | +FileTabCharacter | +File contains tab characters (this is the first instance). | +48 | ||||
| javadoc | @@ -756,32 +734,20 @@sizes | LineLength | -Line is longer than 80 characters (found 100). | +Line is longer than 80 characters (found 90). | 55 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 82). | -57 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 89). | +Line is longer than 80 characters (found 148). | 58 | |||
| sizes | -LineLength | -Line is longer than 80 characters (found 89). | -62 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 84). | -63 | Line is longer than 80 characters (found 93). | +62 | ||
| sizes | @@ -814,82 +780,82 @@82 | ||||||
| sizes | -LineLength | -Line is longer than 80 characters (found 94). | -85 | ||||
| javadoc | JavadocVariable | Missing a Javadoc comment. | 85 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 99). | -88 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 113). | +Line is longer than 80 characters (found 90). | 89 | |||
| sizes | +LineLength | +Line is longer than 80 characters (found 87). | +90 | ||||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -95 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 102). | -99 | 96 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 83). | +Line is longer than 80 characters (found 89). | 100 | |||
| sizes | +LineLength | +Line is longer than 80 characters (found 87). | +101 | ||||
| javadoc | JavadocStyle | Extra HTML tag found: </p> | -101 | ||||
| 103 | |||||||
| sizes | LineLength | -Line is longer than 80 characters (found 125). | -102 | ||||
| Line is longer than 80 characters (found 94). | +104 | ||||||
| javadoc | JavadocStyle | First sentence should end with a period. | -108 | ||||
| 110 | |||||||
| sizes | LineLength | -Line is longer than 80 characters (found 103). | -115 | ||||
| Line is longer than 80 characters (found 88). | +117 | ||||||
| javadoc | JavadocStyle | First sentence should end with a period. | -145 | 147 | +|||
| sizes | +LineLength | +Line is longer than 80 characters (found 92). | +148 | ||||
| sizes | LineLength | Line is longer than 80 characters (found 83). | -151 | 152 | |||
| javadoc | @@ -898,373 +864,349 @@156 | ||||||
| sizes | +LineLength | +Line is longer than 80 characters (found 94). | +157 | ||||
| javadoc | JavadocStyle | First sentence should end with a period. | 163 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 133). | -164 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -169 | Line is longer than 80 characters (found 86). | +165 | ||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -172 | 170 | |||
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +173 | ||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -174 | ||||
| 175 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 85). | -175 | ||||
| 176 | |||||||
| misc | FinalParameters | Parameter aetherStubDownloaderFactory should be final. | -175 | ||||
| 176 | |||||||
| coding | HiddenField | 'aetherStubDownloaderFactory' hides a field. | -175 | ||||
| 176 | |||||||
| design | DesignForExtension | Class 'ConvertMojo' looks like designed for extension (can be subclassed), but the method 'execute' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'ConvertMojo' final or making the method 'execute' static/final/abstract/empty, or adding allowed annotation for the method. | -179 | ||||
| 180 | |||||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -179 | ||||
| 180 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 132). | -182 | ||||
| 183 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 91). | -189 | ||||
| 190 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 97). | -191 | ||||
| 192 | |||||||
| coding | HiddenField | 'contractsDirectory' hides a field. | -193 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 100). | 194 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 84). | -195 | Line is longer than 80 characters (found 102). | +196 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 104). | -196 | Line is longer than 80 characters (found 84). | +197 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 89). | -197 | Line is longer than 80 characters (found 96). | +198 | ||
| sizes | LineLength | +Line is longer than 80 characters (found 96). | +199 | ||||
| sizes | +LineLength | Line is longer than 80 characters (found 88). | -202 | ||||
| 204 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 94). | -206 | ||||
| 208 | |||||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -206 | ||||
| misc | -FinalParameters | -Parameter config should be final. | -206 | 208 | |||
| misc | FinalParameters | -Parameter contractsDslDir should be final. | -206 | Parameter config should be final. | +208 | ||
| sizes | -LineLength | -Line is longer than 80 characters (found 93). | +misc | +FinalParameters | +Parameter contractsDslDir should be final. | 208 | |
| sizes | LineLength | -Line is longer than 80 characters (found 119). | -211 | Line is longer than 80 characters (found 93). | +210 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 94). | +Line is longer than 80 characters (found 119). | 213 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 87). | +Line is longer than 80 characters (found 94). | 215 | |||
| sizes | +LineLength | +Line is longer than 80 characters (found 87). | +217 | ||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -219 | ||||
| 221 | |||||||
| misc | FinalParameters | Parameter contractsDirectory should be final. | -219 | ||||
| 221 | |||||||
| coding | HiddenField | 'contractsDirectory' hides a field. | -219 | ||||
| 221 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 84). | -220 | ||||
| 222 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 123). | -224 | ||||
| 226 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 83). | -231 | ||||
| 233 | |||||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -231 | ||||
| 233 | |||||||
| misc | FinalParameters | Parameter config should be final. | -231 | ||||
| 233 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 90). | -232 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 118). | -233 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 107). | 234 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 109). | -235 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 92). | -236 | Line is longer than 80 characters (found 100). | +235 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 111). | +Line is longer than 80 characters (found 100). | 237 | |||
| sizes | +LineLength | +Line is longer than 80 characters (found 93). | +238 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 102). | +239 | ||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -240 | 243 | |||
| misc | FinalParameters | Parameter rootPath should be final. | -240 | 243 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 118). | -241 | Line is longer than 80 characters (found 98). | +244 | ||
| coding | AvoidInlineConditionals | Avoid inline conditionals. | -241 | 244 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -244 | 248 | |||
| misc | FinalParameters | Parameter contractsDirectory should be final. | -244 | 248 | |||
| coding | HiddenField | 'contractsDirectory' hides a field. | -244 | 248 | |||
| whitespace | -OperatorWrap | -'?' should be on a new line. | -245 | ||||
| coding | AvoidInlineConditionals | Avoid inline conditionals. | -245 | ||||
| 249 | |||||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -249 |
| Severity | Category | Rule | Message | Line | |||
|---|---|---|---|---|---|---|---|
| javadoc | JavadocType | Missing a Javadoc comment. | 32 | ||||
| sizes | LineLength | Line is longer than 80 characters (found 87). | -33 | ||||
| 34 | |||||||
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | -33 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -33 | ||||
| naming | -ConstantName | -Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | -33 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | 34 | ||||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -35 | 34 | |||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -36 | naming | +ConstantName | +Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | +34 |
| javadoc | JavadocVariable | Missing a Javadoc comment. | -37 | 36 | |||
| javadoc | @@ -1274,249 +1216,261 @@|||||||
| javadoc | -JavadocMethod | +JavadocVariable | Missing a Javadoc comment. | 40 | |||
| misc | -FinalParameters | -Parameter project should be final. | -40 | ||||
| coding | -HiddenField | -'project' hides a field. | -40 | ||||
| misc | -FinalParameters | -Parameter mavenSession should be final. | -40 | ||||
| coding | -HiddenField | -'mavenSession' hides a field. | -40 | ||||
| misc | -FinalParameters | -Parameter mavenResourcesFiltering should be final. | -41 | ||||
| coding | -HiddenField | -'mavenResourcesFiltering' hides a field. | -41 | ||||
| misc | -FinalParameters | -Parameter config should be final. | +javadoc | +JavadocVariable | +Missing a Javadoc comment. | 42 | |
| coding | -HiddenField | -'config' hides a field. | -42 | javadoc | +JavadocVariable | +Missing a Javadoc comment. | +44 |
| sizes | -LineLength | -Line is longer than 80 characters (found 88). | -49 | ||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -49 | 46 | +|||
| misc | +FinalParameters | +Parameter project should be final. | +46 | ||||
| coding | +HiddenField | +'project' hides a field. | +46 | ||||
| misc | +FinalParameters | +Parameter mavenSession should be final. | +46 | ||||
| coding | +HiddenField | +'mavenSession' hides a field. | +46 | ||||
| misc | +FinalParameters | +Parameter mavenResourcesFiltering should be final. | +47 | ||||
| coding | +HiddenField | +'mavenResourcesFiltering' hides a field. | +47 | ||||
| misc | +FinalParameters | +Parameter config should be final. | +48 | ||||
| coding | +HiddenField | +'config' hides a field. | +48 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 88). | +55 | ||||
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +55 | ||||
| misc | FinalParameters | Parameter contractsDirectory should be final. | -49 | ||||
| 55 | |||||||
| misc | FinalParameters | Parameter outputDirectory should be final. | -49 | ||||
| 55 | |||||||
| misc | FinalParameters | Parameter rootPath should be final. | -49 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 98). | -51 | ||||
| whitespace | -OperatorWrap | -'?' should be on a new line. | -51 | ||||
| coding | -AvoidInlineConditionals | -Avoid inline conditionals. | -51 | 55 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 103). | -52 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 113). | -53 | ||||
| whitespace | -WhitespaceAround | -'+' is not preceded with whitespace. | -53 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 126). | -54 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 108). | +Line is longer than 80 characters (found 96). | 57 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 94). | -59 | Line is longer than 80 characters (found 105). | +58 | +||
| coding | +AvoidInlineConditionals | +Avoid inline conditionals. | +58 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 93). | +60 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 92). | +Line is longer than 80 characters (found 98). | 61 | |||
| sizes | LineLength | +Line is longer than 80 characters (found 89). | +66 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 85). | +68 | ||||
| sizes | +LineLength | Line is longer than 80 characters (found 93). | -64 | 72 | |||
| sizes | LineLength | Line is longer than 80 characters (found 82). | -73 | 81 | |||
| blocks | RightCurly | '}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | -85 | 93 | |||
| sizes | LineLength | Line is longer than 80 characters (found 93). | -91 | 99 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -91 | 99 | |||
| misc | FinalParameters | Parameter includedRootFolderAntPattern should be final. | -91 | 99 | |||
| sizes | LineLength | Line is longer than 80 characters (found 85). | -92 | 100 | +|||
| blocks | +RightCurly | +'}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | +102 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 88). | +103 | ||||
| sizes | LineLength | Line is longer than 80 characters (found 90). | -94 | 104 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 116). | -95 | ||||
| sizes | -LineLength | Line is longer than 80 characters (found 91). | -100 | ||||
| 110 | |||||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -100 | ||||
| 110 | |||||||
| misc | FinalParameters | Parameter includedRootFolderAntPattern should be final. | -100 | ||||
| 110 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 83). | -101 | 111 | +|||
| blocks | +RightCurly | +'}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | +113 | ||||
| sizes | LineLength | +Line is longer than 80 characters (found 90). | +114 | ||||
| sizes | +LineLength | Line is longer than 80 characters (found 92). | -103 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 116). | -104 | 115 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -109 | 121 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -113 |
| sizes | LineLength | -Line is longer than 80 characters (found 91). | +Line is longer than 80 characters (found 85). | 36 |
| sizes | +LineLength | +Line is longer than 80 characters (found 92). | +39 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 97). | +42 | |
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | -39 | 42 |
| javadoc | @@ -1546,27 +1512,270 @@42 | |||
| sizes | +LineLength | +Line is longer than 80 characters (found 98). | +45 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | -46 | |
| 45 | ||||
| javadoc | JavadocStyle | First sentence should end with a period. | -50 | |
| 48 | ||||
| sizes | LineLength | Line is longer than 80 characters (found 92). | -53 | 51 | +
| javadoc | +JavadocStyle | +First sentence should end with a period. | +54 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 96). | +57 | |
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +60 | |
| javadoc | JavadocStyle | First sentence should end with a period. | +63 | |
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +69 | |
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +72 | |
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +75 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 83). | +78 | |
| design | +DesignForExtension | +Class 'GenerateStubsMojo' looks like designed for extension (can be subclassed), but the method 'execute' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GenerateStubsMojo' final or making the method 'execute' static/final/abstract/empty, or adding allowed annotation for the method. | +78 | |
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +78 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 129). | +81 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 114). | +82 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 87). | +87 | |
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +91 | |
| misc | +FinalParameters | +Parameter stubsOutputDir should be final. | +91 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 86). | +94 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 114). | +96 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 85). | +98 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 90). | +100 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 82). | +102 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 83). | +103 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 93). | +105 | |
| whitespace | +NoWhitespaceAfter | +'{' is followed by whitespace. | +105 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 99). | +106 | |
| coding | +AvoidInlineConditionals | +Avoid inline conditionals. | +106 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 109). | +110 | |
| blocks | +RightCurly | +'}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | +112 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 101). | +115 | |
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +120 | |
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +130 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 84). | +131 |
| Severity | +Category | +Rule | +Message | +Line |
|---|---|---|---|---|
| javadoc | +JavadocPackage | +Missing package-info.java file. | +||
| javadoc | +JavadocStyle | +First sentence should end with a period. | +46 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 83). | +47 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 135). | +50 | |
| whitespace | +FileTabCharacter | +File contains tab characters (this is the first instance). | +53 | |
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +53 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 148). | +56 | |
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | 56 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | -62 | |
| javadoc | -JavadocStyle | -First sentence should end with a period. | -65 | |
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -71 | 59 |
| javadoc | JavadocVariable | Missing a Javadoc comment. | -74 | |
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -77 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 83). | -80 | |
| design | -DesignForExtension | -Class 'GenerateStubsMojo' looks like designed for extension (can be subclassed), but the method 'execute' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GenerateStubsMojo' final or making the method 'execute' static/final/abstract/empty, or adding allowed annotation for the method. | -80 | |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -80 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 129). | -83 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 131). | -84 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 102). | -88 | |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -91 | |
| misc | -FinalParameters | -Parameter stubsOutputDir should be final. | -91 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 104). | -95 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 130). | -96 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 104). | -99 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 90). | -100 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 82). | -102 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 83). | -103 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 93). | -105 | |
| whitespace | -NoWhitespaceAfter | -'{' is followed by whitespace. | -105 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 99). | -106 | |
| coding | -AvoidInlineConditionals | -Avoid inline conditionals. | -106 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 106). | -109 | |
| blocks | -RightCurly | -'}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | -111 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 101). | -114 | |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -119 | |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -129 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 84). | -130 |
| Severity | -Category | -Rule | -Message | -Line | |||
|---|---|---|---|---|---|---|---|
| javadoc | -JavadocPackage | -Missing package-info.java file. | -|||||
| javadoc | -JavadocStyle | -First sentence should end with a period. | -46 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 82). | -50 | ||||
| whitespace | -FileTabCharacter | -File contains tab characters (this is the first instance). | -51 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -54 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 82). | -57 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -57 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 89). | -58 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -61 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 101). | 62 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | +JavadocStyle | +First sentence should end with a period. | 80 | ||
| javadoc | JavadocStyle | First sentence should end with a period. | -83 | 86 | |||
| javadoc | JavadocStyle | First sentence should end with a period. | -89 | 92 | |||
| javadoc | JavadocStyle | First sentence should end with a period. | -95 | 98 | |||
| javadoc | -JavadocStyle | -First sentence should end with a period. | -101 | ||||
| sizes | LineLength | Line is longer than 80 characters (found 86). | +105 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 99). | 108 | ||||
| javadoc | JavadocStyle | First sentence should end with a period. | -115 | 111 | |||
| sizes | LineLength | Line is longer than 80 characters (found 91). | -116 | 112 | |||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -121 | 117 | |||
| sizes | LineLength | Line is longer than 80 characters (found 92). | -124 | 120 | |||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -124 | 120 | |||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -127 | 123 | |||
| sizes | -LineLength | -Line is longer than 80 characters (found 93). | -130 | ||||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -130 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 113). | -134 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -140 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 102). | -144 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 83). | -145 | ||||
| javadoc | -JavadocStyle | -Extra HTML tag found: </p> | -146 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 125). | -147 | ||||
| javadoc | -JavadocStyle | -First sentence should end with a period. | -153 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 114). | -160 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 114). | -161 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 107). | -162 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 109). | -163 | 126 | |||
| sizes | LineLength | Line is longer than 80 characters (found 84). | -164 | 130 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 122). | -170 | Line is longer than 80 characters (found 88). | +131 | ||
| javadoc | -JavadocStyle | -Extra HTML tag found: </p> | -172 | ||||
| javadoc | -JavadocStyle | -Extra HTML tag found: </p> | -174 | ||||
| javadoc | -JavadocStyle | -Extra HTML tag found: </p> | -176 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 113). | -177 | ||||
| javadoc | -JavadocStyle | -First sentence should end with a period. | -207 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 83). | -213 | ||||
| javadoc | -JavadocStyle | -First sentence should end with a period. | -217 | ||||
| javadoc | -JavadocStyle | -First sentence should end with a period. | -224 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 133). | -225 | ||||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -230 | 137 | +|||
| sizes | +LineLength | +Line is longer than 80 characters (found 89). | +141 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 87). | +142 | ||||
| javadoc | +JavadocStyle | +Extra HTML tag found: </p> | +144 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 94). | +145 | ||||
| javadoc | +JavadocStyle | +First sentence should end with a period. | +151 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 92). | +158 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 93). | +159 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 93). | +161 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 89). | +163 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 93). | +170 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 89). | +171 | ||||
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -232 | JavadocStyle | +Extra HTML tag found: </p> | +173 | +|
| javadoc | +JavadocStyle | +Extra HTML tag found: </p> | +175 | ||||
| javadoc | +JavadocStyle | +Extra HTML tag found: </p> | +177 | ||||
| sizes | LineLength | Line is longer than 80 characters (found 91). | -233 | 178 | |||
| misc | -FinalParameters | -Parameter aetherStubDownloaderFactory should be final. | -233 | javadoc | +JavadocStyle | +First sentence should end with a period. | +209 |
| coding | -HiddenField | -'aetherStubDownloaderFactory' hides a field. | -233 | sizes | +LineLength | +Line is longer than 80 characters (found 92). | +210 |
| sizes | LineLength | Line is longer than 80 characters (found 83). | -237 | 214 | +|||
| javadoc | +JavadocStyle | +First sentence should end with a period. | +218 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 94). | +219 | ||||
| javadoc | +JavadocStyle | +First sentence should end with a period. | +225 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 86). | +227 | ||||
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +232 | ||||
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +234 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 91). | +235 | ||||
| misc | +FinalParameters | +Parameter aetherStubDownloaderFactory should be final. | +235 | ||||
| coding | +HiddenField | +'aetherStubDownloaderFactory' hides a field. | +235 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 83). | +239 | ||||
| design | DesignForExtension | Class 'GenerateTestsMojo' looks like designed for extension (can be subclassed), but the method 'execute' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GenerateTestsMojo' final or making the method 'execute' static/final/abstract/empty, or adding allowed annotation for the method. | -237 | 239 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -237 | ||||
| blocks | -NeedBraces | -'if' construct must use '{}'s. | 239 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 156). | -239 | ||||
| blocks | -NeedBraces | -'if' construct must use '{}'s. | -240 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 154). | -240 | ||||
| blocks | @@ -2145,278 +2111,326 @@sizes | LineLength | -Line is longer than 80 characters (found 139). | -241 | Line is longer than 80 characters (found 137). | +243 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 128). | +blocks | +NeedBraces | +'if' construct must use '{}'s. | 245 | |
| sizes | LineLength | -Line is longer than 80 characters (found 103). | -246 | Line is longer than 80 characters (found 117). | +247 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 109). | +Line is longer than 80 characters (found 86). | 248 | |||
| coding | -HiddenField | -'contractsDirectory' hides a field. | -248 | blocks | +NeedBraces | +'if' construct must use '{}'s. | +249 |
| sizes | LineLength | Line is longer than 80 characters (found 110). | -249 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 99). | -250 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 101). | 251 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 155). | +Line is longer than 80 characters (found 82). | 252 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 100). | -253 | Line is longer than 80 characters (found 128). | +256 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 102). | -255 | Line is longer than 80 characters (found 103). | +257 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 114). | -258 | Line is longer than 80 characters (found 84). | +259 | +||
| coding | +HiddenField | +'contractsDirectory' hides a field. | +259 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 105). | +260 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 128). | +Line is longer than 80 characters (found 95). | 261 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 124). | +Line is longer than 80 characters (found 100). | 262 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 105). | +Line is longer than 80 characters (found 93). | 263 | |||
| sizes | LineLength | +Line is longer than 80 characters (found 102). | +264 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 102). | +267 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 106). | +270 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 102). | +273 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 88). | +276 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 101). | +277 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 101). | +278 | ||||
| sizes | +LineLength | Line is longer than 80 characters (found 90). | -268 | 284 | +|||
| blocks | +RightCurly | +'}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | +285 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 108). | +288 | ||||
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +294 | ||||
| misc | +FinalParameters | +Parameter config should be final. | +294 | ||||
| misc | +FinalParameters | +Parameter contractsDirectory should be final. | +295 | ||||
| coding | +HiddenField | +'contractsDirectory' hides a field. | +295 | ||||
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +316 | ||||
| misc | +FinalParameters | +Parameter contractsDirectory should be final. | +316 | ||||
| coding | +HiddenField | +'contractsDirectory' hides a field. | +316 | ||||
| blocks | RightCurly | '}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | -269 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 108). | -272 | ||||
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -277 | ||||
| misc | -FinalParameters | -Parameter config should be final. | -277 | ||||
| misc | -FinalParameters | -Parameter contractsDirectory should be final. | -278 | ||||
| coding | -HiddenField | -'contractsDirectory' hides a field. | -278 | ||||
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -299 | ||||
| misc | -FinalParameters | -Parameter contractsDirectory should be final. | -299 | ||||
| coding | -HiddenField | -'contractsDirectory' hides a field. | -299 | 319 | |||
| design | DesignForExtension | Class 'GenerateTestsMojo' looks like designed for extension (can be subclassed), but the method 'mappingsToMap' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GenerateTestsMojo' final or making the method 'mappingsToMap' static/final/abstract/empty, or adding allowed annotation for the method. | -307 | 325 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -307 | 325 | |||
| sizes | LineLength | Line is longer than 80 characters (found 94). | -313 | 331 | |||
| design | DesignForExtension | Class 'GenerateTestsMojo' looks like designed for extension (can be subclassed), but the method 'getExcludedFiles' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GenerateTestsMojo' final or making the method 'getExcludedFiles' static/final/abstract/empty, or adding allowed annotation for the method. | -318 | 336 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -318 | 336 | |||
| design | DesignForExtension | Class 'GenerateTestsMojo' looks like designed for extension (can be subclassed), but the method 'setExcludedFiles' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GenerateTestsMojo' final or making the method 'setExcludedFiles' static/final/abstract/empty, or adding allowed annotation for the method. | -322 | 340 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -322 | 340 | |||
| misc | FinalParameters | Parameter excludedFiles should be final. | -322 | 340 | |||
| coding | HiddenField | 'excludedFiles' hides a field. | -322 | 340 | |||
| design | DesignForExtension | Class 'GenerateTestsMojo' looks like designed for extension (can be subclassed), but the method 'getIgnoredFiles' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GenerateTestsMojo' final or making the method 'getIgnoredFiles' static/final/abstract/empty, or adding allowed annotation for the method. | -326 | 344 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -326 | 344 | |||
| design | DesignForExtension | Class 'GenerateTestsMojo' looks like designed for extension (can be subclassed), but the method 'setIgnoredFiles' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GenerateTestsMojo' final or making the method 'setIgnoredFiles' static/final/abstract/empty, or adding allowed annotation for the method. | -330 | 348 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -330 | 348 | |||
| misc | FinalParameters | Parameter ignoredFiles should be final. | -330 | 348 | |||
| coding | HiddenField | 'ignoredFiles' hides a field. | -330 | 348 | |||
| design | DesignForExtension | Class 'GenerateTestsMojo' looks like designed for extension (can be subclassed), but the method 'isAssertJsonSize' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GenerateTestsMojo' final or making the method 'isAssertJsonSize' static/final/abstract/empty, or adding allowed annotation for the method. | -334 | 352 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -334 | 352 | |||
| design | DesignForExtension | Class 'GenerateTestsMojo' looks like designed for extension (can be subclassed), but the method 'setAssertJsonSize' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'GenerateTestsMojo' final or making the method 'setAssertJsonSize' static/final/abstract/empty, or adding allowed annotation for the method. | -338 | 356 | |||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -338 | 356 | |||
| misc | FinalParameters | Parameter assertJsonSize should be final. | -338 | 356 | |||
| coding | HiddenField | 'assertJsonSize' hides a field. | -338 |
| sizes | LineLength | Line is longer than 80 characters (found 94). | -27 | +28 |
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | -27 | 28 |
| javadoc | JavadocMethod | Missing a Javadoc comment. | -27 | 28 |
| misc | FinalParameters | Parameter project should be final. | -27 | 28 |
| sizes | LineLength | Line is longer than 80 characters (found 88). | -29 | 30 |
| sizes | LineLength | -Line is longer than 80 characters (found 121). | -32 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 102). | -34 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 118). | -35 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 89). | -37 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 128). | -38 | |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -45 | |
| misc | -FinalParameters | -Parameter plugins should be final. | -45 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 98). | -47 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 81). | -54 | |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -54 | |
| misc | -FinalParameters | -Parameter deps should be final. | -54 | Line is longer than 80 characters (found 108). | +33 |
| sizes | LineLength | Line is longer than 80 characters (found 91). | -56 |
| Severity | Category | Rule | Message | Line | |||
|---|---|---|---|---|---|---|---|
| javadoc | JavadocStyle | First sentence should end with a period. | 35 | ||||
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 43 | ||||
| javadoc | JavadocVariable | Missing a Javadoc comment. | 43 | ||||
| sizes | LineLength | Line is longer than 80 characters (found 85). | -44 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -44 | 45 | |||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -46 | 45 | |||
| javadoc | @@ -2596,19 +2610,7 @@javadoc | JavadocVariable | Missing a Javadoc comment. | -48 | |||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | 49 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -50 | ||||
| javadoc | @@ -2617,18 +2619,6 @@51 | ||||||
| sizes | -LineLength | -Line is longer than 80 characters (found 82). | -52 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -52 | ||||
| javadoc | JavadocVariable | Missing a Javadoc comment. | @@ -2638,19 +2628,7 @@javadoc | JavadocVariable | Missing a Javadoc comment. | -54 | |
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | 55 | ||||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -56 | ||||
| javadoc | @@ -2659,352 +2637,328 @@57 | ||||||
| sizes | +LineLength | +Line is longer than 80 characters (found 82). | +59 | ||||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -58 | 59 | +|||
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +61 | ||||
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +63 | ||||
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +65 | ||||
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +67 | ||||
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +69 | ||||
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +71 | ||||
| sizes | LineLength | Line is longer than 80 characters (found 85). | -60 | ||||
| 73 | |||||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -60 | ||||
| 73 | |||||||
| sizes | ParameterNumber | More than 7 parameters (found 12). | -60 | ||||
| 73 | |||||||
| misc | FinalParameters | Parameter project should be final. | -60 | ||||
| 73 | |||||||
| coding | HiddenField | 'project' hides a field. | -60 | ||||
| 73 | |||||||
| misc | FinalParameters | Parameter contractDependency should be final. | -60 | ||||
| 73 | |||||||
| coding | HiddenField | 'contractDependency' hides a field. | -60 | ||||
| 73 | |||||||
| misc | FinalParameters | Parameter contractsPath should be final. | -61 | ||||
| 74 | |||||||
| coding | HiddenField | 'contractsPath' hides a field. | -61 | ||||
| 74 | |||||||
| misc | FinalParameters | Parameter contractsRepositoryUrl should be final. | -61 | ||||
| 74 | |||||||
| coding | HiddenField | 'contractsRepositoryUrl' hides a field. | -61 | ||||
| 74 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 101). | -62 | ||||
| 75 | |||||||
| misc | FinalParameters | Parameter stubsMode should be final. | -62 | ||||
| 75 | |||||||
| coding | HiddenField | 'stubsMode' hides a field. | -62 | ||||
| 75 | |||||||
| misc | FinalParameters | Parameter log should be final. | -62 | ||||
| 75 | |||||||
| coding | HiddenField | 'log' hides a field. | -62 | ||||
| 75 | |||||||
| misc | FinalParameters | Parameter repositoryUsername should be final. | -62 | ||||
| 75 | |||||||
| coding | HiddenField | 'repositoryUsername' hides a field. | -62 | ||||
| 75 | |||||||
| misc | FinalParameters | Parameter repositoryPassword should be final. | -63 | ||||
| 76 | |||||||
| coding | HiddenField | 'repositoryPassword' hides a field. | -63 | ||||
| 76 | |||||||
| misc | FinalParameters | Parameter repositoryProxyHost should be final. | -63 | ||||
| 76 | |||||||
| coding | HiddenField | 'repositoryProxyHost' hides a field. | -63 | ||||
| 76 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 82). | -64 | ||||
| 77 | |||||||
| misc | FinalParameters | Parameter repositoryProxyPort should be final. | -64 | ||||
| 77 | |||||||
| coding | HiddenField | 'repositoryProxyPort' hides a field. | -64 | ||||
| 77 | |||||||
| misc | FinalParameters | Parameter deleteStubsAfterTest should be final. | -64 | ||||
| 77 | |||||||
| coding | HiddenField | 'deleteStubsAfterTest' hides a field. | -64 | ||||
| 77 | |||||||
| misc | FinalParameters | Parameter contractsProperties should be final. | -65 | ||||
| 78 | |||||||
| coding | HiddenField | 'contractsProperties' hides a field. | -65 | ||||
| 78 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 89). | -76 | ||||
| 89 | |||||||
| sizes | LineLength | -Line is longer than 80 characters (found 118). | -81 | ||||
| Line is longer than 80 characters (found 90). | +94 | ||||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -81 | ||||
| 94 | |||||||
| misc | FinalParameters | Parameter config should be final. | -81 | ||||
| 94 | |||||||
| misc | FinalParameters | Parameter defaultContractsDir should be final. | -81 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 113). | -82 | 95 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 89). | -83 | Line is longer than 80 characters (found 87). | +98 | ||
| whitespace | -OperatorWrap | -'?' should be on a new line. | -83 | ||||
| coding | AvoidInlineConditionals | Avoid inline conditionals. | -83 | ||||
| 99 | |||||||
| sizes | LineLength | Line is longer than 80 characters (found 88). | -86 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 139). | -87 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 107). | -88 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 122). | -91 | 101 | |||
| sizes | LineLength | Line is longer than 80 characters (found 108). | -92 | 103 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 130). | -93 | Line is longer than 80 characters (found 88). | +104 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 105). | -96 | ||||
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -100 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 121). | -101 | ||||
| whitespace | -OperatorWrap | -'||' should be on a new line. | -101 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 81). | -102 | ||||
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | +Line is longer than 80 characters (found 98). | 105 | |||
| sizes | -LineLength | -Line is longer than 80 characters (found 84). | -106 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 108). | -107 | blocks | +RightCurly | +'}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | +108 |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | +sizes | +LineLength | +Line is longer than 80 characters (found 124). | 111 | |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -117 | sizes | +LineLength | +Line is longer than 80 characters (found 90). | +114 |
| sizes | LineLength | -Line is longer than 80 characters (found 81). | +Line is longer than 80 characters (found 97). | 118 | |||
| sizes | -LineLength | -Line is longer than 80 characters (found 81). | -119 | javadoc | +JavadocMethod | +Missing a Javadoc comment. | +123 |
| sizes | LineLength | -Line is longer than 80 characters (found 84). | -123 | Line is longer than 80 characters (found 95). | +125 | ||
| sizes | @@ -3013,46 +2967,106 @@126 | ||||||
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +129 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 94). | -129 | Line is longer than 80 characters (found 84). | +130 | +||
| sizes | +LineLength | +Line is longer than 80 characters (found 89). | +132 | ||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -134 | 135 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 92). | +Line is longer than 80 characters (found 81). | 137 | |||
| whitespace | -OperatorWrap | -'?' should be on a new line. | -137 | javadoc | +JavadocMethod | +Missing a Javadoc comment. | +140 |
| sizes | +LineLength | +Line is longer than 80 characters (found 81). | +141 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 81). | +142 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 100). | +143 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 84). | +145 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 84). | +148 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 94). | +151 | ||||
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +156 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 90). | +159 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 88). | +160 | ||||
| coding | AvoidInlineConditionals | Avoid inline conditionals. | -137 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 86). | -138 | 160 | |||
| sizes | LineLength | Line is longer than 80 characters (found 87). | -140 |
| Line | |||||||
|---|---|---|---|---|---|---|---|
| whitespace | -FileTabCharacter | -File contains tab characters (this is the first instance). | +sizes | +LineLength | +Line is longer than 80 characters (found 97). | 40 | |
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | +whitespace | +FileTabCharacter | +File contains tab characters (this is the first instance). | 40 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | -44 | 40 | +|||
| sizes | +LineLength | +Line is longer than 80 characters (found 98). | +43 | ||||
| javadoc | +JavadocVariable | +Missing a Javadoc comment. | +43 | ||||
| javadoc | JavadocStyle | First sentence should end with a period. | -48 | 46 | |||
| sizes | LineLength | Line is longer than 80 characters (found 92). | -51 | 49 | |||
| javadoc | JavadocStyle | First sentence should end with a period. | -54 | 52 | |||
| sizes | LineLength | Line is longer than 80 characters (found 113). | -57 | 55 | |||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -60 | 58 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 113). | -77 | ||||
| javadoc | -JavadocStyle | -First sentence should end with a period. | -83 | ||||
| javadoc | -JavadocStyle | -First sentence should end with a period. | -90 | ||||
| javadoc | -JavadocStyle | -First sentence should end with a period. | -97 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 133). | -98 | ||||
| design | -DesignForExtension | -Class 'PushStubsToScmMojo' looks like designed for extension (can be subclassed), but the method 'execute' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'PushStubsToScmMojo' final or making the method 'execute' static/final/abstract/empty, or adding allowed annotation for the method. | -103 | ||||
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -103 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 129). | -106 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 149). | -107 | ||||
| whitespace | -OperatorWrap | -'||' should be on a new line. | -110 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 108). | -111 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 160). | -112 | ||||
| sizes | -LineLength | -Line is longer than 80 characters (found 134). | -115 | Line is longer than 80 characters (found 84). | +74 | ||
| sizes | LineLength | Line is longer than 80 characters (found 88). | -116 | 75 | +|||
| javadoc | +JavadocStyle | +First sentence should end with a period. | +81 | ||||
| javadoc | +JavadocStyle | +First sentence should end with a period. | +87 | ||||
| sizes | LineLength | -Line is longer than 80 characters (found 125). | -117 | Line is longer than 80 characters (found 94). | +88 | +||
| javadoc | +JavadocStyle | +First sentence should end with a period. | +94 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 86). | +96 | ||||
| design | DesignForExtension | -Class 'PushStubsToScmMojo' looks like designed for extension (can be subclassed), but the method 'buildOptions' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'PushStubsToScmMojo' final or making the method 'buildOptions' static/final/abstract/empty, or adding allowed annotation for the method. | -120 | Class 'PushStubsToScmMojo' looks like designed for extension (can be subclassed), but the method 'execute' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'PushStubsToScmMojo' final or making the method 'execute' static/final/abstract/empty, or adding allowed annotation for the method. | +101 | ||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -120 | 101 | |||
| sizes | LineLength | -Line is longer than 80 characters (found 81). | -121 | Line is longer than 80 characters (found 129). | +104 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 81). | -122 | Line is longer than 80 characters (found 119). | +106 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 84). | +Line is longer than 80 characters (found 97). | +110 | |||
| sizes | +LineLength | +Line is longer than 80 characters (found 83). | +111 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 162). | +113 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 97). | +117 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 88). | +118 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 93). | +119 | ||||
| design | +DesignForExtension | +Class 'PushStubsToScmMojo' looks like designed for extension (can be subclassed), but the method 'buildOptions' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'PushStubsToScmMojo' final or making the method 'buildOptions' static/final/abstract/empty, or adding allowed annotation for the method. | 123 | ||||
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +123 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 81). | +124 | ||||
| sizes | +LineLength | +Line is longer than 80 characters (found 81). | +125 | ||||
| sizes | LineLength | Line is longer than 80 characters (found 84). | -127 |
| sizes | LineLength | -Line is longer than 80 characters (found 119). | -118 | +Line is longer than 80 characters (found 107). | +119 |
| sizes | LineLength | Line is longer than 80 characters (found 88). | -122 | 124 |
| sizes | LineLength | -Line is longer than 80 characters (found 85). | -126 | Line is longer than 80 characters (found 94). | +128 |
| sizes | LineLength | -Line is longer than 80 characters (found 125). | -128 | Line is longer than 80 characters (found 97). | +130 |
| sizes | LineLength | Line is longer than 80 characters (found 97). | -129 | 131 |
| sizes | -LineLength | -Line is longer than 80 characters (found 83). | -133 | blocks | +RightCurly | +'}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | +132 |
| sizes | LineLength | -Line is longer than 80 characters (found 95). | -135 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 103). | -142 | |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -147 | |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -155 | |
| javadoc | -JavadocMethod | -Missing a Javadoc comment. | -166 |
| Severity | -Category | -Rule | -Message | -Line |
|---|---|---|---|---|
| javadoc | -JavadocType | -Missing a Javadoc comment. | -34 | Line is longer than 80 characters (found 88). | +134 |
| sizes | LineLength | Line is longer than 80 characters (found 92). | -37 | 135 |
| sizes | +LineLength | +Line is longer than 80 characters (found 95). | +136 | |
| blocks | +RightCurly | +'}' at column 4 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | +142 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 103). | +144 | |
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +149 | |
| blocks | +RightCurly | +'}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | +152 | |
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +158 | |
| blocks | +RightCurly | +'}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | +165 | |
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +170 |
| Severity | +Category | +Rule | +Message | +Line | |
|---|---|---|---|---|---|
| javadoc | +JavadocType | +Missing a Javadoc comment. | +34 | ||
| sizes | +LineLength | +Line is longer than 80 characters (found 92). | +38 | ||
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | -37 | ||
| 38 | |||||
| javadoc | JavadocVariable | Missing a Javadoc comment. | -37 | ||
| 38 | |||||
| naming | ConstantName | Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | -37 | ||
| javadoc | -JavadocVariable | -Missing a Javadoc comment. | -39 | 38 | |
| javadoc | @@ -3545,99 +3607,111 @@|||||
| javadoc | -JavadocMethod | +JavadocVariable | Missing a Javadoc comment. | 42 | |
| misc | -FinalParameters | -Parameter repoSystem should be final. | -43 | ||
| coding | -HiddenField | -'repoSystem' hides a field. | -43 | ||
| misc | -FinalParameters | -Parameter project should be final. | -44 | ||
| coding | -HiddenField | -'project' hides a field. | -44 | ||
| sizes | -LineLength | -Line is longer than 80 characters (found 87). | -49 | ||
| design | -DesignForExtension | -Class 'AetherStubDownloaderFactory' looks like designed for extension (can be subclassed), but the method 'build' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'AetherStubDownloaderFactory' final or making the method 'build' static/final/abstract/empty, or adding allowed annotation for the method. | -49 | ||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -49 | 44 | +|
| misc | +FinalParameters | +Parameter repoSystem should be final. | +45 | ||
| coding | +HiddenField | +'repoSystem' hides a field. | +45 | ||
| misc | +FinalParameters | +Parameter project should be final. | +46 | ||
| coding | +HiddenField | +'project' hides a field. | +46 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 100). | +Line is longer than 80 characters (found 87). | 51 | |
| design | +DesignForExtension | +Class 'AetherStubDownloaderFactory' looks like designed for extension (can be subclassed), but the method 'build' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'AetherStubDownloaderFactory' final or making the method 'build' static/final/abstract/empty, or adding allowed annotation for the method. | +51 | ||
| javadoc | +JavadocMethod | +Missing a Javadoc comment. | +51 | ||
| sizes | +LineLength | +Line is longer than 80 characters (found 90). | +54 | ||
| misc | FinalParameters | Parameter stubRunnerOptions should be final. | -51 | ||
| sizes | -LineLength | -Line is longer than 80 characters (found 113). | -52 | ||
| sizes | -LineLength | -Line is longer than 80 characters (found 108). | -53 | ||
| sizes | -LineLength | -Line is longer than 80 characters (found 134). | 54 | ||
| sizes | LineLength | -Line is longer than 80 characters (found 97). | +Line is longer than 80 characters (found 120). | +56 | |
| sizes | +LineLength | +Line is longer than 80 characters (found 92). | 58 | ||
| sizes | +LineLength | +Line is longer than 80 characters (found 88). | +59 | ||
| sizes | +LineLength | +Line is longer than 80 characters (found 96). | +60 | ||
| sizes | +LineLength | +Line is longer than 80 characters (found 97). | +65 | ||
| misc | FinalParameters | Parameter location should be final. | -58 | 65 | |
| misc | FinalParameters | Parameter resourceLoader should be final. | -58 |
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | -28 | +29 |
| javadoc | JavadocVariable | Missing a Javadoc comment. | -28 | 29 |
| naming | ConstantName | Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | -28 | 29 |
| sizes | LineLength | Line is longer than 80 characters (found 85). | -30 | 31 |
| design | DesignForExtension | Class 'LocalStubRunner' looks like designed for extension (can be subclassed), but the method 'run' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'LocalStubRunner' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | -30 | 31 |
| javadoc | JavadocMethod | Missing a Javadoc comment. | -30 | 31 |
| misc | FinalParameters | Parameter options should be final. | -30 | 31 |
| sizes | LineLength | Line is longer than 80 characters (found 85). | -31 |
| sizes | LineLength | Line is longer than 80 characters (found 81). | -32 | +33 |
| whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | -32 | 33 |
| javadoc | JavadocVariable | Missing a Javadoc comment. | -32 | 33 |
| naming | ConstantName | Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | -32 | 33 |
| javadoc | JavadocVariable | Missing a Javadoc comment. | -34 | 35 |
| javadoc | JavadocMethod | Missing a Javadoc comment. | -36 | 37 |
| sizes | LineLength | Line is longer than 80 characters (found 90). | -37 | 38 |
| misc | FinalParameters | Parameter aetherStubDownloaderFactory should be final. | -37 | 38 |
| coding | HiddenField | 'aetherStubDownloaderFactory' hides a field. | -37 | 38 |
| sizes | -LineLength | -Line is longer than 80 characters (found 112). | -41 | |
| design | DesignForExtension | Class 'RemoteStubRunner' looks like designed for extension (can be subclassed), but the method 'run' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'RemoteStubRunner' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | -41 | |
| 42 | ||||
| javadoc | JavadocMethod | Missing a Javadoc comment. | -41 | |
| 42 | ||||
| misc | FinalParameters | Parameter options should be final. | -41 | |
| 42 | ||||
| misc | FinalParameters | Parameter repositorySystemSession should be final. | -41 | |
| sizes | -LineLength | -Line is longer than 80 characters (found 127). | -42 | 43 |
| sizes | LineLength | Line is longer than 80 characters (found 88). | -45 | 48 |
| sizes | LineLength | Line is longer than 80 characters (found 88). | -47 | 50 |
| sizes | LineLength | Line is longer than 80 characters (found 82). | -49 | 52 |
| blocks | RightCurly | '}' at column 3 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally). | -52 | 55 |
| sizes | LineLength | -Line is longer than 80 characters (found 112). | -54 |