yujunjun/OpenMetadata
1
0
Fork 0
mirror of https://github.com/open-metadata/OpenMetadata.git synced 2025-09-25 17:04:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: removed single relationship test between testCase and testSuite + throw error if no executable test suite can be found for a test case (#13032)

Browse Source
This commit is contained in:
Teddy 2023-08-31 18:53:05 +02:00 committed by GitHub
parent ba2201f4ea
commit c198970217
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
openmetadata-service/src/main/java/org/openmetadata/service/jdbi3/TestCaseRepository.java
View File

@ -122,25 +122,25 @@ public class TestCaseRepository extends EntityRepository<TestCase> {
validateTestParameters(test.getParameterValues(), testDefinition.getParameterDefinition());
}
private EntityReference getTestSuite(TestCase test) {
private EntityReference getTestSuite(TestCase test) throws EntityNotFoundException {
// `testSuite` field returns the executable `testSuite` linked to that testCase
List<CollectionDAO.EntityRelationshipRecord> records =
findFromRecords(test.getId(), entityType, Relationship.CONTAINS, TEST_SUITE);
ensureSingleRelationship(entityType, test.getId(), records, Relationship.CONTAINS.value(), true);
for (CollectionDAO.EntityRelationshipRecord testSuiteId : records) {
TestSuite testSuite = Entity.getEntity(TEST_SUITE, testSuiteId.getId(), "", Include.ALL);
if (Boolean.TRUE.equals(testSuite.getExecutable())) {
return testSuite.getEntityReference();
}
}
return null;
throw new EntityNotFoundException(
String.format("Error occurred when retrieving executable test suite for testCase %s. ", test.getName())
+ "No executable test suite was found.");
}
private List<TestSuite> getTestSuites(TestCase test) {
// `testSuites` field returns all the `testSuite` (executable and logical) linked to that testCase
List<CollectionDAO.EntityRelationshipRecord> records =
findFromRecords(test.getId(), entityType, Relationship.CONTAINS, TEST_SUITE);
ensureSingleRelationship(entityType, test.getId(), records, Relationship.CONTAINS.value(), true);
return records.stream()
.map(testSuiteId -> Entity.<TestSuite>getEntity(TEST_SUITE, testSuiteId.getId(), "", Include.ALL))
.collect(Collectors.toList());