From f5d5347a3aa947cafe8db074cf7b053db6eb1217 Mon Sep 17 00:00:00 2001 From: Nigel Jones Date: Thu, 26 Jan 2023 14:03:59 +0000 Subject: [PATCH 01/11] #7374 ensure open types tests are built Signed-off-by: Nigel Jones --- .../open-types-test-generator/build.gradle | 11 +++++++++-- .../open-types-fvt/open-types-test/build.gradle | 11 +++++++---- settings.gradle | 15 +++++---------- 3 files changed, 21 insertions(+), 16 deletions(-) diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/build.gradle b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/build.gradle index 1a360a5c4cb..95b5a5aafc7 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/build.gradle +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/build.gradle @@ -3,14 +3,19 @@ * Copyright Contributors to the ODPi Egeria project. */ +plugins { + id 'application' + id "com.github.johnrengelman.shadow" +} + dependencies { implementation 'org.slf4j:slf4j-api' implementation 'com.fasterxml.jackson.core:jackson-databind' compileOnly 'com.fasterxml.jackson.core:jackson-annotations' implementation project(':open-metadata-implementation:repository-services:repository-services-apis') implementation project(':open-metadata-resources:open-metadata-archives:open-metadata-types') - implementation project(':open-metadata-implementation:frameworks:open-connector-framework') - implementation project(':open-metadata-implementation:frameworks:audit-log-framework') + compileOnly project(':open-metadata-implementation:frameworks:open-connector-framework') + compileOnly project(':open-metadata-implementation:frameworks:audit-log-framework') } description = 'Open Types Test generator' @@ -24,3 +29,5 @@ shadowJar { mainClassName = 'org.odpi.openmetadata.fvt.opentypes.generators.OmrsBeanGenerator' archiveClassifier = 'jar-with-dependencies' } + + diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle index ec4da3e18ba..d84d4247d58 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle @@ -3,6 +3,7 @@ * Copyright Contributors to the ODPi Egeria project. */ + dependencies { implementation 'org.slf4j:slf4j-api' implementation 'com.fasterxml.jackson.core:jackson-databind' @@ -11,8 +12,10 @@ dependencies { implementation project(':open-metadata-resources:open-metadata-archives:open-metadata-types') } -description = 'Open Types Test generator' +description = 'Open Types Test' + +java { + withJavadocJar() +} -//java { -// withJavadocJar() -//} +build.dependsOn(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator:build') diff --git a/settings.gradle b/settings.gradle index eacc4d37b10..134c429e672 100644 --- a/settings.gradle +++ b/settings.gradle @@ -445,11 +445,8 @@ include(':open-metadata-test:open-metadata-fvt:access-services-fvt:subject-area- include(':open-metadata-test:open-metadata-fvt:access-services-fvt') include(':open-metadata-test:open-metadata-fvt:fvt-utilities') //TODO: restore tests once ready -//include(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator') -//include(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test') -//include(':open-metadata-test:open-metadata-fvt:open-types-fvt') -//include(':open-metadata-test:open-metadata-fvt') -//include(':open-metadata-test') +include(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator') +include(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test') project(':open-metadata-implementation:frameworks:audit-log-framework').projectDir = file('open-metadata-implementation/frameworks/audit-log-framework') project(':open-metadata-implementation:frameworks:open-connector-framework').projectDir = file('open-metadata-implementation/frameworks/open-connector-framework') project(':open-metadata-implementation:frameworks:open-discovery-framework').projectDir = file('open-metadata-implementation/frameworks/open-discovery-framework') @@ -863,8 +860,6 @@ project(':open-metadata-test:open-metadata-fvt:access-services-fvt').projectDir project(':open-metadata-test:open-metadata-fvt:fvt-utilities').projectDir = file('open-metadata-test/open-metadata-fvt/fvt-utilities') project(':open-metadata-distribution:open-metadata-assemblies').projectDir = file('open-metadata-distribution/open-metadata-assemblies') -//TODO: restore tests -//project(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator').projectDir = file('open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator') -//project(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test').projectDir = file('open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test') -//project(':open-metadata-test:open-metadata-fvt:open-types-fvt').projectDir = file('open-metadata-test/open-metadata-fvt/open-types-fvt') -//project(':open-metadata-test:open-metadata-fvt').projectDir = file('open-metadata-test/open-metadata-fvt') +project(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator').projectDir = file('open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator') +project(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test').projectDir = file('open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test') + From 4d12e236e96cd540b9e3f628e5a15e83ad80d118 Mon Sep 17 00:00:00 2001 From: David Radley Date: Thu, 26 Jan 2023 13:25:44 +0000 Subject: [PATCH 02/11] git73245 fix generator around maps Signed-off-by: David Radley Signed-off-by: Nigel Jones --- .../main/resources/templates/TestOmrsBeanAccessorTemplate | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/TestOmrsBeanAccessorTemplate b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/TestOmrsBeanAccessorTemplate index 2279257266f..6e00eb03c23 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/TestOmrsBeanAccessorTemplate +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/TestOmrsBeanAccessorTemplate @@ -368,14 +368,16 @@ public class TestBeansToAccessOMRS { $$Enum$$> MapPropertyValue mapTypePropertyValue; + PrimitivePropertyValue primitivePropertyValueForMap; + InstanceProperties mapValues; // In the models we are generating from we only have map types, this code assumes those types. <$$Map$$ mapTypePropertyValue = new MapPropertyValue(); - PrimitivePropertyValue primitivePropertyValueForMap = new PrimitivePropertyValue(); + primitivePropertyValueForMap = new PrimitivePropertyValue(); primitivePropertyValueForMap.setPrimitiveDefCategory(PrimitiveDefCategory.OM_PRIMITIVE_TYPE_STRING); primitivePropertyValueForMap.setPrimitiveValue("test string value for map"); - InstanceProperties mapValues = new InstanceProperties(); + mapValues = new InstanceProperties(); mapValues.setProperty("a_new_property_name",primitivePropertyValueForMap); mapTypePropertyValue.setMapValues(mapValues); instanceProperties.setProperty("<$$uMapName$$>", mapTypePropertyValue); From 3b7943bfb96e849254b5c1b0fe7a311368467289 Mon Sep 17 00:00:00 2001 From: Nigel Jones Date: Thu, 26 Jan 2023 14:57:58 +0000 Subject: [PATCH 03/11] #7374 fix execution of open types test Signed-off-by: Nigel Jones --- .../asset-consumer-fvt/build.gradle | 6 ++++-- .../open-types-fvt/open-types-test/build.gradle | 14 ++++++++++++++ 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/open-metadata-test/open-metadata-fvt/access-services-fvt/asset-consumer-fvt/build.gradle b/open-metadata-test/open-metadata-fvt/access-services-fvt/asset-consumer-fvt/build.gradle index 1c36fc11cb7..29a97b88c01 100644 --- a/open-metadata-test/open-metadata-fvt/access-services-fvt/asset-consumer-fvt/build.gradle +++ b/open-metadata-test/open-metadata-fvt/access-services-fvt/asset-consumer-fvt/build.gradle @@ -104,7 +104,9 @@ task cleanData(type: Delete) { test { description 'Run Integration Tests' group = 'verification' - outputs.upToDateWhen {return false} + outputs.upToDateWhen { + + } systemProperties = [ 'fvt.url': 'https://localhost:10443' ] @@ -117,4 +119,4 @@ test { // Resolves logging conflict with Jena loggingCapabilities { enforceLogback()// Configuration goes here -} \ No newline at end of file +} diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle index d84d4247d58..fdc084cc2cc 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle @@ -10,10 +10,24 @@ dependencies { compileOnly 'com.fasterxml.jackson.core:jackson-annotations' implementation project(':open-metadata-implementation:repository-services:repository-services-apis') implementation project(':open-metadata-resources:open-metadata-archives:open-metadata-types') + implementation project(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator') + implementation project(':open-metadata-implementation:frameworks:open-connector-framework') + implementation project(':open-metadata-implementation:frameworks:audit-log-framework') + + testImplementation 'org.mockito:mockito-core' + testImplementation 'org.junit.jupiter:junit-jupiter-api' + testImplementation 'org.junit.jupiter:junit-jupiter-params' + testImplementation 'org.junit.jupiter:junit-jupiter-engine' + testImplementation 'org.testng:testng' + } description = 'Open Types Test' +test { + useTestNG() +} + java { withJavadocJar() } From 3ee49f6cdf6575499a1a06084fbd75cee2b1fe00 Mon Sep 17 00:00:00 2001 From: Nigel Jones Date: Thu, 26 Jan 2023 18:56:46 +0000 Subject: [PATCH 04/11] #7374 further fixes to open types FVT test Signed-off-by: Nigel Jones --- .../open-types-test-generator/build.gradle | 2 ++ .../open-types-fvt/open-types-test/build.gradle | 15 ++++++++++++++- 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/build.gradle b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/build.gradle index 95b5a5aafc7..a15c5fc0a22 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/build.gradle +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/build.gradle @@ -16,6 +16,8 @@ dependencies { implementation project(':open-metadata-resources:open-metadata-archives:open-metadata-types') compileOnly project(':open-metadata-implementation:frameworks:open-connector-framework') compileOnly project(':open-metadata-implementation:frameworks:audit-log-framework') + runtimeOnly 'ch.qos.logback:logback-classic' + } description = 'Open Types Test generator' diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle index fdc084cc2cc..7ee93568c14 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle @@ -32,4 +32,17 @@ java { withJavadocJar() } -build.dependsOn(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator:build') +clean { + delete += fileTree('src').include('**/*') +} + +task createTestSrc(type: Exec) { + dependsOn ':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator:shadowJar' + group = "Execution" + description = "Create test source" + workingDir '../../../..' + commandLine "java", "-jar", "${project.projectDir}/../open-types-test-generator/build/libs/open-types-test-generator-${project.version}-jar-with-dependencies.jar" +} + +//build.dependsOn(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator:build') +classes.dependsOn(createTestSrc) From 08d5ed9f9e06134f5db7ef9720a59bbc18b0285d Mon Sep 17 00:00:00 2001 From: Nigel Jones Date: Fri, 27 Jan 2023 09:24:51 +0000 Subject: [PATCH 05/11] #7374 further fixes to open types FVT test Signed-off-by: Nigel Jones --- .../open-types-fvt/open-types-test/build.gradle | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle index 7ee93568c14..5ed12c28573 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle @@ -32,10 +32,12 @@ java { withJavadocJar() } -clean { - delete += fileTree('src').include('**/*') +task cleanGeneratedSrc(type: Delete) { + delete 'src' } +clean.dependsOn(cleanGeneratedSrc) + task createTestSrc(type: Exec) { dependsOn ':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator:shadowJar' group = "Execution" From 707df56b0c9fc2cbdceaeb6214972290c760ab4a Mon Sep 17 00:00:00 2001 From: Nigel Jones Date: Fri, 27 Jan 2023 09:44:21 +0000 Subject: [PATCH 06/11] #7374 Ensure open types test dependent on build of test code Signed-off-by: Nigel Jones --- .../open-types-fvt/open-types-test/build.gradle | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle index 5ed12c28573..7511747cefe 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle @@ -46,5 +46,4 @@ task createTestSrc(type: Exec) { commandLine "java", "-jar", "${project.projectDir}/../open-types-test-generator/build/libs/open-types-test-generator-${project.version}-jar-with-dependencies.jar" } -//build.dependsOn(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator:build') -classes.dependsOn(createTestSrc) +compileJava.dependsOn(createTestSrc) From 31b306e093f99d8513e3463dddc119e5fd2beda7 Mon Sep 17 00:00:00 2001 From: Nigel Jones Date: Fri, 27 Jan 2023 10:02:47 +0000 Subject: [PATCH 07/11] Z#7374 Use official build action Signed-off-by: Nigel Jones --- .github/workflows/pr-v4.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/pr-v4.yml b/.github/workflows/pr-v4.yml index 3158aa96d5a..c3ee6e6a51a 100644 --- a/.github/workflows/pr-v4.yml +++ b/.github/workflows/pr-v4.yml @@ -29,6 +29,7 @@ jobs: # Only cache for main build cache-read-only: true arguments: build publishToMavenLocal + arguments: build - name: Upload Test coverage report uses: actions/upload-artifact@v3 with: From cc9aa7b067ddcf0a2970a440053917318733b89d Mon Sep 17 00:00:00 2001 From: David Radley Date: Fri, 27 Jan 2023 15:26:32 +0000 Subject: [PATCH 08/11] git7324 amend the open types tests to allow them to work Signed-off-by: David Radley --- .../opentypes/generators/OmrsBeanGenerator.java | 16 ++++++++-------- .../fvt/opentypes/model/OmrsBeanModel.java | 4 ++++ .../resources/templates/OmrsBeanEntityTemplate | 2 +- .../templates/OmrsBeanRelationshipTemplate | 2 +- 4 files changed, 14 insertions(+), 10 deletions(-) diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/java/org/odpi/openmetadata/fvt/opentypes/generators/OmrsBeanGenerator.java b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/java/org/odpi/openmetadata/fvt/opentypes/generators/OmrsBeanGenerator.java index 16bcf575d80..27fc0ac10af 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/java/org/odpi/openmetadata/fvt/opentypes/generators/OmrsBeanGenerator.java +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/java/org/odpi/openmetadata/fvt/opentypes/generators/OmrsBeanGenerator.java @@ -673,13 +673,13 @@ private void mapOMRSToOMAS(String omrsType, Map replacementMap, attrLine = attrLine.replaceAll(GeneratorUtilities.getRegexToken("AttrType"), "String"); attrValue = "\"string" + attributeNumber + "\""; - } else if (uppercaseattributeType.equals("DATE")) { - attrLine = attrLine.replaceAll(GeneratorUtilities.getRegexToken("AttrType"), "java.util.Date"); - attrValue = "new java.util.Date()"; +// } else if (uppercaseattributeType.equals("DATE")) { +// attrLine = attrLine.replaceAll(GeneratorUtilities.getRegexToken("AttrType"), "java.util.Date"); +// attrValue = "new java.util.Date()"; } else if (uppercaseattributeType.equals("INT")) { attrLine = attrLine.replaceAll(GeneratorUtilities.getRegexToken("AttrType"), "Integer"); attrValue = String.valueOf(attributeNumber); - } else if (uppercaseattributeType.equals("LONG")) { + } else if (uppercaseattributeType.equals("LONG") || uppercaseattributeType.equals("DATE")) { attrLine = attrLine.replaceAll(GeneratorUtilities.getRegexToken("AttrType"), "Long"); attrValue = String.valueOf(attributeNumber) +"L"; } else if (uppercaseattributeType.equals("FLOAT")) { @@ -746,13 +746,13 @@ private void mapOMRSToOMAS(String omrsType, Map replacementMap, attrLine = attrLine.replaceAll(GeneratorUtilities.getRegexToken("AttrType"), "String"); attrValue = "\"string" + attributeNumber + "\""; - } else if (uppercaseattributeType.equals("DATE")) { - attrLine = attrLine.replaceAll(GeneratorUtilities.getRegexToken("AttrType"), "java.util.Date"); - attrValue = "new java.util.Date()"; +// } else if (uppercaseattributeType.equals("DATE")) { +// attrLine = attrLine.replaceAll(GeneratorUtilities.getRegexToken("AttrType"), "java.util.Date"); +// attrValue = "new java.util.Date()"; } else if (uppercaseattributeType.equals("INT")) { attrLine = attrLine.replaceAll(GeneratorUtilities.getRegexToken("AttrType"), "Integer"); attrValue = String.valueOf(attributeNumber); - } else if (uppercaseattributeType.equals("LONG")) { + } else if (uppercaseattributeType.equals("LONG") || uppercaseattributeType.equals("DATE")) { attrLine = attrLine.replaceAll(GeneratorUtilities.getRegexToken("AttrType"), "Long"); attrValue = String.valueOf(attributeNumber) +"L"; } else if (uppercaseattributeType.equals("FLOAT")) { diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/java/org/odpi/openmetadata/fvt/opentypes/model/OmrsBeanModel.java b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/java/org/odpi/openmetadata/fvt/opentypes/model/OmrsBeanModel.java index 4f43f85ae0b..19c6cfc86bd 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/java/org/odpi/openmetadata/fvt/opentypes/model/OmrsBeanModel.java +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/java/org/odpi/openmetadata/fvt/opentypes/model/OmrsBeanModel.java @@ -483,6 +483,10 @@ private String normalizeTypeName(String typeName) { if (typeName.equals("int")) { normalizedTypeName = "Integer"; } + //TODO do we need to get the primitive class name from the PrimitiveDefCategory javaClassName + if (typeName.equals("date")) { + normalizedTypeName = "Long"; + } return normalizedTypeName; } } diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/OmrsBeanEntityTemplate b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/OmrsBeanEntityTemplate index 6373edf0987..94525a403e4 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/OmrsBeanEntityTemplate +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/OmrsBeanEntityTemplate @@ -162,7 +162,7 @@ public class <$$uname$$> implements Serializable { * @return Set of Strings of the <$$uname$$> map property names */ public static java.util.Set getMapPropertyNames() { - return ENUM_NAMES_SET; + return MAP_NAMES_SET; } /** diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/OmrsBeanRelationshipTemplate b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/OmrsBeanRelationshipTemplate index 70608ec23d7..be897b0e2ef 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/OmrsBeanRelationshipTemplate +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator/src/main/resources/templates/OmrsBeanRelationshipTemplate @@ -171,7 +171,7 @@ public class <$$uname$$> extends OMRSLine { * @return Set of Strings of the <$$uname$$> map property names */ public static java.util.Set getMapPropertyNames() { - return ENUM_NAMES_SET; + return MAP_NAMES_SET; } /** From 6c5e929104debb866c695981c7f9367539c9eb90 Mon Sep 17 00:00:00 2001 From: Nigel Jones Date: Tue, 7 Feb 2023 09:47:24 +0000 Subject: [PATCH 09/11] #7324 skip javadoc (long!) for auto generated tests Signed-off-by: Nigel Jones --- .../open-types-fvt/open-types-test/build.gradle | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle index 7511747cefe..bae674e3ef5 100644 --- a/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle +++ b/open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test/build.gradle @@ -29,7 +29,8 @@ test { } java { - withJavadocJar() + // no javadoc for this module + //withJavadocJar() } task cleanGeneratedSrc(type: Delete) { From 191286fee295c10bc43c3fb6146d03744710c41a Mon Sep 17 00:00:00 2001 From: Nigel Jones Date: Wed, 1 Mar 2023 18:07:36 +0000 Subject: [PATCH 10/11] #7324 fix typo in pr build Signed-off-by: Nigel Jones --- .github/workflows/pr-v4.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/pr-v4.yml b/.github/workflows/pr-v4.yml index c3ee6e6a51a..3158aa96d5a 100644 --- a/.github/workflows/pr-v4.yml +++ b/.github/workflows/pr-v4.yml @@ -29,7 +29,6 @@ jobs: # Only cache for main build cache-read-only: true arguments: build publishToMavenLocal - arguments: build - name: Upload Test coverage report uses: actions/upload-artifact@v3 with: From 9eabccd1a00dd0011a2fdec28038c3cec11f302c Mon Sep 17 00:00:00 2001 From: Nigel Jones Date: Tue, 7 Mar 2023 12:29:21 +0000 Subject: [PATCH 11/11] #7324 remove open types fvt from codeQL build Signed-off-by: Nigel Jones --- .github/workflows/codeql-v4.yml | 2 +- settings.gradle | 15 ++++++++------- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/.github/workflows/codeql-v4.yml b/.github/workflows/codeql-v4.yml index c7930d3e4bb..30b687acd44 100644 --- a/.github/workflows/codeql-v4.yml +++ b/.github/workflows/codeql-v4.yml @@ -47,6 +47,6 @@ jobs: # codeQL requires a full before/after build to compare results. Caching can result in action failing #cache-read-only: true cache-disabled: true - arguments: -x javadoc -x test build + arguments: -x javadoc -x test build -PskipOpenTypesFVT - name: Perform CodeQL Analysis uses: github/codeql-action/analyze@v2 diff --git a/settings.gradle b/settings.gradle index 134c429e672..a7481754935 100644 --- a/settings.gradle +++ b/settings.gradle @@ -444,9 +444,10 @@ include(':open-metadata-test:open-metadata-fvt:access-services-fvt:stewardship-a include(':open-metadata-test:open-metadata-fvt:access-services-fvt:subject-area-fvt') include(':open-metadata-test:open-metadata-fvt:access-services-fvt') include(':open-metadata-test:open-metadata-fvt:fvt-utilities') -//TODO: restore tests once ready -include(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator') -include(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test') +if (!startParameter.projectProperties.hasProperty("skipOpenTypesFVT")) { + include(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator') + include(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test') +} project(':open-metadata-implementation:frameworks:audit-log-framework').projectDir = file('open-metadata-implementation/frameworks/audit-log-framework') project(':open-metadata-implementation:frameworks:open-connector-framework').projectDir = file('open-metadata-implementation/frameworks/open-connector-framework') project(':open-metadata-implementation:frameworks:open-discovery-framework').projectDir = file('open-metadata-implementation/frameworks/open-discovery-framework') @@ -859,7 +860,7 @@ project(':open-metadata-test:open-metadata-fvt:view-services-fvt:glossary-author project(':open-metadata-test:open-metadata-fvt:access-services-fvt').projectDir = file('open-metadata-test/open-metadata-fvt/access-services-fvt') project(':open-metadata-test:open-metadata-fvt:fvt-utilities').projectDir = file('open-metadata-test/open-metadata-fvt/fvt-utilities') project(':open-metadata-distribution:open-metadata-assemblies').projectDir = file('open-metadata-distribution/open-metadata-assemblies') - -project(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator').projectDir = file('open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator') -project(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test').projectDir = file('open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test') - +if (!startParameter.projectProperties.hasProperty("skipOpenTypesFVT")) { + project(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test-generator').projectDir = file('open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test-generator') + project(':open-metadata-test:open-metadata-fvt:open-types-fvt:open-types-test').projectDir = file('open-metadata-test/open-metadata-fvt/open-types-fvt/open-types-test') +}