Skip to content

Commit 3b4e97b

Browse files
committed
build(sonar): Fix some sonar issues
1 parent a80a695 commit 3b4e97b

File tree

18 files changed

+97
-80
lines changed

18 files changed

+97
-80
lines changed

async/async-kafka/src/main/java/org/reactivecommons/async/kafka/KafkaDirectAsyncGateway.java

Lines changed: 16 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -8,68 +8,71 @@
88
import reactor.core.publisher.Mono;
99

1010
public class KafkaDirectAsyncGateway implements DirectAsyncGateway {
11+
12+
public static final String NOT_IMPLEMENTED_YET = "Not implemented yet";
13+
1114
@Override
1215
public <T> Mono<Void> sendCommand(Command<T> command, String targetName) {
13-
throw new UnsupportedOperationException("Not implemented yet");
16+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
1417
}
1518

1619
@Override
1720
public <T> Mono<Void> sendCommand(Command<T> command, String targetName, long delayMillis) {
18-
throw new UnsupportedOperationException("Not implemented yet");
21+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
1922
}
2023

2124
@Override
2225
public <T> Mono<Void> sendCommand(Command<T> command, String targetName, String domain) {
23-
throw new UnsupportedOperationException("Not implemented yet");
26+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
2427
}
2528

2629
@Override
2730
public <T> Mono<Void> sendCommand(Command<T> command, String targetName, long delayMillis, String domain) {
28-
throw new UnsupportedOperationException("Not implemented yet");
31+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
2932
}
3033

3134
@Override
3235
public Mono<Void> sendCommand(CloudEvent command, String targetName) {
33-
throw new UnsupportedOperationException("Not implemented yet");
36+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
3437
}
3538

3639
@Override
3740
public Mono<Void> sendCommand(CloudEvent command, String targetName, long delayMillis) {
38-
throw new UnsupportedOperationException("Not implemented yet");
41+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
3942
}
4043

4144
@Override
4245
public Mono<Void> sendCommand(CloudEvent command, String targetName, String domain) {
43-
throw new UnsupportedOperationException("Not implemented yet");
46+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
4447
}
4548

4649
@Override
4750
public Mono<Void> sendCommand(CloudEvent command, String targetName, long delayMillis, String domain) {
48-
throw new UnsupportedOperationException("Not implemented yet");
51+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
4952
}
5053

5154
@Override
5255
public <T, R> Mono<R> requestReply(AsyncQuery<T> query, String targetName, Class<R> type) {
53-
throw new UnsupportedOperationException("Not implemented yet");
56+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
5457
}
5558

5659
@Override
5760
public <T, R> Mono<R> requestReply(AsyncQuery<T> query, String targetName, Class<R> type, String domain) {
58-
throw new UnsupportedOperationException("Not implemented yet");
61+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
5962
}
6063

6164
@Override
6265
public <R extends CloudEvent> Mono<R> requestReply(CloudEvent query, String targetName, Class<R> type) {
63-
throw new UnsupportedOperationException("Not implemented yet");
66+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
6467
}
6568

6669
@Override
6770
public <R extends CloudEvent> Mono<R> requestReply(CloudEvent query, String targetName, Class<R> type, String domain) {
68-
throw new UnsupportedOperationException("Not implemented yet");
71+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
6972
}
7073

7174
@Override
7275
public <T> Mono<Void> reply(T response, From from) {
73-
throw new UnsupportedOperationException("Not implemented yet");
76+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
7477
}
7578
}

