From 2ef3886c9b0223f4813ac1385318d6f2e600a4a8 Mon Sep 17 00:00:00 2001 From: Tal Gitelman Date: Mon, 30 Apr 2018 19:06:20 +0300 Subject: Remove dead code asdctool Change-Id: I467f70b442a02a15b8a18b160309bac9a8ab3f9a Issue-ID: SDC-1270 Signed-off-by: Tal Gitelman --- .../sdc/asdctool/impl/ArtifactUuidFix.java | 2 +- .../sdc/asdctool/impl/GraphMLConverter.java | 2 -- .../asdctool/impl/validator/ArtifactToolBL.java | 4 --- .../asdctool/impl/validator/ValidationToolBL.java | 4 --- .../TopologyTemplateValidatorExecuter.java | 3 -- .../tasks/artifacts/VfArtifactValidationTask.java | 3 -- .../sdc/asdctool/main/ArtifactValidatorTool.java | 2 +- .../sdc/asdctool/main/DataSchemaMenu.java | 2 +- .../main/EsToCassandraDataMigrationMenu.java | 2 +- .../openecomp/sdc/asdctool/main/RemoveUtils.java | 2 +- .../sdc/asdctool/main/UpdateIsVnfMenu.java | 2 +- .../sdc/asdctool/main/ValidationTool.java | 2 +- .../tasks/mig1710/UpgradeMigration1710.java | 15 -------- .../servlets/ExportImportTitanServlet.java | 2 -- .../config/ValidationConfigManagerTest.java | 10 +++--- .../config/ValidationToolConfigurationTest.java | 40 +++++++++++----------- .../executers/ServiceValidatorExecuterTest.java | 2 +- .../TopologyTemplateValidatorExecuterTest.java | 4 +-- .../executers/VfValidatorExecuterTest.java | 2 +- .../tasks/artifacts/ArtifactsVertexResultTest.java | 4 +-- .../impl/validator/utils/ElementTypeEnumTest.java | 12 +++---- .../validator/utils/ValidationTaskResultTest.java | 12 +++---- .../impl/validator/utils/VertexResultTest.java | 6 ++-- .../sdc/asdctool/migration/core/DBVersionTest.java | 2 +- .../migration/core/SdcMigrationToolTest.java | 8 ++--- .../core/execution/MigrationExecutorImplTest.java | 2 +- .../migration/core/task/MigrationResultTest.java | 8 ++--- .../resolver/SpringBeansMigrationResolverTest.java | 8 ++--- .../migration/service/SdcRepoServiceTest.java | 8 ++--- .../migration/task/MigrationTasksTest.java | 6 ++-- .../sdc/asdctool/servlets/EntryPointTest.java | 2 +- 31 files changed, 75 insertions(+), 108 deletions(-) (limited to 'asdctool/src') diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/ArtifactUuidFix.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/ArtifactUuidFix.java index 5d4610c91f..d413c3f23a 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/ArtifactUuidFix.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/ArtifactUuidFix.java @@ -330,7 +330,7 @@ public class ArtifactUuidFix { return true; } - private boolean isProblematicService( Service service, String serviceName) throws IOException { + private boolean isProblematicService( Service service, String serviceName) { List componentInstances = service.getComponentInstances(); diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/GraphMLConverter.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/GraphMLConverter.java index 7bea2eaa4a..b6925416e6 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/GraphMLConverter.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/GraphMLConverter.java @@ -457,8 +457,6 @@ public class GraphMLConverter { } catch (Exception e) { e.printStackTrace(); - GraphPropertiesDictionary[] values = GraphPropertiesDictionary.values(); - Object property1 = vertex.value(GraphPropertiesDictionary.HEALTH_CHECK.getProperty()); System.out.println(property1); diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/ArtifactToolBL.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/ArtifactToolBL.java index 341a56f68a..f70236432d 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/ArtifactToolBL.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/ArtifactToolBL.java @@ -16,12 +16,8 @@ public class ArtifactToolBL { @Autowired protected List validators; - @Autowired - protected ReportManager reportManager; - private boolean allValid = true; - public boolean validateAll() { for (IArtifactValidatorExecuter validatorExec: validators) { System.out.println("ValidatorExecuter "+validatorExec.getName()+" started"); diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/ValidationToolBL.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/ValidationToolBL.java index e373107081..9406351a6c 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/ValidationToolBL.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/ValidationToolBL.java @@ -20,12 +20,8 @@ public class ValidationToolBL { @Autowired protected List validators; - @Autowired - protected ReportManager reportManager; - private boolean allValid = true; - public boolean validateAll() { for (ValidatorExecuter validatorExec: validators) { System.out.println("ValidatorExecuter "+validatorExec.getName()+" started"); diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuter.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuter.java index 04f014fbc9..f875aa4742 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuter.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuter.java @@ -34,9 +34,6 @@ public class TopologyTemplateValidatorExecuter { @Autowired protected TitanDao titanDao; - @Autowired - protected TopologyTemplateOperation topologyTemplateOperation; - protected String name; public void setName(String name) { diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/VfArtifactValidationTask.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/VfArtifactValidationTask.java index bb25d25e2b..1c18fb751e 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/VfArtifactValidationTask.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/VfArtifactValidationTask.java @@ -15,9 +15,6 @@ public class VfArtifactValidationTask extends VfValidationTask { @Autowired ArtifactValidationUtils artifactValidationUtils; - @Autowired - protected TopologyTemplateOperation topologyTemplateOperation; - public VfArtifactValidationTask() { this.name = "VF Artifact Validation Task"; } diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/ArtifactValidatorTool.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/ArtifactValidatorTool.java index 5ae543cb73..b1180f9483 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/ArtifactValidatorTool.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/ArtifactValidatorTool.java @@ -15,7 +15,7 @@ import org.springframework.context.annotation.AnnotationConfigApplicationContext public class ArtifactValidatorTool { private static Logger log = LoggerFactory.getLogger(ValidationTool.class.getName()); - public static void main(String[] args) throws Exception { + public static void main(String[] args) { String outputPath = args[0]; ValidationConfigManager.setOutputFullFilePath(outputPath); diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/DataSchemaMenu.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/DataSchemaMenu.java index 2fb3270bf3..b0e3dbea6d 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/DataSchemaMenu.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/DataSchemaMenu.java @@ -33,7 +33,7 @@ public class DataSchemaMenu { private static Logger log = LoggerFactory.getLogger(DataSchemaMenu.class.getName()); - public static void main(String[] args) throws Exception { + public static void main(String[] args) { String operation = args[0]; diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/EsToCassandraDataMigrationMenu.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/EsToCassandraDataMigrationMenu.java index 0bac1df353..f791098493 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/EsToCassandraDataMigrationMenu.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/EsToCassandraDataMigrationMenu.java @@ -34,7 +34,7 @@ public class EsToCassandraDataMigrationMenu { private static Logger log = LoggerFactory.getLogger(EsToCassandraDataMigrationMenu.class.getName()); - public static void main(String[] args) throws Exception { + public static void main(String[] args) { if (args == null || args.length < 2) { usageAndExit(); diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/RemoveUtils.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/RemoveUtils.java index ddece759c0..5546300740 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/RemoveUtils.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/RemoveUtils.java @@ -27,7 +27,7 @@ import org.openecomp.sdc.asdctool.impl.ProductLogic; */ public class RemoveUtils { - public static void main(String[] args) throws Exception { + public static void main(String[] args) { if (args == null || args.length < 1) { removeUsage(); diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/UpdateIsVnfMenu.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/UpdateIsVnfMenu.java index dce9f9c072..b9a1e1741b 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/UpdateIsVnfMenu.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/UpdateIsVnfMenu.java @@ -45,7 +45,7 @@ public class UpdateIsVnfMenu { "Usage: updateIsVnfTrue "); } - public static void main(String[] args) throws Exception { + public static void main(String[] args) { if (args == null || args.length < 1) { usageAndExit(); diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/ValidationTool.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/ValidationTool.java index 51e2d500da..e07b3af551 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/ValidationTool.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/main/ValidationTool.java @@ -19,7 +19,7 @@ public class ValidationTool { private static Logger log = LoggerFactory.getLogger(ValidationTool.class.getName()); - public static void main(String[] args) throws Exception { + public static void main(String[] args) { String outputPath = args[0]; ValidationConfigManager.setOutputFullFilePath(outputPath); diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/migration/tasks/mig1710/UpgradeMigration1710.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/migration/tasks/mig1710/UpgradeMigration1710.java index ca8c4c8a3c..e6acb7a1d4 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/migration/tasks/mig1710/UpgradeMigration1710.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/migration/tasks/mig1710/UpgradeMigration1710.java @@ -803,21 +803,6 @@ public class UpgradeMigration1710 implements PostMigration { return titanDao.getByCriteria(vertexType, propertiesToMatch, propertiesNotToMatch, JsonParseFlagEnum.ParseMetadata); } - protected Either, TitanOperationStatus> findResourcesPathRecursively(GraphVertex nodeTypeV, List allCertifiedUids) { - Either, TitanOperationStatus> parentResourceRes = titanDao.getParentVertecies(nodeTypeV, EdgeLabelEnum.DERIVED_FROM, JsonParseFlagEnum.ParseMetadata); - if (parentResourceRes.isRight()) { - return Either.right(parentResourceRes.right().value()); - } - List derivedResourcesUid = new ArrayList<>(); - for (GraphVertex chV : parentResourceRes.left().value()) { - Optional op = allCertifiedUids.stream().filter(id -> id.equals((String) chV.getJsonMetadataField(JsonPresentationFields.UNIQUE_ID))).findAny(); - if (op.isPresent()) { - derivedResourcesUid.add(chV); - } - } - return null; - } - private Either, StorageOperationStatus> getLatestByName(GraphPropertyEnum property, String nodeName) { Map propertiesToMatch = new EnumMap<>(GraphPropertyEnum.class); diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/servlets/ExportImportTitanServlet.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/servlets/ExportImportTitanServlet.java index 898bd96177..9d40354651 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/servlets/ExportImportTitanServlet.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/servlets/ExportImportTitanServlet.java @@ -161,8 +161,6 @@ public class ExportImportTitanServlet { // graph.commit(); graph.tx().commit(); - String exportedGraph = out.toString(); - result = outputFile; } catch (Exception e) { diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/config/ValidationConfigManagerTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/config/ValidationConfigManagerTest.java index 6adbdccf1d..1dd5926cc5 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/config/ValidationConfigManagerTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/config/ValidationConfigManagerTest.java @@ -13,7 +13,7 @@ public class ValidationConfigManagerTest { @Test - public void testGetOutputFilePath() throws Exception { + public void testGetOutputFilePath() { String result; // default test @@ -21,7 +21,7 @@ public class ValidationConfigManagerTest { } @Test - public void testGetCsvReportFilePath() throws Exception { + public void testGetCsvReportFilePath() { String result; // default test @@ -30,7 +30,7 @@ public class ValidationConfigManagerTest { @Test - public void testSetCsvReportFilePath() throws Exception { + public void testSetCsvReportFilePath() { String outputPath = ""; // default test @@ -39,7 +39,7 @@ public class ValidationConfigManagerTest { @Test - public void testSetValidationConfiguration() throws Exception { + public void testSetValidationConfiguration() { String path = ""; Properties result; @@ -49,7 +49,7 @@ public class ValidationConfigManagerTest { @Test - public void testGetValidationConfiguration() throws Exception { + public void testGetValidationConfiguration() { Properties result; // default test diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/config/ValidationToolConfigurationTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/config/ValidationToolConfigurationTest.java index fb916541c0..8dc2c4f0ec 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/config/ValidationToolConfigurationTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/config/ValidationToolConfigurationTest.java @@ -32,7 +32,7 @@ public class ValidationToolConfigurationTest { @Test - public void testBasicServiceValidator() throws Exception { + public void testBasicServiceValidator() { ValidationToolConfiguration testSubject; ServiceValidatorExecuter result; @@ -43,7 +43,7 @@ public class ValidationToolConfigurationTest { @Test - public void testVfArtifactValidationTask() throws Exception { + public void testVfArtifactValidationTask() { ValidationToolConfiguration testSubject; VfArtifactValidationTask result; @@ -54,7 +54,7 @@ public class ValidationToolConfigurationTest { @Test - public void testServiceArtifactValidationTask() throws Exception { + public void testServiceArtifactValidationTask() { ValidationToolConfiguration testSubject; ServiceArtifactValidationTask result; @@ -65,7 +65,7 @@ public class ValidationToolConfigurationTest { @Test - public void testModuleJsonTask() throws Exception { + public void testModuleJsonTask() { ValidationToolConfiguration testSubject; ModuleJsonTask result; @@ -76,7 +76,7 @@ public class ValidationToolConfigurationTest { @Test - public void testValidationToolBL() throws Exception { + public void testValidationToolBL() { ValidationToolConfiguration testSubject; ValidationToolBL result; @@ -87,7 +87,7 @@ public class ValidationToolConfigurationTest { @Test - public void testBasicVfValidator() throws Exception { + public void testBasicVfValidator() { ValidationToolConfiguration testSubject; VfValidatorExecuter result; @@ -97,7 +97,7 @@ public class ValidationToolConfigurationTest { } @Test - public void testArtifactCassandraDao() throws Exception { + public void testArtifactCassandraDao() { ValidationToolConfiguration testSubject; ArtifactCassandraDao result; @@ -108,7 +108,7 @@ public class ValidationToolConfigurationTest { @Test - public void testArtifactValidationUtils() throws Exception { + public void testArtifactValidationUtils() { ValidationToolConfiguration testSubject; ArtifactValidationUtils result; @@ -119,7 +119,7 @@ public class ValidationToolConfigurationTest { @Test - public void testJsonGroupsOperation() throws Exception { + public void testJsonGroupsOperation() { ValidationToolConfiguration testSubject; GroupsOperation result; @@ -130,7 +130,7 @@ public class ValidationToolConfigurationTest { @Test - public void testCassandraClient() throws Exception { + public void testCassandraClient() { ValidationToolConfiguration testSubject; CassandraClient result; @@ -141,7 +141,7 @@ public class ValidationToolConfigurationTest { @Test - public void testDaoStrategy() throws Exception { + public void testDaoStrategy() { ValidationToolConfiguration testSubject; TitanClientStrategy result; @@ -155,7 +155,7 @@ public class ValidationToolConfigurationTest { @Test - public void testToscaOperationFacade() throws Exception { + public void testToscaOperationFacade() { ValidationToolConfiguration testSubject; ToscaOperationFacade result; @@ -166,7 +166,7 @@ public class ValidationToolConfigurationTest { @Test - public void testNodeTypeOperation() throws Exception { + public void testNodeTypeOperation() { ValidationToolConfiguration testSubject; DerivedNodeTypeResolver migrationDerivedNodeTypeResolver = null; NodeTypeOperation result; @@ -178,7 +178,7 @@ public class ValidationToolConfigurationTest { @Test - public void testTopologyTemplateOperation() throws Exception { + public void testTopologyTemplateOperation() { ValidationToolConfiguration testSubject; TopologyTemplateOperation result; @@ -192,7 +192,7 @@ public class ValidationToolConfigurationTest { @Test - public void testMigrationDerivedNodeTypeResolver() throws Exception { + public void testMigrationDerivedNodeTypeResolver() { ValidationToolConfiguration testSubject; DerivedNodeTypeResolver result; @@ -203,7 +203,7 @@ public class ValidationToolConfigurationTest { @Test - public void testTitanDao() throws Exception { + public void testTitanDao() { ValidationToolConfiguration testSubject; TitanGraphClient titanGraphClient = null; TitanDao result; @@ -215,7 +215,7 @@ public class ValidationToolConfigurationTest { @Test - public void testCategoryOperation() throws Exception { + public void testCategoryOperation() { ValidationToolConfiguration testSubject; CategoryOperation result; @@ -226,7 +226,7 @@ public class ValidationToolConfigurationTest { @Test - public void testArtifactsOperation() throws Exception { + public void testArtifactsOperation() { ValidationToolConfiguration testSubject; ArtifactsOperations result; @@ -237,7 +237,7 @@ public class ValidationToolConfigurationTest { @Test - public void testToscaDataOperation() throws Exception { + public void testToscaDataOperation() { ValidationToolConfiguration testSubject; ToscaDataOperation result; @@ -248,7 +248,7 @@ public class ValidationToolConfigurationTest { @Test - public void testToscaElementLifecycleOperation() throws Exception { + public void testToscaElementLifecycleOperation() { ValidationToolConfiguration testSubject; ToscaElementLifecycleOperation result; diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/ServiceValidatorExecuterTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/ServiceValidatorExecuterTest.java index 05acd9aa6f..890bdfd002 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/ServiceValidatorExecuterTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/ServiceValidatorExecuterTest.java @@ -14,7 +14,7 @@ public class ServiceValidatorExecuterTest { @Test - public void testGetName() throws Exception { + public void testGetName() { ServiceValidatorExecuter testSubject; String result; diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuterTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuterTest.java index 3747d7fd09..f3fdebe8e9 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuterTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuterTest.java @@ -11,7 +11,7 @@ public class TopologyTemplateValidatorExecuterTest { @Test - public void testSetName() throws Exception { + public void testSetName() { TopologyTemplateValidatorExecuter testSubject; String name = ""; @@ -22,7 +22,7 @@ public class TopologyTemplateValidatorExecuterTest { @Test - public void testGetName() throws Exception { + public void testGetName() { TopologyTemplateValidatorExecuter testSubject; String result; diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/VfValidatorExecuterTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/VfValidatorExecuterTest.java index 12136df609..83dac8a35a 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/VfValidatorExecuterTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/executers/VfValidatorExecuterTest.java @@ -12,7 +12,7 @@ public class VfValidatorExecuterTest { @Test - public void testGetName() throws Exception { + public void testGetName() { VfValidatorExecuter testSubject; String result; diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/ArtifactsVertexResultTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/ArtifactsVertexResultTest.java index 8ba509cf9c..d16c3fc555 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/ArtifactsVertexResultTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/ArtifactsVertexResultTest.java @@ -11,7 +11,7 @@ public class ArtifactsVertexResultTest { @Test - public void testAddNotFoundArtifact() throws Exception { + public void testAddNotFoundArtifact() { ArtifactsVertexResult testSubject; String artifactId = ""; @@ -22,7 +22,7 @@ public class ArtifactsVertexResultTest { @Test - public void testGetResult() throws Exception { + public void testGetResult() { ArtifactsVertexResult testSubject; String result; diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ElementTypeEnumTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ElementTypeEnumTest.java index 7467699fc0..e6e564e709 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ElementTypeEnumTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ElementTypeEnumTest.java @@ -13,7 +13,7 @@ public class ElementTypeEnumTest { @Test - public void testGetByType() throws Exception { + public void testGetByType() { String elementType = ""; ElementTypeEnum result; @@ -23,7 +23,7 @@ public class ElementTypeEnumTest { @Test - public void testGetAllTypes() throws Exception { + public void testGetAllTypes() { List result; // default test @@ -32,7 +32,7 @@ public class ElementTypeEnumTest { @Test - public void testGetElementType() throws Exception { + public void testGetElementType() { ElementTypeEnum testSubject; String result; @@ -43,7 +43,7 @@ public class ElementTypeEnumTest { @Test - public void testSetElementType() throws Exception { + public void testSetElementType() { ElementTypeEnum testSubject; String elementType = ""; @@ -54,7 +54,7 @@ public class ElementTypeEnumTest { @Test - public void testGetClazz() throws Exception { + public void testGetClazz() { ElementTypeEnum testSubject; Class result; @@ -65,7 +65,7 @@ public class ElementTypeEnumTest { @Test - public void testSetClazz() throws Exception { + public void testSetClazz() { ElementTypeEnum testSubject; Class clazz = null; diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ValidationTaskResultTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ValidationTaskResultTest.java index d51075bc89..78ef2e629f 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ValidationTaskResultTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ValidationTaskResultTest.java @@ -12,7 +12,7 @@ public class ValidationTaskResultTest { @Test - public void testGetName() throws Exception { + public void testGetName() { ValidationTaskResult testSubject; String result; @@ -23,7 +23,7 @@ public class ValidationTaskResultTest { @Test - public void testSetName() throws Exception { + public void testSetName() { ValidationTaskResult testSubject; String name = ""; @@ -34,7 +34,7 @@ public class ValidationTaskResultTest { @Test - public void testGetResultMessage() throws Exception { + public void testGetResultMessage() { ValidationTaskResult testSubject; String result; @@ -45,7 +45,7 @@ public class ValidationTaskResultTest { @Test - public void testSetResultMessage() throws Exception { + public void testSetResultMessage() { ValidationTaskResult testSubject; String resultMessage = ""; @@ -56,7 +56,7 @@ public class ValidationTaskResultTest { @Test - public void testIsSuccessful() throws Exception { + public void testIsSuccessful() { ValidationTaskResult testSubject; boolean result; @@ -67,7 +67,7 @@ public class ValidationTaskResultTest { @Test - public void testSetSuccessful() throws Exception { + public void testSetSuccessful() { ValidationTaskResult testSubject; boolean successful = false; diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/VertexResultTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/VertexResultTest.java index 9b44eef1f5..d65fb001f8 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/VertexResultTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/VertexResultTest.java @@ -11,7 +11,7 @@ public class VertexResultTest { @Test - public void testGetStatus() throws Exception { + public void testGetStatus() { VertexResult testSubject; boolean result; @@ -22,7 +22,7 @@ public class VertexResultTest { @Test - public void testSetStatus() throws Exception { + public void testSetStatus() { VertexResult testSubject; boolean status = false; @@ -33,7 +33,7 @@ public class VertexResultTest { @Test - public void testGetResult() throws Exception { + public void testGetResult() { VertexResult testSubject; String result; diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/DBVersionTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/DBVersionTest.java index afdd0d6da8..d5473cd121 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/DBVersionTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/DBVersionTest.java @@ -51,7 +51,7 @@ public class DBVersionTest { } @Test(dataProvider = "versionComparisionProvider") - public void testVersionCompareTo2(String firstVersion, String otherVersion, int expectedComparisionResult) throws Exception { + public void testVersionCompareTo2(String firstVersion, String otherVersion, int expectedComparisionResult) { assertEquals(DBVersion.fromString(firstVersion).compareTo(DBVersion.fromString(otherVersion)), expectedComparisionResult); } } diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/SdcMigrationToolTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/SdcMigrationToolTest.java index fe74891950..8783071842 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/SdcMigrationToolTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/SdcMigrationToolTest.java @@ -31,12 +31,12 @@ public class SdcMigrationToolTest { private SdcRepoService sdcRepoServiceMock; @BeforeMethod - public void setUp() throws Exception { + public void setUp() { MockitoAnnotations.initMocks(this); } @Test - public void testMigrate_noMigrations() throws Exception { + public void testMigrate_noMigrations() { when(migrationResolverMock.resolveMigrations()).thenReturn(Collections.emptyList()); testInstance.migrate(false); verify(sdcRepoServiceMock, new Times(0)).clearTasksForCurrentMajor(); @@ -44,14 +44,14 @@ public class SdcMigrationToolTest { } @Test - public void testMigrate_enforceFlag_removeAllMigrationTasksForCurrentVersion() throws Exception { + public void testMigrate_enforceFlag_removeAllMigrationTasksForCurrentVersion() { when(migrationResolverMock.resolveMigrations()).thenReturn(Collections.emptyList()); testInstance.migrate(true); verify(sdcRepoServiceMock, new Times(1)).clearTasksForCurrentMajor(); } @Test - public void testMigrate_stopAfterFirstFailure() throws Exception { + public void testMigrate_stopAfterFirstFailure() { when(migrationResolverMock.resolveMigrations()).thenReturn(Arrays.asList(new SuccessfulMigration(), new FailedMigration(), new SuccessfulMigration())); testInstance.migrate(false); verify(sdcRepoServiceMock, new Times(0)).clearTasksForCurrentMajor(); diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/execution/MigrationExecutorImplTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/execution/MigrationExecutorImplTest.java index 9547abc00f..3f9766d9c5 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/execution/MigrationExecutorImplTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/execution/MigrationExecutorImplTest.java @@ -11,7 +11,7 @@ import org.testng.annotations.Test; public class MigrationExecutorImplTest { @Test - public void testExecuteMigration() throws Exception { + public void testExecuteMigration() { MigrationExecutionResult execute = new MigrationExecutorImpl().execute(DummyMigrationFactory.SUCCESSFUL_MIGRATION); assertMigrationTaskEntryByMigrationExecutionResult(execute, DummyMigrationFactory.SUCCESSFUL_MIGRATION); } diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/task/MigrationResultTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/task/MigrationResultTest.java index 04a8bd213b..8167b927e9 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/task/MigrationResultTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/core/task/MigrationResultTest.java @@ -12,7 +12,7 @@ public class MigrationResultTest { @Test - public void testGetMsg() throws Exception { + public void testGetMsg() { MigrationResult testSubject; String result; @@ -23,7 +23,7 @@ public class MigrationResultTest { @Test - public void testSetMsg() throws Exception { + public void testSetMsg() { MigrationResult testSubject; String msg = ""; @@ -34,7 +34,7 @@ public class MigrationResultTest { @Test - public void testGetMigrationStatus() throws Exception { + public void testGetMigrationStatus() { MigrationResult testSubject; MigrationStatus result; @@ -45,7 +45,7 @@ public class MigrationResultTest { @Test - public void testSetMigrationStatus() throws Exception { + public void testSetMigrationStatus() { MigrationResult testSubject; MigrationStatus migrationStatus = null; diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/resolver/SpringBeansMigrationResolverTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/resolver/SpringBeansMigrationResolverTest.java index 360b3aab40..636b1c6744 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/resolver/SpringBeansMigrationResolverTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/resolver/SpringBeansMigrationResolverTest.java @@ -31,13 +31,13 @@ public class SpringBeansMigrationResolverTest { @BeforeMethod - public void setUp() throws Exception { + public void setUp() { MockitoAnnotations.initMocks(this); testInstance.setMigrations(migrations); } @Test - public void testResolveMigrations_getMigrationsWithVersionGreaterThanLatest() throws Exception { + public void testResolveMigrations_getMigrationsWithVersionGreaterThanLatest() { when(sdcRepoServiceMock.getLatestDBVersion()).thenReturn(DBVersion.fromString("1710.2")); testInstance.setPostMigrations(Collections.emptyList()); List resolvedMigrations = testInstance.resolveMigrations(); @@ -47,7 +47,7 @@ public class SpringBeansMigrationResolverTest { } @Test - public void testResolveMigration_noLatestVersionForCurrentMajorVersion() throws Exception { + public void testResolveMigration_noLatestVersionForCurrentMajorVersion() { when(sdcRepoServiceMock.getLatestDBVersion()).thenReturn(DBVersion.fromString("1710.-1")); testInstance.setPostMigrations(Collections.emptyList()); List resolvedMigrations = testInstance.resolveMigrations(); @@ -58,7 +58,7 @@ public class SpringBeansMigrationResolverTest { } @Test - public void testResolveMigrations_emptyMigrationsList() throws Exception { + public void testResolveMigrations_emptyMigrationsList() { testInstance.setMigrations(Collections.emptyList()); testInstance.setPostMigrations(Collections.emptyList()); when(sdcRepoServiceMock.getLatestDBVersion()).thenReturn(DBVersion.fromString("1710.-1")); diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/service/SdcRepoServiceTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/service/SdcRepoServiceTest.java index 31e1a63689..2745f59006 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/service/SdcRepoServiceTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/service/SdcRepoServiceTest.java @@ -25,12 +25,12 @@ public class SdcRepoServiceTest { private MigrationTasksDao migrationTasksDaoMock; @BeforeMethod - public void setUp() throws Exception { + public void setUp() { MockitoAnnotations.initMocks(this); } @Test - public void testGetLatestVersion_noMinorVersionForCurrentVersion() throws Exception { + public void testGetLatestVersion_noMinorVersionForCurrentVersion() { when(migrationTasksDaoMock.getLatestMinorVersion(DBVersion.CURRENT_VERSION.getMajor())).thenReturn(null); DBVersion latestDBVersion = testInstance.getLatestDBVersion(); assertEquals(latestDBVersion.getMajor(), DBVersion.CURRENT_VERSION.getMajor()); @@ -38,7 +38,7 @@ public class SdcRepoServiceTest { } @Test - public void testGetLatestVersion() throws Exception { + public void testGetLatestVersion() { when(migrationTasksDaoMock.getLatestMinorVersion(DBVersion.CURRENT_VERSION.getMajor())).thenReturn(BigInteger.TEN); DBVersion latestDBVersion = testInstance.getLatestDBVersion(); assertEquals(latestDBVersion.getMajor(), DBVersion.CURRENT_VERSION.getMajor()); @@ -46,7 +46,7 @@ public class SdcRepoServiceTest { } @Test - public void testCreateMigrationTask() throws Exception { + public void testCreateMigrationTask() { MigrationTaskEntry taskEntry = new MigrationTaskEntry(); testInstance.createMigrationTask(taskEntry); verify(migrationTasksDaoMock, new Times(1)).createMigrationTask(taskEntry); diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/task/MigrationTasksTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/task/MigrationTasksTest.java index fa61b7e19e..52dc7c9fcd 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/task/MigrationTasksTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/migration/task/MigrationTasksTest.java @@ -21,13 +21,13 @@ public class MigrationTasksTest { private List migrations; @BeforeMethod - public void setUp() throws Exception { + public void setUp() { ClassScanner classScanner = new ClassScanner(); migrations = classScanner.getAllClassesOfType(MIGRATIONS_BASE_PACKAGE, Migration.class); } @Test - public void testNoTasksWithSameVersion() throws Exception { + public void testNoTasksWithSameVersion() { Map> migrationsByVersion = migrations.stream().collect(Collectors.groupingBy(Migration::getVersion)); migrationsByVersion.forEach((version, migrations) -> { if (migrations.size() > 1) { @@ -38,7 +38,7 @@ public class MigrationTasksTest { } @Test - public void testNoTaskWithVersionGreaterThanCurrentVersion() throws Exception { + public void testNoTaskWithVersionGreaterThanCurrentVersion() { Set migrationsWithVersionsGreaterThanCurrent = migrations.stream().filter(mig -> mig.getVersion().compareTo(DBVersion.CURRENT_VERSION) > 0) .collect(Collectors.toSet()); diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/servlets/EntryPointTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/servlets/EntryPointTest.java index 217bc3919f..4b0b398a81 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/servlets/EntryPointTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/servlets/EntryPointTest.java @@ -11,7 +11,7 @@ public class EntryPointTest { @Test - public void testTest() throws Exception { + public void testTest() { EntryPoint testSubject; String result; -- cgit 1.2.3-korg