[ https://issues.apache.org/jira/browse/KAFKA-7431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16678483#comment-16678483 ]
ASF GitHub Bot commented on KAFKA-7431: --------------------------------------- ewencp closed pull request #5681: KAFKA-7431 : Clean up connect unit tests URL: https://github.com/apache/kafka/pull/5681 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/connect/api/src/test/java/org/apache/kafka/connect/connector/ConnectorReconfigurationTest.java b/connect/api/src/test/java/org/apache/kafka/connect/connector/ConnectorReconfigurationTest.java index 85a6cfae3f1..6f768f9f2dc 100644 --- a/connect/api/src/test/java/org/apache/kafka/connect/connector/ConnectorReconfigurationTest.java +++ b/connect/api/src/test/java/org/apache/kafka/connect/connector/ConnectorReconfigurationTest.java @@ -29,7 +29,7 @@ public class ConnectorReconfigurationTest { @Test - public void testDefaultReconfigure() throws Exception { + public void testDefaultReconfigure() { TestConnector conn = new TestConnector(false); conn.reconfigure(Collections.<String, String>emptyMap()); assertEquals(conn.stopOrder, 0); @@ -37,7 +37,7 @@ public void testDefaultReconfigure() throws Exception { } @Test(expected = ConnectException.class) - public void testReconfigureStopException() throws Exception { + public void testReconfigureStopException() { TestConnector conn = new TestConnector(true); conn.reconfigure(Collections.<String, String>emptyMap()); } diff --git a/connect/api/src/test/java/org/apache/kafka/connect/data/SchemaProjectorTest.java b/connect/api/src/test/java/org/apache/kafka/connect/data/SchemaProjectorTest.java index 151114e2d29..d2e8acbe90b 100644 --- a/connect/api/src/test/java/org/apache/kafka/connect/data/SchemaProjectorTest.java +++ b/connect/api/src/test/java/org/apache/kafka/connect/data/SchemaProjectorTest.java @@ -35,7 +35,7 @@ public class SchemaProjectorTest { @Test - public void testPrimitiveTypeProjection() throws Exception { + public void testPrimitiveTypeProjection() { Object projected; projected = SchemaProjector.project(Schema.BOOLEAN_SCHEMA, false, Schema.BOOLEAN_SCHEMA); assertEquals(false, projected); @@ -79,7 +79,7 @@ public void testPrimitiveTypeProjection() throws Exception { } @Test - public void testNumericTypeProjection() throws Exception { + public void testNumericTypeProjection() { Schema[] promotableSchemas = {Schema.INT8_SCHEMA, Schema.INT16_SCHEMA, Schema.INT32_SCHEMA, Schema.INT64_SCHEMA, Schema.FLOAT32_SCHEMA, Schema.FLOAT64_SCHEMA}; Schema[] promotableOptionalSchemas = {Schema.OPTIONAL_INT8_SCHEMA, Schema.OPTIONAL_INT16_SCHEMA, Schema.OPTIONAL_INT32_SCHEMA, Schema.OPTIONAL_INT64_SCHEMA, Schema.OPTIONAL_FLOAT32_SCHEMA, Schema.OPTIONAL_FLOAT64_SCHEMA}; @@ -146,7 +146,7 @@ public void testNumericTypeProjection() throws Exception { } @Test - public void testPrimitiveOptionalProjection() throws Exception { + public void testPrimitiveOptionalProjection() { verifyOptionalProjection(Schema.OPTIONAL_BOOLEAN_SCHEMA, Type.BOOLEAN, false, true, false, true); verifyOptionalProjection(Schema.OPTIONAL_BOOLEAN_SCHEMA, Type.BOOLEAN, false, true, false, false); @@ -208,7 +208,7 @@ public void testPrimitiveOptionalProjection() throws Exception { } @Test - public void testStructAddField() throws Exception { + public void testStructAddField() { Schema source = SchemaBuilder.struct() .field("field", Schema.INT32_SCHEMA) .build(); @@ -240,7 +240,7 @@ public void testStructAddField() throws Exception { } @Test - public void testStructRemoveField() throws Exception { + public void testStructRemoveField() { Schema source = SchemaBuilder.struct() .field("field", Schema.INT32_SCHEMA) .field("field2", Schema.INT32_SCHEMA) @@ -264,7 +264,7 @@ public void testStructRemoveField() throws Exception { } @Test - public void testStructDefaultValue() throws Exception { + public void testStructDefaultValue() { Schema source = SchemaBuilder.struct().optional() .field("field", Schema.INT32_SCHEMA) .field("field2", Schema.INT32_SCHEMA) @@ -289,7 +289,7 @@ public void testStructDefaultValue() throws Exception { } @Test - public void testNestedSchemaProjection() throws Exception { + public void testNestedSchemaProjection() { Schema sourceFlatSchema = SchemaBuilder.struct() .field("field", Schema.INT32_SCHEMA) .build(); @@ -326,8 +326,8 @@ public void testNestedSchemaProjection() throws Exception { targetNestedSchema); assertEquals(1, targetNestedStruct.get("first")); assertEquals("abc", targetNestedStruct.get("second")); - assertEquals(Arrays.asList(1, 2), (List<Integer>) targetNestedStruct.get("array")); - assertEquals(Collections.singletonMap(5, "def"), (Map<Integer, String>) targetNestedStruct.get("map")); + assertEquals(Arrays.asList(1, 2), targetNestedStruct.get("array")); + assertEquals(Collections.singletonMap(5, "def"), targetNestedStruct.get("map")); Struct projectedStruct = (Struct) targetNestedStruct.get("nested"); assertEquals(113, projectedStruct.get("field")); @@ -335,7 +335,7 @@ public void testNestedSchemaProjection() throws Exception { } @Test - public void testLogicalTypeProjection() throws Exception { + public void testLogicalTypeProjection() { Schema[] logicalTypeSchemas = {Decimal.schema(2), Date.SCHEMA, Time.SCHEMA, Timestamp.SCHEMA}; Object projected; @@ -378,25 +378,25 @@ public void testLogicalTypeProjection() throws Exception { } @Test - public void testArrayProjection() throws Exception { + public void testArrayProjection() { Schema source = SchemaBuilder.array(Schema.INT32_SCHEMA).build(); Object projected = SchemaProjector.project(source, Arrays.asList(1, 2, 3), source); - assertEquals(Arrays.asList(1, 2, 3), (List<Integer>) projected); + assertEquals(Arrays.asList(1, 2, 3), projected); Schema optionalSource = SchemaBuilder.array(Schema.INT32_SCHEMA).optional().build(); Schema target = SchemaBuilder.array(Schema.INT32_SCHEMA).defaultValue(Arrays.asList(1, 2, 3)).build(); projected = SchemaProjector.project(optionalSource, Arrays.asList(4, 5), target); - assertEquals(Arrays.asList(4, 5), (List<Integer>) projected); + assertEquals(Arrays.asList(4, 5), projected); projected = SchemaProjector.project(optionalSource, null, target); - assertEquals(Arrays.asList(1, 2, 3), (List<Integer>) projected); + assertEquals(Arrays.asList(1, 2, 3), projected); Schema promotedTarget = SchemaBuilder.array(Schema.INT64_SCHEMA).defaultValue(Arrays.asList(1L, 2L, 3L)).build(); projected = SchemaProjector.project(optionalSource, Arrays.asList(4, 5), promotedTarget); List<Long> expectedProjected = Arrays.asList(4L, 5L); - assertEquals(expectedProjected, (List<Long>) projected); + assertEquals(expectedProjected, projected); projected = SchemaProjector.project(optionalSource, null, promotedTarget); - assertEquals(Arrays.asList(1L, 2L, 3L), (List<Long>) projected); + assertEquals(Arrays.asList(1L, 2L, 3L), projected); Schema noDefaultValueTarget = SchemaBuilder.array(Schema.INT32_SCHEMA).build(); try { @@ -416,21 +416,21 @@ public void testArrayProjection() throws Exception { } @Test - public void testMapProjection() throws Exception { + public void testMapProjection() { Schema source = SchemaBuilder.map(Schema.INT32_SCHEMA, Schema.INT32_SCHEMA).optional().build(); Schema target = SchemaBuilder.map(Schema.INT32_SCHEMA, Schema.INT32_SCHEMA).defaultValue(Collections.singletonMap(1, 2)).build(); Object projected = SchemaProjector.project(source, Collections.singletonMap(3, 4), target); - assertEquals(Collections.singletonMap(3, 4), (Map<Integer, Integer>) projected); + assertEquals(Collections.singletonMap(3, 4), projected); projected = SchemaProjector.project(source, null, target); - assertEquals(Collections.singletonMap(1, 2), (Map<Integer, Integer>) projected); + assertEquals(Collections.singletonMap(1, 2), projected); Schema promotedTarget = SchemaBuilder.map(Schema.INT64_SCHEMA, Schema.FLOAT32_SCHEMA).defaultValue( Collections.singletonMap(3L, 4.5F)).build(); projected = SchemaProjector.project(source, Collections.singletonMap(3, 4), promotedTarget); - assertEquals(Collections.singletonMap(3L, 4.F), (Map<Long, Float>) projected); + assertEquals(Collections.singletonMap(3L, 4.F), projected); projected = SchemaProjector.project(source, null, promotedTarget); - assertEquals(Collections.singletonMap(3L, 4.5F), (Map<Long, Float>) projected); + assertEquals(Collections.singletonMap(3L, 4.5F), projected); Schema noDefaultValueTarget = SchemaBuilder.map(Schema.INT32_SCHEMA, Schema.INT32_SCHEMA).build(); try { @@ -450,7 +450,7 @@ public void testMapProjection() throws Exception { } @Test - public void testMaybeCompatible() throws Exception { + public void testMaybeCompatible() { Schema source = SchemaBuilder.int32().name("source").build(); Schema target = SchemaBuilder.int32().name("target").build(); diff --git a/connect/basic-auth-extension/src/test/java/org/apache/kafka/connect/rest/basic/auth/extension/JaasBasicAuthFilterTest.java b/connect/basic-auth-extension/src/test/java/org/apache/kafka/connect/rest/basic/auth/extension/JaasBasicAuthFilterTest.java index d61fc06cd90..835bef0f37c 100644 --- a/connect/basic-auth-extension/src/test/java/org/apache/kafka/connect/rest/basic/auth/extension/JaasBasicAuthFilterTest.java +++ b/connect/basic-auth-extension/src/test/java/org/apache/kafka/connect/rest/basic/auth/extension/JaasBasicAuthFilterTest.java @@ -53,7 +53,7 @@ private Configuration previousConfiguration; @Before - public void setup() throws IOException { + public void setup() { EasyMock.reset(requestContext); } diff --git a/connect/json/src/test/java/org/apache/kafka/connect/json/JsonConverterTest.java b/connect/json/src/test/java/org/apache/kafka/connect/json/JsonConverterTest.java index 7686fdb7ab5..5bc27db7cd1 100644 --- a/connect/json/src/test/java/org/apache/kafka/connect/json/JsonConverterTest.java +++ b/connect/json/src/test/java/org/apache/kafka/connect/json/JsonConverterTest.java @@ -588,7 +588,7 @@ public void decimalToJson() throws IOException { } @Test - public void dateToJson() throws IOException { + public void dateToJson() { GregorianCalendar calendar = new GregorianCalendar(1970, Calendar.JANUARY, 1, 0, 0, 0); calendar.setTimeZone(TimeZone.getTimeZone("UTC")); calendar.add(Calendar.DATE, 10000); @@ -604,7 +604,7 @@ public void dateToJson() throws IOException { } @Test - public void timeToJson() throws IOException { + public void timeToJson() { GregorianCalendar calendar = new GregorianCalendar(1970, Calendar.JANUARY, 1, 0, 0, 0); calendar.setTimeZone(TimeZone.getTimeZone("UTC")); calendar.add(Calendar.MILLISECOND, 14400000); @@ -620,7 +620,7 @@ public void timeToJson() throws IOException { } @Test - public void timestampToJson() throws IOException { + public void timestampToJson() { GregorianCalendar calendar = new GregorianCalendar(1970, Calendar.JANUARY, 1, 0, 0, 0); calendar.setTimeZone(TimeZone.getTimeZone("UTC")); calendar.add(Calendar.MILLISECOND, 2000000000); @@ -756,7 +756,7 @@ public void testJsonSchemaCacheSizeFromConfigFile() throws URISyntaxException, I // The following simply verify that the delegation works. @Test - public void testStringHeaderToJson() throws UnsupportedEncodingException { + public void testStringHeaderToJson() { JsonNode converted = parse(converter.fromConnectHeader(TOPIC, "headerName", Schema.STRING_SCHEMA, "test-string")); validateEnvelope(converted); assertEquals(parse("{ \"type\": \"string\", \"optional\": false }"), converted.get(JsonSchema.ENVELOPE_SCHEMA_FIELD_NAME)); diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/AbstractHerderTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/AbstractHerderTest.java index 8dbda185401..b65612ab095 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/AbstractHerderTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/AbstractHerderTest.java @@ -300,7 +300,7 @@ public void testConfigValidationTransformsExtendResults() { } @Test - public void testReverseTransformConfigs() throws Exception { + public void testReverseTransformConfigs() { // Construct a task config with constant values for TEST_KEY and TEST_KEY2 Map<String, String> newTaskConfig = new HashMap<>(); newTaskConfig.put(TaskConfig.TASK_CLASS_CONFIG, BogusSourceTask.class.getName()); diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/SourceTaskOffsetCommitterTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/SourceTaskOffsetCommitterTest.java index 2d5da983d29..e9511443233 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/SourceTaskOffsetCommitterTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/SourceTaskOffsetCommitterTest.java @@ -82,7 +82,7 @@ public void setup() { } @Test - public void testSchedule() throws Exception { + public void testSchedule() { Capture<Runnable> taskWrapper = EasyMock.newCapture(); EasyMock.expect(executor.scheduleWithFixedDelay( diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerConfigTransformerTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerConfigTransformerTest.java index 300022de76e..b5da2646c3d 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerConfigTransformerTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerConfigTransformerTest.java @@ -61,13 +61,13 @@ public void setup() { } @Test - public void testReplaceVariable() throws Exception { + public void testReplaceVariable() { Map<String, String> result = configTransformer.transform(MY_CONNECTOR, Collections.singletonMap(MY_KEY, "${test:testPath:testKey}")); assertEquals(TEST_RESULT, result.get(MY_KEY)); } @Test - public void testReplaceVariableWithTTL() throws Exception { + public void testReplaceVariableWithTTL() { EasyMock.expect(worker.herder()).andReturn(herder); EasyMock.expect(herder.connectorConfigReloadAction(MY_CONNECTOR)).andReturn(Herder.ConfigReloadAction.NONE); @@ -78,7 +78,7 @@ public void testReplaceVariableWithTTL() throws Exception { } @Test - public void testReplaceVariableWithTTLAndScheduleRestart() throws Exception { + public void testReplaceVariableWithTTLAndScheduleRestart() { EasyMock.expect(worker.herder()).andReturn(herder); EasyMock.expect(herder.connectorConfigReloadAction(MY_CONNECTOR)).andReturn(Herder.ConfigReloadAction.RESTART); EasyMock.expect(herder.restartConnector(1L, MY_CONNECTOR, null)).andReturn(requestId); @@ -90,7 +90,7 @@ public void testReplaceVariableWithTTLAndScheduleRestart() throws Exception { } @Test - public void testReplaceVariableWithTTLFirstCancelThenScheduleRestart() throws Exception { + public void testReplaceVariableWithTTLFirstCancelThenScheduleRestart() { EasyMock.expect(worker.herder()).andReturn(herder); EasyMock.expect(herder.connectorConfigReloadAction(MY_CONNECTOR)).andReturn(Herder.ConfigReloadAction.RESTART); EasyMock.expect(herder.restartConnector(1L, MY_CONNECTOR, null)).andReturn(requestId); diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerSinkTaskTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerSinkTaskTest.java index 33ab2ef06e0..fad44451dbf 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerSinkTaskTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerSinkTaskTest.java @@ -1407,7 +1407,7 @@ private void assertTaskMetricValue(String name, String expected) { } private void printMetrics() { - System.out.println(""); + System.out.println(); sinkMetricValue("sink-record-read-rate"); sinkMetricValue("sink-record-read-total"); sinkMetricValue("sink-record-send-rate"); diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerTest.java index 77238e9aaad..4dd560a63e1 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/WorkerTest.java @@ -132,7 +132,7 @@ public void setup() { } @Test - public void testStartAndStopConnector() throws Exception { + public void testStartAndStopConnector() { expectConverters(); expectStartStorage(); @@ -201,7 +201,7 @@ public void testStartAndStopConnector() throws Exception { } @Test - public void testStartConnectorFailure() throws Exception { + public void testStartConnectorFailure() { expectConverters(); expectStartStorage(); @@ -245,7 +245,7 @@ public void testStartConnectorFailure() throws Exception { } @Test - public void testAddConnectorByAlias() throws Exception { + public void testAddConnectorByAlias() { expectConverters(); expectStartStorage(); @@ -309,7 +309,7 @@ public void testAddConnectorByAlias() throws Exception { } @Test - public void testAddConnectorByShortAlias() throws Exception { + public void testAddConnectorByShortAlias() { expectConverters(); expectStartStorage(); @@ -385,7 +385,7 @@ public void testStopInvalidConnector() { } @Test - public void testReconfigureConnectorTasks() throws Exception { + public void testReconfigureConnectorTasks() { expectConverters(); expectStartStorage(); @@ -567,7 +567,7 @@ public void testAddRemoveTask() throws Exception { } @Test - public void testStartTaskFailure() throws Exception { + public void testStartTaskFailure() { expectConverters(); expectStartStorage(); diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/distributed/DistributedHerderTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/distributed/DistributedHerderTest.java index a0de8cf14ac..5f450b1693e 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/distributed/DistributedHerderTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/distributed/DistributedHerderTest.java @@ -836,7 +836,7 @@ public void testRestartUnknownTask() throws Exception { } @Test - public void testRequestProcessingOrder() throws Exception { + public void testRequestProcessingOrder() { final DistributedHerder.DistributedHerderRequest req1 = herder.addRequest(100, null, null); final DistributedHerder.DistributedHerderRequest req2 = herder.addRequest(10, null, null); final DistributedHerder.DistributedHerderRequest req3 = herder.addRequest(200, null, null); @@ -1423,7 +1423,7 @@ public Object answer() throws Throwable { } @Test - public void testInconsistentConfigs() throws Exception { + public void testInconsistentConfigs() { // FIXME: if we have inconsistent configs, we need to request forced reconfig + write of the connector's task configs // This requires inter-worker communication, so needs the REST API } diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/isolation/PluginDescTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/isolation/PluginDescTest.java index 3d8f7035953..950aa902101 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/isolation/PluginDescTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/isolation/PluginDescTest.java @@ -49,7 +49,7 @@ public void setUp() throws Exception { } @Test - public void testRegularPluginDesc() throws Exception { + public void testRegularPluginDesc() { PluginDesc<Connector> connectorDesc = new PluginDesc<>( Connector.class, regularVersion, @@ -76,7 +76,7 @@ public void testRegularPluginDesc() throws Exception { } @Test - public void testPluginDescWithSystemClassLoader() throws Exception { + public void testPluginDescWithSystemClassLoader() { String location = "classpath"; PluginDesc<SinkConnector> connectorDesc = new PluginDesc<>( SinkConnector.class, @@ -104,7 +104,7 @@ public void testPluginDescWithSystemClassLoader() throws Exception { } @Test - public void testPluginDescWithNullVersion() throws Exception { + public void testPluginDescWithNullVersion() { String nullVersion = "null"; PluginDesc<SourceConnector> connectorDesc = new PluginDesc<>( SourceConnector.class, @@ -130,7 +130,7 @@ public void testPluginDescWithNullVersion() throws Exception { } @Test - public void testPluginDescEquality() throws Exception { + public void testPluginDescEquality() { PluginDesc<Connector> connectorDescPluginPath = new PluginDesc<>( Connector.class, snaphotVersion, @@ -177,7 +177,7 @@ public void testPluginDescEquality() throws Exception { } @Test - public void testPluginDescComparison() throws Exception { + public void testPluginDescComparison() { PluginDesc<Connector> connectorDescPluginPath = new PluginDesc<>( Connector.class, regularVersion, diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/isolation/PluginUtilsTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/isolation/PluginUtilsTest.java index 191eec4449d..f9a2d8f4afa 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/isolation/PluginUtilsTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/isolation/PluginUtilsTest.java @@ -45,7 +45,7 @@ public void setUp() throws Exception { } @Test - public void testJavaLibraryClasses() throws Exception { + public void testJavaLibraryClasses() { assertFalse(PluginUtils.shouldLoadInIsolation("java.")); assertFalse(PluginUtils.shouldLoadInIsolation("java.lang.Object")); assertFalse(PluginUtils.shouldLoadInIsolation("java.lang.String")); @@ -64,13 +64,13 @@ public void testJavaLibraryClasses() throws Exception { } @Test - public void testThirdPartyClasses() throws Exception { + public void testThirdPartyClasses() { assertFalse(PluginUtils.shouldLoadInIsolation("org.slf4j.")); assertFalse(PluginUtils.shouldLoadInIsolation("org.slf4j.LoggerFactory")); } @Test - public void testConnectFrameworkClasses() throws Exception { + public void testConnectFrameworkClasses() { assertFalse(PluginUtils.shouldLoadInIsolation("org.apache.kafka.common.")); assertFalse(PluginUtils.shouldLoadInIsolation( "org.apache.kafka.common.config.AbstractConfig") @@ -120,7 +120,7 @@ public void testConnectFrameworkClasses() throws Exception { } @Test - public void testAllowedConnectFrameworkClasses() throws Exception { + public void testAllowedConnectFrameworkClasses() { assertTrue(PluginUtils.shouldLoadInIsolation("org.apache.kafka.connect.transforms.")); assertTrue(PluginUtils.shouldLoadInIsolation( "org.apache.kafka.connect.transforms.ExtractField") @@ -173,7 +173,7 @@ public void testAllowedConnectFrameworkClasses() throws Exception { } @Test - public void testClientConfigProvider() throws Exception { + public void testClientConfigProvider() { assertFalse(PluginUtils.shouldLoadInIsolation( "org.apache.kafka.common.config.provider.ConfigProvider") ); diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/rest/resources/ConnectorPluginsResourceTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/rest/resources/ConnectorPluginsResourceTest.java index ad360b66990..f9b5ad78d32 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/rest/resources/ConnectorPluginsResourceTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/rest/resources/ConnectorPluginsResourceTest.java @@ -196,29 +196,26 @@ private void expectPlugins() { public void testValidateConfigWithSingleErrorDueToMissingConnectorClassname() throws Throwable { herder.validateConnectorConfig(EasyMock.eq(partialProps)); - PowerMock.expectLastCall().andAnswer(new IAnswer<ConfigInfos>() { - @Override - public ConfigInfos answer() { - ConfigDef connectorConfigDef = ConnectorConfig.configDef(); - List<ConfigValue> connectorConfigValues = connectorConfigDef.validate(partialProps); - - Connector connector = new ConnectorPluginsResourceTestConnector(); - Config config = connector.validate(partialProps); - ConfigDef configDef = connector.config(); - Map<String, ConfigDef.ConfigKey> configKeys = configDef.configKeys(); - List<ConfigValue> configValues = config.configValues(); - - Map<String, ConfigDef.ConfigKey> resultConfigKeys = new HashMap<>(configKeys); - resultConfigKeys.putAll(connectorConfigDef.configKeys()); - configValues.addAll(connectorConfigValues); - - return AbstractHerder.generateResult( + PowerMock.expectLastCall().andAnswer((IAnswer<ConfigInfos>) () -> { + ConfigDef connectorConfigDef = ConnectorConfig.configDef(); + List<ConfigValue> connectorConfigValues = connectorConfigDef.validate(partialProps); + + Connector connector = new ConnectorPluginsResourceTestConnector(); + Config config = connector.validate(partialProps); + ConfigDef configDef = connector.config(); + Map<String, ConfigDef.ConfigKey> configKeys = configDef.configKeys(); + List<ConfigValue> configValues = config.configValues(); + + Map<String, ConfigDef.ConfigKey> resultConfigKeys = new HashMap<>(configKeys); + resultConfigKeys.putAll(connectorConfigDef.configKeys()); + configValues.addAll(connectorConfigValues); + + return AbstractHerder.generateResult( ConnectorPluginsResourceTestConnector.class.getName(), resultConfigKeys, configValues, Collections.singletonList("Test") - ); - } + ); }); PowerMock.replayAll(); @@ -244,29 +241,26 @@ public ConfigInfos answer() { public void testValidateConfigWithSimpleName() throws Throwable { herder.validateConnectorConfig(EasyMock.eq(props)); - PowerMock.expectLastCall().andAnswer(new IAnswer<ConfigInfos>() { - @Override - public ConfigInfos answer() { - ConfigDef connectorConfigDef = ConnectorConfig.configDef(); - List<ConfigValue> connectorConfigValues = connectorConfigDef.validate(props); + PowerMock.expectLastCall().andAnswer((IAnswer<ConfigInfos>) () -> { + ConfigDef connectorConfigDef = ConnectorConfig.configDef(); + List<ConfigValue> connectorConfigValues = connectorConfigDef.validate(props); - Connector connector = new ConnectorPluginsResourceTestConnector(); - Config config = connector.validate(props); - ConfigDef configDef = connector.config(); - Map<String, ConfigDef.ConfigKey> configKeys = configDef.configKeys(); - List<ConfigValue> configValues = config.configValues(); + Connector connector = new ConnectorPluginsResourceTestConnector(); + Config config = connector.validate(props); + ConfigDef configDef = connector.config(); + Map<String, ConfigDef.ConfigKey> configKeys = configDef.configKeys(); + List<ConfigValue> configValues = config.configValues(); - Map<String, ConfigDef.ConfigKey> resultConfigKeys = new HashMap<>(configKeys); - resultConfigKeys.putAll(connectorConfigDef.configKeys()); - configValues.addAll(connectorConfigValues); + Map<String, ConfigDef.ConfigKey> resultConfigKeys = new HashMap<>(configKeys); + resultConfigKeys.putAll(connectorConfigDef.configKeys()); + configValues.addAll(connectorConfigValues); - return AbstractHerder.generateResult( + return AbstractHerder.generateResult( ConnectorPluginsResourceTestConnector.class.getName(), resultConfigKeys, configValues, Collections.singletonList("Test") - ); - } + ); }); PowerMock.replayAll(); @@ -288,29 +282,26 @@ public ConfigInfos answer() { public void testValidateConfigWithAlias() throws Throwable { herder.validateConnectorConfig(EasyMock.eq(props)); - PowerMock.expectLastCall().andAnswer(new IAnswer<ConfigInfos>() { - @Override - public ConfigInfos answer() { - ConfigDef connectorConfigDef = ConnectorConfig.configDef(); - List<ConfigValue> connectorConfigValues = connectorConfigDef.validate(props); + PowerMock.expectLastCall().andAnswer((IAnswer<ConfigInfos>) () -> { + ConfigDef connectorConfigDef = ConnectorConfig.configDef(); + List<ConfigValue> connectorConfigValues = connectorConfigDef.validate(props); - Connector connector = new ConnectorPluginsResourceTestConnector(); - Config config = connector.validate(props); - ConfigDef configDef = connector.config(); - Map<String, ConfigDef.ConfigKey> configKeys = configDef.configKeys(); - List<ConfigValue> configValues = config.configValues(); + Connector connector = new ConnectorPluginsResourceTestConnector(); + Config config = connector.validate(props); + ConfigDef configDef = connector.config(); + Map<String, ConfigDef.ConfigKey> configKeys = configDef.configKeys(); + List<ConfigValue> configValues = config.configValues(); - Map<String, ConfigDef.ConfigKey> resultConfigKeys = new HashMap<>(configKeys); - resultConfigKeys.putAll(connectorConfigDef.configKeys()); - configValues.addAll(connectorConfigValues); + Map<String, ConfigDef.ConfigKey> resultConfigKeys = new HashMap<>(configKeys); + resultConfigKeys.putAll(connectorConfigDef.configKeys()); + configValues.addAll(connectorConfigValues); - return AbstractHerder.generateResult( + return AbstractHerder.generateResult( ConnectorPluginsResourceTestConnector.class.getName(), resultConfigKeys, configValues, Collections.singletonList("Test") - ); - } + ); }); PowerMock.replayAll(); @@ -332,29 +323,26 @@ public ConfigInfos answer() { public void testValidateConfigWithNonExistentName() throws Throwable { herder.validateConnectorConfig(EasyMock.eq(props)); - PowerMock.expectLastCall().andAnswer(new IAnswer<ConfigInfos>() { - @Override - public ConfigInfos answer() { - ConfigDef connectorConfigDef = ConnectorConfig.configDef(); - List<ConfigValue> connectorConfigValues = connectorConfigDef.validate(props); + PowerMock.expectLastCall().andAnswer((IAnswer<ConfigInfos>) () -> { + ConfigDef connectorConfigDef = ConnectorConfig.configDef(); + List<ConfigValue> connectorConfigValues = connectorConfigDef.validate(props); - Connector connector = new ConnectorPluginsResourceTestConnector(); - Config config = connector.validate(props); - ConfigDef configDef = connector.config(); - Map<String, ConfigDef.ConfigKey> configKeys = configDef.configKeys(); - List<ConfigValue> configValues = config.configValues(); + Connector connector = new ConnectorPluginsResourceTestConnector(); + Config config = connector.validate(props); + ConfigDef configDef = connector.config(); + Map<String, ConfigDef.ConfigKey> configKeys = configDef.configKeys(); + List<ConfigValue> configValues = config.configValues(); - Map<String, ConfigDef.ConfigKey> resultConfigKeys = new HashMap<>(configKeys); - resultConfigKeys.putAll(connectorConfigDef.configKeys()); - configValues.addAll(connectorConfigValues); + Map<String, ConfigDef.ConfigKey> resultConfigKeys = new HashMap<>(configKeys); + resultConfigKeys.putAll(connectorConfigDef.configKeys()); + configValues.addAll(connectorConfigValues); - return AbstractHerder.generateResult( + return AbstractHerder.generateResult( ConnectorPluginsResourceTestConnector.class.getName(), resultConfigKeys, configValues, Collections.singletonList("Test") - ); - } + ); }); PowerMock.replayAll(); @@ -372,29 +360,26 @@ public ConfigInfos answer() { public void testValidateConfigWithNonExistentAlias() throws Throwable { herder.validateConnectorConfig(EasyMock.eq(props)); - PowerMock.expectLastCall().andAnswer(new IAnswer<ConfigInfos>() { - @Override - public ConfigInfos answer() { - ConfigDef connectorConfigDef = ConnectorConfig.configDef(); - List<ConfigValue> connectorConfigValues = connectorConfigDef.validate(props); + PowerMock.expectLastCall().andAnswer((IAnswer<ConfigInfos>) () -> { + ConfigDef connectorConfigDef = ConnectorConfig.configDef(); + List<ConfigValue> connectorConfigValues = connectorConfigDef.validate(props); - Connector connector = new ConnectorPluginsResourceTestConnector(); - Config config = connector.validate(props); - ConfigDef configDef = connector.config(); - Map<String, ConfigDef.ConfigKey> configKeys = configDef.configKeys(); - List<ConfigValue> configValues = config.configValues(); + Connector connector = new ConnectorPluginsResourceTestConnector(); + Config config = connector.validate(props); + ConfigDef configDef = connector.config(); + Map<String, ConfigDef.ConfigKey> configKeys = configDef.configKeys(); + List<ConfigValue> configValues = config.configValues(); - Map<String, ConfigDef.ConfigKey> resultConfigKeys = new HashMap<>(configKeys); - resultConfigKeys.putAll(connectorConfigDef.configKeys()); - configValues.addAll(connectorConfigValues); + Map<String, ConfigDef.ConfigKey> resultConfigKeys = new HashMap<>(configKeys); + resultConfigKeys.putAll(connectorConfigDef.configKeys()); + configValues.addAll(connectorConfigValues); - return AbstractHerder.generateResult( + return AbstractHerder.generateResult( ConnectorPluginsResourceTestConnector.class.getName(), resultConfigKeys, configValues, Collections.singletonList("Test") - ); - } + ); }); PowerMock.replayAll(); @@ -546,7 +531,7 @@ public ConfigDef config() { @Override public List<Object> validValues(String name, Map<String, Object> parsedConfig) { - return Arrays.<Object>asList(1, 2, 3); + return Arrays.asList(1, 2, 3); } @Override @@ -558,7 +543,7 @@ public boolean visible(String name, Map<String, Object> parsedConfig) { private static class ListRecommender implements Recommender { @Override public List<Object> validValues(String name, Map<String, Object> parsedConfig) { - return Arrays.<Object>asList("a", "b", "c"); + return Arrays.asList("a", "b", "c"); } @Override diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/standalone/StandaloneHerderTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/standalone/StandaloneHerderTest.java index b98c15e7014..9763fa9dd58 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/standalone/StandaloneHerderTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/runtime/standalone/StandaloneHerderTest.java @@ -140,7 +140,7 @@ public void testCreateSourceConnector() throws Exception { } @Test - public void testCreateConnectorFailedBasicValidation() throws Exception { + public void testCreateConnectorFailedBasicValidation() { // Basic validation should be performed and return an error, but should still evaluate the connector's config connector = PowerMock.createMock(BogusSourceConnector.class); @@ -172,7 +172,7 @@ public void testCreateConnectorFailedBasicValidation() throws Exception { } @Test - public void testCreateConnectorFailedCustomValidation() throws Exception { + public void testCreateConnectorFailedCustomValidation() { connector = PowerMock.createMock(BogusSourceConnector.class); Connector connectorMock = PowerMock.createMock(SourceConnector.class); @@ -604,7 +604,7 @@ public void testCorruptConfig() { PowerMock.verifyAll(); } - private void expectAdd(SourceSink sourceSink) throws Exception { + private void expectAdd(SourceSink sourceSink) { Map<String, String> connectorProps = connectorConfig(sourceSink); ConnectorConfig connConfig = sourceSink == SourceSink.SOURCE ? diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/storage/KafkaConfigBackingStoreTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/storage/KafkaConfigBackingStoreTest.java index ed62d9b6f01..8a02c1e68b7 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/storage/KafkaConfigBackingStoreTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/storage/KafkaConfigBackingStoreTest.java @@ -761,7 +761,7 @@ private void expectConfigure() throws Exception { // If non-empty, deserializations should be a LinkedHashMap private void expectStart(final List<ConsumerRecord<String, byte[]>> preexistingRecords, - final Map<byte[], Struct> deserializations) throws Exception { + final Map<byte[], Struct> deserializations) { storeLog.start(); PowerMock.expectLastCall().andAnswer(new IAnswer<Object>() { @Override diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/storage/KafkaOffsetBackingStoreTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/storage/KafkaOffsetBackingStoreTest.java index 8cd2f0b0560..36649a3da71 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/storage/KafkaOffsetBackingStoreTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/storage/KafkaOffsetBackingStoreTest.java @@ -111,7 +111,7 @@ @Before public void setUp() throws Exception { - store = PowerMock.createPartialMockAndInvokeDefaultConstructor(KafkaOffsetBackingStore.class, new String[]{"createKafkaBasedLog"}); + store = PowerMock.createPartialMockAndInvokeDefaultConstructor(KafkaOffsetBackingStore.class, "createKafkaBasedLog"); } @Test diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/storage/OffsetStorageWriterTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/storage/OffsetStorageWriterTest.java index 5a671ca6a73..3aad0d5314b 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/storage/OffsetStorageWriterTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/storage/OffsetStorageWriterTest.java @@ -165,7 +165,7 @@ public void testFlushFailureReplacesOffsets() throws Exception { } @Test(expected = ConnectException.class) - public void testAlreadyFlushing() throws Exception { + public void testAlreadyFlushing() { @SuppressWarnings("unchecked") final Callback<Void> callback = PowerMock.createMock(Callback.class); // Trigger the send, but don't invoke the callback so we'll still be mid-flush diff --git a/connect/runtime/src/test/java/org/apache/kafka/connect/util/KafkaBasedLogTest.java b/connect/runtime/src/test/java/org/apache/kafka/connect/util/KafkaBasedLogTest.java index 6d5efe8a378..53ca0dccd5f 100644 --- a/connect/runtime/src/test/java/org/apache/kafka/connect/util/KafkaBasedLogTest.java +++ b/connect/runtime/src/test/java/org/apache/kafka/connect/util/KafkaBasedLogTest.java @@ -132,7 +132,7 @@ public void onCompletion(Throwable error, ConsumerRecord<String, String> record) }; @Before - public void setUp() throws Exception { + public void setUp() { store = PowerMock.createPartialMock(KafkaBasedLog.class, new String[]{"createConsumer", "createProducer"}, TOPIC, PRODUCER_PROPS, CONSUMER_PROPS, consumedCallback, time, initializer); consumer = new MockConsumer<>(OffsetResetStrategy.EARLIEST); ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Clean up connect unit tests code > -------------------------------- > > Key: KAFKA-7431 > URL: https://issues.apache.org/jira/browse/KAFKA-7431 > Project: Kafka > Issue Type: Task > Reporter: Srinivas Reddy > Assignee: Srinivas Reddy > Priority: Minor > Fix For: 2.2.0 > > > JUnit test methods in connect module declared as `throws SomeException` even > though there is no chance of getting an exception in method body. > Those things should be removed from declaration to make the code more > readable. -- This message was sent by Atlassian JIRA (v7.6.3#76005)