Skip to content

Commit dc02170

Browse files
committed
[frontend/backend] feat : clean code
Signed-off-by: Marine LM <[email protected]>
1 parent b4ab316 commit dc02170

File tree

6 files changed

+9
-10
lines changed

6 files changed

+9
-10
lines changed

openaev-api/src/main/java/io/openaev/service/catalog_connectors/CatalogConnectorIngestionService.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -192,6 +192,9 @@ public Set<CatalogConnectorConfiguration> buildConnectorConfigurations(
192192

193193
for (Iterator<String> it = properties.fieldNames(); it.hasNext(); ) {
194194
String key = it.next();
195+
if ("OPENAEV_URL".equals(key)) {
196+
continue;
197+
}
195198
JsonNode prop = properties.get(key);
196199

197200
CatalogConnectorConfiguration conf =

openaev-api/src/main/java/io/openaev/service/connector_instances/ConnectorInstanceService.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,7 @@ private ConnectorInstance buildNewConnectorInstanceFromCatalog(
114114
newInstance.setCatalogConnector(catalogConnector);
115115
newInstance.setRequestedStatus(ConnectorInstance.REQUESTED_STATUS_TYPE.stopping);
116116
newInstance.setCurrentStatus(ConnectorInstance.CURRENT_STATUS_TYPE.stopped);
117+
newInstance.setSource(ConnectorInstance.SOURCE.CATALOG_DEPLOYMENT);
117118
return newInstance;
118119
}
119120

openaev-api/src/test/java/io/openaev/service/CatalogConnectorIngestionServiceTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@
77
import io.openaev.database.model.CatalogConnector;
88
import io.openaev.database.model.CatalogConnectorConfiguration;
99
import io.openaev.database.repository.ConnectorInstanceConfigurationRepository;
10-
import io.openaev.rest.connector_instance.service.ConnectorInstanceService;
1110
import io.openaev.service.catalog_connectors.CatalogConnectorIngestionService;
1211
import io.openaev.service.catalog_connectors.CatalogConnectorService;
12+
import io.openaev.service.connector_instances.ConnectorInstanceService;
1313
import java.util.HashSet;
1414
import java.util.List;
1515
import java.util.Set;

openaev-api/src/test/java/io/openaev/service/ScenarioServiceTest.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@
1818
import io.openaev.healthcheck.dto.HealthCheck;
1919
import io.openaev.healthcheck.enums.ExternalServiceDependency;
2020
import io.openaev.healthcheck.utils.HealthCheckUtils;
21-
import io.openaev.rest.collector.service.CollectorService;
2221
import io.openaev.rest.inject.service.InjectDuplicateService;
2322
import io.openaev.rest.inject.service.InjectService;
2423
import io.openaev.telemetry.metric_collectors.ActionMetricCollector;
@@ -56,7 +55,6 @@ class ScenarioServiceTest extends IntegrationTest {
5655
@Autowired private HealthCheckUtils healthCheckUtils;
5756

5857
@Mock Ee eeService;
59-
@Mock GrantService grantService;
6058
@Mock VariableService variableService;
6159
@Mock ChallengeService challengeService;
6260
@Autowired private TeamService teamService;
@@ -65,7 +63,6 @@ class ScenarioServiceTest extends IntegrationTest {
6563
@Mock private InjectService injectService;
6664
@Mock private TagRuleService tagRuleService;
6765
@Mock private UserService userService;
68-
@Mock private CollectorService collectorService;
6966
@InjectMocks private ScenarioService scenarioService;
7067
@Autowired private ScenarioMapper scenarioMapper;
7168

openaev-front/src/admin/components/integrations/common/ConnectorContext.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ export const injectorConfig: ConnectorContextType<InjectorOutput> = {
6262
}),
6363
};
6464

65+
type ConnectorUnion = ExecutorOutput | InjectorOutput | CollectorOutput;
6566
export const collectorConfig: ConnectorContextType<CollectorOutput> = {
6667
connectorType: 'collector',
6768
apiRequest: {
@@ -106,10 +107,7 @@ export const executorConfig: ConnectorContextType<ExecutorOutput> = {
106107
}),
107108
};
108109

109-
export const ConnectorContext = createContext<
110-
ConnectorContextType<ExecutorOutput>
111-
| ConnectorContextType<CollectorOutput>
112-
| ConnectorContextType<InjectorOutput>>({
110+
export const ConnectorContext = createContext<ConnectorContextType<ConnectorUnion>>({
113111
connectorType: 'collector',
114112
logoUrl: _type => '',
115113
apiRequest: {
@@ -121,5 +119,5 @@ export const ConnectorContext = createContext<
121119
list: '/admin/integrations',
122120
detail: (_id: string) => '/admin/integrations',
123121
},
124-
normalizeSingle: (_data: ExecutorOutput | InjectorOutput | CollectorOutput) => ({} as ConnectorOutput),
122+
normalizeSingle: (_data: ConnectorUnion) => ({} as ConnectorOutput),
125123
});

openaev-model/src/main/java/io/openaev/database/model/ConnectorInstance.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ public enum SOURCE {
6262
@JdbcTypeCode(SqlTypes.NAMED_ENUM)
6363
@Column(name = "connector_instance_source")
6464
@JsonProperty("connector_instance_source")
65-
@NotBlank
65+
@NotNull
6666
private SOURCE source = SOURCE.OTHER;
6767

6868
@Enumerated(EnumType.STRING)

0 commit comments

Comments
 (0)