async/async-kafka/src/main/java/org/reactivecommons/async/kafka/KafkaDomainEventBus.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
@AllArgsConstructor
1111
public class KafkaDomainEventBus implements DomainEventBus {
12+
public static final String NOT_IMPLEMENTED_YET = "Not implemented yet";
1213
private final ReactiveMessageSender sender;
1314

1415
@Override
@@ -18,7 +19,7 @@ public <T> Publisher<Void> emit(DomainEvent<T> event) {
1819

1920
@Override
2021
public <T> Publisher<Void> emit(String domain, DomainEvent<T> event) {
21-
throw new UnsupportedOperationException("Not implemented yet");
22+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
2223
}
2324

2425
@Override
@@ -28,6 +29,6 @@ public Publisher<Void> emit(CloudEvent event) {
2829

2930
@Override
3031
public Publisher<Void> emit(String domain, CloudEvent event) {
31-
throw new UnsupportedOperationException("Not implemented yet");
32+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
3233
}
3334
}

main.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ allprojects {
1111
maven { url 'https://repo.spring.io/milestone' }
1212
}
1313

14-
if (toPublish.split(',').contains(project.name) || project.name == 'ReactiveArchitectureCommons') {
14+
if (toPublish.split(',').contains(project.name) || project.name == rootProject.name) {
1515

1616
sonar {
1717
properties {

starters/async-commons-starter/src/main/java/org/reactivecommons/async/impl/config/annotations/EnableDirectAsyncGateway.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package org.reactivecommons.async.impl.config.annotations;
22

3-
import org.reactivecommons.async.starter.config.ReactiveCommonsConfig;
43
import org.reactivecommons.async.starter.senders.DirectAsyncGatewayConfig;
54
import org.springframework.context.annotation.Configuration;
65
import org.springframework.context.annotation.Import;

starters/async-commons-starter/src/main/java/org/reactivecommons/async/starter/config/ConnectionManager.java

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,15 +19,6 @@ public ConnectionManager addDomain(String domain, BrokerProvider domainConn) {
1919
return this;
2020
}
2121

22-
private BrokerProvider getChecked(String domain) {
23-
BrokerProvider domainProvider = connections.get(domain);
24-
if (domainProvider == null) {
25-
throw new RuntimeException("You are trying to use the domain " + domain
26-
+ " but this connection is not defined");
27-
}
28-
return domainProvider;
29-
}
30-
3122
public Map<String, BrokerProvider> getProviders() {
3223
return connections;
3324
}

starters/async-commons-starter/src/main/java/org/reactivecommons/async/starter/config/DomainHandlers.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package org.reactivecommons.async.starter.config;
22

33
import org.reactivecommons.async.commons.HandlerResolver;
4+
import org.reactivecommons.async.starter.exceptions.InvalidConfigurationException;
45

56
import java.util.Map;
67
import java.util.TreeMap;
@@ -15,7 +16,7 @@ public void add(String domain, HandlerResolver resolver) {
1516
public HandlerResolver get(String domain) {
1617
HandlerResolver handlerResolver = handlers.get(domain);
1718
if (handlerResolver == null) {
18-
throw new RuntimeException("You are trying to use the domain " + domain
19+
throw new InvalidConfigurationException("You are trying to use the domain " + domain
1920
+ " but this connection is not defined");
2021
}
2122
return handlerResolver;

starters/async-commons-starter/src/main/java/org/reactivecommons/async/starter/listeners/AbstractListenerConfig.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
package org.reactivecommons.async.starter.listeners;
22

33
import org.reactivecommons.async.commons.HandlerResolver;
4-
import org.reactivecommons.async.starter.config.ConnectionManager;
54
import org.reactivecommons.async.starter.broker.BrokerProvider;
5+
import org.reactivecommons.async.starter.config.ConnectionManager;
66
import org.reactivecommons.async.starter.config.DomainHandlers;
77

88
public abstract class AbstractListenerConfig {
99

10-
public AbstractListenerConfig(ConnectionManager manager, DomainHandlers handlers) {
10+
protected AbstractListenerConfig(ConnectionManager manager, DomainHandlers handlers) {
1111
manager.forDomain((domain, provider) -> listen(domain, provider, handlers.get(domain)));
1212
}
1313

starters/async-commons-starter/src/main/java/org/reactivecommons/async/starter/props/GenericAsyncProps.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,13 @@ public abstract class GenericAsyncProps<P> {
1515
private String appName;
1616
private String secret;
1717

18-
abstract public void setConnectionProperties(P properties);
18+
public abstract void setConnectionProperties(P properties);
1919

20-
abstract public P getConnectionProperties();
20+
public abstract P getConnectionProperties();
2121

22-
abstract public String getBrokerType();
22+
public abstract String getBrokerType();
2323

24-
abstract public boolean isEnabled();
24+
public abstract boolean isEnabled();
2525

26-
abstract public void setUseDiscardNotifierPerDomain(boolean enabled);
26+
public abstract void setUseDiscardNotifierPerDomain(boolean enabled);
2727
}

starters/async-commons-starter/src/main/java/org/reactivecommons/async/starter/props/GenericAsyncPropsDomain.java

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import com.fasterxml.jackson.databind.ObjectMapper;
44
import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule;
5+
import lombok.EqualsAndHashCode;
56
import lombok.Getter;
67
import lombok.Setter;
78
import lombok.SneakyThrows;
@@ -10,6 +11,7 @@
1011
import java.lang.reflect.Constructor;
1112
import java.util.HashMap;
1213
import java.util.Map;
14+
import java.util.Objects;
1315

1416
import static org.reactivecommons.async.api.HandlerRegistry.DEFAULT_DOMAIN;
1517

@@ -61,7 +63,7 @@ public GenericAsyncPropsDomain(String defaultAppName,
6163
}
6264

6365
protected void fillCustoms(T asyncProps) {
64-
66+
// To be overridden called after the default properties are set
6567
}
6668

6769
public T getProps(String domain) {
@@ -153,4 +155,17 @@ public interface SecretFiller<P> {
153155
void fillWithSecret(String domain, GenericAsyncProps<P> props);
154156
}
155157

158+
@Override
159+
public boolean equals(Object o) {
160+
if (this == o) return true;
161+
if (o == null || getClass() != o.getClass()) return false;
162+
if (!super.equals(o)) return false;
163+
GenericAsyncPropsDomain<?, ?> that = (GenericAsyncPropsDomain<?, ?>) o;
164+
return Objects.equals(asyncPropsClass, that.asyncPropsClass) && Objects.equals(propsClass, that.propsClass);
165+
}
166+
167+
@Override
168+
public int hashCode() {
169+
return Objects.hash(super.hashCode(), asyncPropsClass, propsClass);
170+
}
156171
}

starters/async-commons-starter/src/main/java/org/reactivecommons/async/starter/props/GenericAsyncPropsDomainProperties.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
@Setter
1111
public class GenericAsyncPropsDomainProperties<T extends GenericAsyncProps<P>, P> extends HashMap<String, T> {
1212

13-
public GenericAsyncPropsDomainProperties(Map<? extends String, ? extends T> m) {
13+
public GenericAsyncPropsDomainProperties(Map<String, ? extends T> m) {
1414
super(m);
1515
}
1616

0 commit comments

Comments
 (0)