yujunjun/OpenMetadata
1
0
Fork 0
mirror of https://github.com/open-metadata/OpenMetadata.git synced 2025-11-10 07:53:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

Update test conn definition on initialize (#11940)

Browse Source
This commit is contained in:
Pere Miquel Brull 2023-06-09 14:06:55 +02:00 committed by GitHub
parent e858eae87a
commit 20ebe70d9e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 21 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
openmetadata-service/src/main/java/org/openmetadata/service/jdbi3/TestConnectionDefinitionRepository.java
View File

@ -16,7 +16,7 @@ import org.openmetadata.service.util.EntityUtil;
*/ */
public class TestConnectionDefinitionRepository extends EntityRepository<TestConnectionDefinition> { public class TestConnectionDefinitionRepository extends EntityRepository<TestConnectionDefinition> {
private static final String UPDATE_FIELDS = ""; private static final String UPDATE_FIELDS = "steps";
private static final String PATCH_FIELDS = ""; private static final String PATCH_FIELDS = "";
public TestConnectionDefinitionRepository(CollectionDAO dao) { public TestConnectionDefinitionRepository(CollectionDAO dao) {
@ -72,6 +72,18 @@ public class TestConnectionDefinitionRepository extends EntityRepository<TestCon
@Override @Override
public EntityUpdater getUpdater( public EntityUpdater getUpdater(
TestConnectionDefinition original, TestConnectionDefinition updated, Operation operation) { TestConnectionDefinition original, TestConnectionDefinition updated, Operation operation) {
return null; return new TestConnectionDefinitionUpdater(original, updated, operation);
}
public class TestConnectionDefinitionUpdater extends EntityUpdater {
public TestConnectionDefinitionUpdater(
TestConnectionDefinition original, TestConnectionDefinition updated, Operation operation) {
super(original, updated, operation);
}
@Override
public void entitySpecificUpdate() throws IOException {
recordChange("steps", original.getSteps(), updated.getSteps(), true);
}
} }
} }

13
openmetadata-service/src/main/java/org/openmetadata/service/resources/services/connections/TestConnectionDefinitionResource.java
View File

@ -37,7 +37,6 @@ import org.openmetadata.service.jdbi3.TestConnectionDefinitionRepository;
import org.openmetadata.service.resources.Collection; import org.openmetadata.service.resources.Collection;
import org.openmetadata.service.resources.EntityResource; import org.openmetadata.service.resources.EntityResource;
import org.openmetadata.service.security.Authorizer; import org.openmetadata.service.security.Authorizer;
import org.openmetadata.service.util.EntityUtil;
import org.openmetadata.service.util.RestUtil; import org.openmetadata.service.util.RestUtil;
import org.openmetadata.service.util.ResultList; import org.openmetadata.service.util.ResultList;
@ -70,13 +69,12 @@ public class TestConnectionDefinitionResource
List<TestConnectionDefinition> testConnectionDefinitions = List<TestConnectionDefinition> testConnectionDefinitions =
repository.getEntitiesFromSeedData(".*json/data/testConnections/.*\\.json$"); repository.getEntitiesFromSeedData(".*json/data/testConnections/.*\\.json$");
for (TestConnectionDefinition testConnectionDefinition :
repository.listAll(EntityUtil.Fields.EMPTY_FIELDS, new ListFilter(Include.ALL))) {
repository.delete(ADMIN_USER_NAME, testConnectionDefinition.getId(), true, true);
}
for (TestConnectionDefinition testConnectionDefinition : testConnectionDefinitions) { for (TestConnectionDefinition testConnectionDefinition : testConnectionDefinitions) {
repository.initializeEntity(testConnectionDefinition); repository.prepareInternal(testConnectionDefinition);
testConnectionDefinition.setId(UUID.randomUUID());
testConnectionDefinition.setUpdatedBy(ADMIN_USER_NAME);
testConnectionDefinition.setUpdatedAt(System.currentTimeMillis());
repository.createOrUpdate(null, testConnectionDefinition);
} }
} }
@ -84,7 +82,6 @@ public class TestConnectionDefinitionResource
/* Required for serde */ /* Required for serde */
} }
// TODO remove the list method?
@GET @GET
@Operation( @Operation(
operationId = "listTestConnectionDefinitions", operationId = "listTestConnectionDefinitions",

4
openmetadata-service/src/test/java/org/openmetadata/service/resources/services/connections/TestConnectionDefinitionResourceTest.java
View File

@ -37,9 +37,9 @@ public class TestConnectionDefinitionResourceTest extends OpenMetadataApplicatio
@Test @Test
public void test_list_test_connection_definition() throws HttpResponseException { public void test_list_test_connection_definition() throws HttpResponseException {
WebTarget target = listResource(); WebTarget target = listResource();
ResultList mysqlTest = TestUtils.get(target, ResultList.class, ADMIN_AUTH_HEADERS); ResultList testConnectionDefinitions = TestUtils.get(target, ResultList.class, ADMIN_AUTH_HEADERS);
// we get 10 as it's the default paging size // we get 10 as it's the default paging size
assertEquals(mysqlTest.getData().size(), 10); assertEquals(testConnectionDefinitions.getData().size(), 10);
} }
protected final WebTarget getResourceByName(String name) { protected final WebTarget getResourceByName(String name) {